[PATCH 1/1] ARM: cns3xxx: Fixup the missing second parameter to addruart macro to allow them to build.

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Mon Nov 15 04:01:46 EST 2010


Hello,

On Mon, Nov 15, 2010 at 06:01:54AM +0800, mkl0301 at gmail.com wrote:
> From: Mac Lin <mkl0301 at gmail.com>
> 
> Fixup the missing second parameter to addruart macro to allow them to build,
> according to to commit 0e17226f7cd289504724466f4298abc9bdfca3fe.
> 
> Enabling DEBUG in head.S would cause:
> rch/arm/boot/compressed/head.S: Assembler messages:
> arch/arm/boot/compressed/head.S:1037: Error: too many positional arguments
> arch/arm/boot/compressed/head.S:1055: Error: too many positional arguments
> 
> Signed-off-by: Mac Lin <mkl0301 at gmail.com>
> ---
>  arch/arm/mach-cns3xxx/include/mach/debug-macro.S |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/arm/mach-cns3xxx/include/mach/debug-macro.S b/arch/arm/mach-cns3xxx/include/mach/debug-macro.S
> index d16ce7e..9b50442 100644
> --- a/arch/arm/mach-cns3xxx/include/mach/debug-macro.S
> +++ b/arch/arm/mach-cns3xxx/include/mach/debug-macro.S
> @@ -10,7 +10,7 @@
>   * published by the Free Software Foundation.
>   */
>  
> -		.macro	addruart,rx
> +		.macro	addruart,rx,rtmp
>  		mrc	p15, 0, \rx, c1, c0
>  		tst	\rx, #1			@ MMU enabled?
>  		moveq	\rx,      #0x10000000
This doesn't apply to current Russell's (or Linus') tree.  (And is wrong
there BTW.)  Which version does your patch apply to?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |



More information about the linux-arm-kernel mailing list