Dear Vipin KUMAR,
In message 1260955110-5656-8-git-send-email-vipin.kumar@st.com you wrote:
Signed-off-by: Vipin vipin.kumar@st.com
MAKEALL | 1 + Makefile | 3 + board/spear/spear310/Makefile | 52 +++++ board/spear/spear310/config.mk | 42 ++++ board/spear/spear310/spr310_board.c | 58 ++++++ include/configs/spear310.h | 353 +++++++++++++++++++++++++++++++++++
The commit message seems wrong. You just add a new board, not a new SoC.
diff --git a/board/spear/spear310/config.mk b/board/spear/spear310/config.mk new file mode 100755 index 0000000..8b7aa05 --- /dev/null +++ b/board/spear/spear310/config.mk
...
+ifeq ($(CONSOLE),USB) +PLATFORM_RELFLAGS += -DCONFIG_SPEAR_USBTTY +endif
No trailing empty lines, please.
+int board_nand_init(struct nand_chip *nand) +{
- struct misc_regs *const misc_regs_p =
(struct misc_regs *)CONFIG_SPEAR_MISCBASE;- if (((readl(&misc_regs_p->auto_cfg_reg) & MISC_SOCCFGMSK) ==
MISC_SOCCFG30) ||((readl(&misc_regs_p->auto_cfg_reg) & MISC_SOCCFGMSK) ==MISC_SOCCFG31)) {return spear_nand_init(nand);- } else {
return -1;- }
No else, unindent the "return -1;".
+/*
- Ethernet MAC driver configuration
- */
+/* #define CONFIG_SPEAR_SMII */ +/* #define CONFIG_SPEARMAC */ +/* #define CONFIG_ETHAUTONEG */ +/* #define CONFIG_ETHDEBUG */
Don't add dead code (fix globally).
+#define CONFIG_SYS_SERIAL1 0xB2000000 +#define CONFIG_SYS_SERIAL2 0xB2080000 +#define CONFIG_SYS_SERIAL3 0xB2100000 +#define CONFIG_SYS_SERIAL4 0xB2180000 +#define CONFIG_SYS_SERIAL5 0xB2200000 +#define CONFIG_PL01x_PORTS { (void *)CONFIG_SYS_SERIAL0, \
(void *)CONFIG_SYS_SERIAL1, \(void *)CONFIG_SYS_SERIAL2, \(void *)CONFIG_SYS_SERIAL3, \(void *)CONFIG_SYS_SERIAL4, \(void *)CONFIG_SYS_SERIAL5}
Indentation / avertical alignment by TABs, please. [Globally.]
+#define CONFIG_BOOTARGS "console=ttyS0 mem=128M " \
...
+#define CONFIG_ENV_ADDR (CONFIG_SYS_MONITOR_BASE + \
etc.: Lines too long.
Best regards,
Wolfgang Denk