[U-Boot] [PATCH 3/3] exynos5250: Add arndale board support

[Wolfgang Denk]

Dear Inderpal Singh,

In message <1364290670-7443-4-git-send-email-inderpal.singh at linaro.org> you wrote:
> Arndale board is based on samsung's exynos5250 soc.


WARNING: Avoid CamelCase: <DMC_MEMCONFIGx_CHIP_MAP_INTERLEAVED>
#618: FILE: board/samsung/arndale/clock_init.c:213:
+		.memconfig = DMC_MEMCONFIGx_CHIP_MAP_INTERLEAVED |

WARNING: Avoid CamelCase: <DMC_MEMBASECONFIGx_CHIP_BASE>
#1860: FILE: board/samsung/arndale/setup.h:102:
+#define DMC_MEMBASECONFIGx_CHIP_BASE(x)         (x << 16)

WARNING: Avoid CamelCase: <DMC_MEMBASECONFIGx_CHIP_MASK>
#1861: FILE: board/samsung/arndale/setup.h:103:
+#define DMC_MEMBASECONFIGx_CHIP_MASK(x)         (x << 0)

ERROR: Macros with complex values should be enclosed in parenthesis
#2116: FILE: board/samsung/arndale/setup.h:358:
+#define SCLK_SRC_ISP_VAL	(SPI1_ISP_SEL << 4) \
+				| (SPI0_ISP_SEL << 0)

ERROR: Macros with complex values should be enclosed in parenthesis
#2122: FILE: board/samsung/arndale/setup.h:364:
+#define SCLK_DIV_ISP_VAL	(SPI1_ISP_RATIO << 12) \
+				| (SPI0_ISP_RATIO << 0)

ERROR: Macros with complex values should be enclosed in parenthesis
#2416: FILE: include/configs/arndale.h:29:
+#define CONFIG_DEFAULT_DEVICE_TREE	exynos5250-arndale

total: 3 errors, 3 warnings, 2235 lines checked

/home/wd/Mail/U-Boot/7161 has style problems, please review.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The manager will be continually amazed that policies he took for com-
mon knowledge are totally unknown by some member of his  team.  Since
his fundamental job is to keep everybody going in the same direction,
his chief daily task will be communication, not decision-making.
                              - Fred Brooks, "The Mythical Man Month"