[PATCH] msm: fix debug-macro.S build failure
Russell King - ARM Linux
linux at arm.linux.org.uk
Fri Oct 29 18:28:44 EDT 2010
On Fri, Oct 29, 2010 at 03:21:22PM -0700, Daniel Walker wrote:
> On Fri, 2010-10-29 at 23:14 +0100, Russell King - ARM Linux wrote:
>
> > Your original patch is unsuitable as it leaves the values uninitialized.
> >
>
> All I'm really concerned about is fixing the build issue during this
> merge window. But I'll send another patch with your suggestions added.
As we have had a complaint from Linus about pointless churn, and this
approach causes unnecessary churn, I will refuse to apply this. Fix
the thing properly (it's not hard) or don't bother trying to fix it at
all.
All you need do is to use your initial patch, but change the addruart to
the following:
.macro addruart, rp, rv
mov rv, #0xff000000
orr rv, rv, #0x00f00000
.endm
and you commit the result. Problem fixed in one go, both from the
compile point of view _and_ the runtime point of view. It's really
not difficult.
More information about the linux-arm-kernel
mailing list