2010-02-22 15:33:48

by Steven J. Magnani

[permalink] [raw]
Subject: [PATCH][trivial] microblaze: Fix out_le32() macro

Trailing semicolon causes compilation involving out_le32() to fail.

Signed-off-by: Steven J. Magnani <[email protected]>
diff -uprN a/arch/microblaze/include/asm/io.h b/arch/microblaze/include/asm/io.h
--- a/arch/microblaze/include/asm/io.h 2010-02-22 09:21:14.000000000 -0600
+++ b/arch/microblaze/include/asm/io.h 2010-02-22 09:21:44.000000000 -0600
@@ -217,7 +217,7 @@ static inline void __iomem *__ioremap(ph
* Little endian
*/

-#define out_le32(a, v) __raw_writel(__cpu_to_le32(v), (a));
+#define out_le32(a, v) __raw_writel(__cpu_to_le32(v), (a))
#define out_le16(a, v) __raw_writew(__cpu_to_le16(v), (a))

#define in_le32(a) __le32_to_cpu(__raw_readl(a))


2010-02-23 07:38:23

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH][trivial] microblaze: Fix out_le32() macro

Added to next branch.

Thanks,
Michal

Steven J. Magnani wrote:
> Trailing semicolon causes compilation involving out_le32() to fail.
>
> Signed-off-by: Steven J. Magnani <[email protected]>
> diff -uprN a/arch/microblaze/include/asm/io.h b/arch/microblaze/include/asm/io.h
> --- a/arch/microblaze/include/asm/io.h 2010-02-22 09:21:14.000000000 -0600
> +++ b/arch/microblaze/include/asm/io.h 2010-02-22 09:21:44.000000000 -0600
> @@ -217,7 +217,7 @@ static inline void __iomem *__ioremap(ph
> * Little endian
> */
>
> -#define out_le32(a, v) __raw_writel(__cpu_to_le32(v), (a));
> +#define out_le32(a, v) __raw_writel(__cpu_to_le32(v), (a))
> #define out_le16(a, v) __raw_writew(__cpu_to_le16(v), (a))
>
> #define in_le32(a) __le32_to_cpu(__raw_readl(a))
>


--
Michal Simek, Ing. (M.Eng)
w: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian