Thread (3 messages) 3 messages, 3 authors, 2015-06-09

[PATCH v2 1/2] ARM: Add cpu_resume_arm() for firmwares that resume in ARM state

From: khilman@kernel.org (Kevin Hilman)
Date: 2015-06-08 22:03:10
Also in: linux-arm-msm, lkml

Stephen Boyd [off-list ref] writes:
On 06/08/2015 02:33 PM, Kevin Hilman wrote:
quoted
Stephen Boyd [off-list ref] writes:
quoted
On 06/03, Russell King - ARM Linux wrote:
quoted
On Tue, Jun 02, 2015 at 12:12:57PM -0700, Stephen Boyd wrote:
quoted
Some platforms always enter the kernel in the ARM state even if
the kernel is compiled for THUMB2. Add a small wrapper on top of
cpu_resume() that switches into THUMB2 state.

This fixes a problem reported by Kevin Hilman on next-20150601
where the ifc6410 fails to boot a THUMB2 kernel because the
platform's firmware always enters the kernel in ARM mode from
deep idle states.
I think this bit of the commit message isn't quite correct: this will
only resolve the boot problem with a T2 kernel when the failing platform
is converted to use the cpu_resume_arm() entry point.

Apart from that, the patch looks good to me, thanks.
Sure, it could say "This provides the functionality to fix a problem
reported by..."
Stephen, are you planning to submit this through Russell's patch
tracker?  or...

Russell, should we take this through arm-soc along with PATCH 2/2 which
depends on this one?
I'm happy to see it be applied by arm-soc maintainers directly given
that Russell said "the patch looks good to me". The commit text needs a
slight reword here though, so I can resubmit the patches to arm-soc if
you like.
Go ahead and respin/resubmit, and I can get them ready, but I'll wait
for an official word or acked-by from Russell before taking them both
through arm-soc.

Kevin
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help