[U-Boot] [PATCH 6/9] SPEAr300 SoC support added

[Vipin Kumar]

Dear Wolfgang,

>>
>> Signed-off-by: Vipin <vipin.kumar at st.com>
> ...
>> diff --git a/board/spear/spear300/config.mk b/board/spear/spear300/config.mk
>> new file mode 100755
>> index 0000000..90a6cf9
>> --- /dev/null
>> +++ b/board/spear/spear300/config.mk
> ...
>> +PLATFORM_RELFLAGS += -DCONFIG_SPEAR_USBTTY
>> +endif
>> +
>
> Please do not add trailing empty lines. [Please check globally.]

Ok. Corrected globally

>> diff --git a/board/spear/spear300/spr300_board.c b/board/spear/spear300/spr300_board.c
>> new file mode 100755
>> index 0000000..97de0be
>> --- /dev/null
>> +++ b/board/spear/spear300/spr300_board.c
>
> Usually the file name is the same as the board config name.

Ok. filename corrected

>> +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;
>
> "else" not needed. Unindent the "return -1;".

Ok.
Removed else and unindented

>> diff --git a/include/configs/spear300.h b/include/configs/spear300.h
>> new file mode 100755
>> index 0000000..59b1f45
>> --- /dev/null
>> +++ b/include/configs/spear300.h
> ...
>> +/*
>> + * Ethernet MAC driver configuration
>> + */
>> +/* #define CONFIG_SPEARMAC */
>> +/* #define CONFIG_ETHAUTONEG */
>> +/* #define CONFIG_ETHDEBUG */
>
> Don't add dead code.

Ok. dead code removed

>> +#define CONFIG_SYS_FIFO_BASE                 (0xE1000800)
>> +#define CONFIG_USBD_PRODUCT_NAME             "SPEAr SoC"
>> +#define CONFIG_USBD_MANUFACTURER             "ST Microelectronics"
>> +
>> +#define CONFIG_EXTRA_ENV_SETTINGS               "usbtty=cdc_acm"
>
> Vertial alignment by TABs only, please.

Ok. Used tabs for vertical alignment

>> +/* #define CONFIG_SYS_I2C_MULTI_EEPROMS */
>> +/* #define CONFIG_SYS_I2C_EEPROM_ADDR_LEN        0x1 */
>
> Don't add dead code. Please fix globally.

Ok. Corrected globally

>> +/*
>> + * SMI driver configuration
>> + */
>> +#define CONFIG_SPEARSMI
>> +#undef  CONFIG_SYS_NO_FLASH
>
> Don't undef what is not defined anyway.

Ok. undef removed

>> +#define CONFIG_SYS_HZ                                (26000)
>
> CONFIG_SYS_HZ _must_ be defined as 1000.

Could not understand the reason why it is necessary

>> +/* #define CONFIG_ETHADDR                    00:11:22:33:44:55 */
>> +
>> +/* #define CONFIG_NETMASK                    255.255.255.0 */
>> +/* #define CONFIG_IPADDR                     192.168.1.10 */
>> +/* #define CONFIG_SERVERIP                   192.168.1.1 */
>> +/* #define CONFIG_GATEWAYIP                  192.168.1.1 */
>
> Get rid of all this stuff.

Ok. Dead code removed

>> +#define CONFIG_ENV_ADDR                              (CONFIG_SYS_MONITOR_BASE + \
>> +                                             CONFIG_SYS_MONITOR_LEN)
>> +
>> +#define CONFIG_BOOTARGS                              "console=ttyS0 mem=128M "  \
>
> Lines too long.

Ok. Corrected.

Best Regards
Vipin