[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