[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