[U-Boot] [PATCH 0/8] ARM: Add Seagate GoFlexHome support

Most of the files except goflexhomemenu.c are direct copies of the dockstar files with minor modifications.
goflexhomemenu.c has code which uses the menu functionality of u-boot and present the user with an option to choose the various bootable options.
Suriyan Ramasami (8): Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support
MAINTAINERS | 4 + board/Seagate/goflexhome/Makefile | 51 ++++ board/Seagate/goflexhome/goflexhome.c | 187 ++++++++++++++ board/Seagate/goflexhome/goflexhome.h | 50 ++++ board/Seagate/goflexhome/goflexhomemenu.c | 381 +++++++++++++++++++++++++++++ board/Seagate/goflexhome/kwbimage.cfg | 168 +++++++++++++ boards.cfg | 1 + include/configs/goflexhome.h | 140 +++++++++++ 8 files changed, 982 insertions(+), 0 deletions(-) create mode 100644 board/Seagate/goflexhome/Makefile create mode 100644 board/Seagate/goflexhome/goflexhome.c create mode 100644 board/Seagate/goflexhome/goflexhome.h create mode 100644 board/Seagate/goflexhome/goflexhomemenu.c create mode 100644 board/Seagate/goflexhome/kwbimage.cfg create mode 100644 include/configs/goflexhome.h

Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- MAINTAINERS | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS index 1614b91..6292a58 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -847,6 +847,10 @@ Sricharan R r.sricharan@ti.com omap4_sdp4430 ARM ARMV7 (OMAP4xx SoC) omap5_evm ARM ARMV7 (OMAP5xx Soc)
+Suriyan Ramasami suriyan.r@gmail.com + + goflexhome ARM926EJS (Kirkwood SoC) + Thierry Reding thierry.reding@avionic-design.de
plutux Tegra20 (ARM7 & A9 Dual Core)

Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- boards.cfg | 1 + 1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/boards.cfg b/boards.cfg index 5fc70be..d6f8593 100644 --- a/boards.cfg +++ b/boards.cfg @@ -184,6 +184,7 @@ rd6281a arm arm926ejs - Marvell sheevaplug arm arm926ejs - Marvell kirkwood ib62x0 arm arm926ejs ib62x0 raidsonic kirkwood dockstar arm arm926ejs - Seagate kirkwood +goflexhome arm arm926ejs - Seagate kirkwood tk71 arm arm926ejs tk71 karo kirkwood devkit3250 arm arm926ejs devkit3250 timll lpc32xx jadecpu arm arm926ejs jadecpu syteco mb86r0x

Start with dockstar file and change for GoFlexHome
Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- board/Seagate/goflexhome/Makefile | 51 +++++++++++++++++++++++++++++++++++++ 1 files changed, 51 insertions(+), 0 deletions(-) create mode 100644 board/Seagate/goflexhome/Makefile
diff --git a/board/Seagate/goflexhome/Makefile b/board/Seagate/goflexhome/Makefile new file mode 100644 index 0000000..dc06137 --- /dev/null +++ b/board/Seagate/goflexhome/Makefile @@ -0,0 +1,51 @@ +# +# Copyright (C) 2013 Suriyan Ramasami suriyan.r@gmail.com +# +# Based on dockstar/Makefile originally written by +# Copyright (C) 2010 Eric C. Cooper ecc@cmu.edu +# +# Based on sheevaplug/Makefile originally written by +# Prafulla Wadaskar prafulla@marvell.com +# (C) Copyright 2009 +# Marvell Semiconductor <www.marvell.com> +# +# See file CREDITS for list of people who contributed to this +# project. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License as +# published by the Free Software Foundation; either version 2 of +# the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, +# MA 02110-1301 USA +# + +include $(TOPDIR)/config.mk + +LIB = $(obj)lib$(BOARD).o + +COBJS := goflexhome.o goflexhomemenu.o + +SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c) +OBJS := $(addprefix $(obj),$(COBJS)) +SOBJS := $(addprefix $(obj),$(SOBJS)) + +$(LIB): $(obj).depend $(OBJS) $(SOBJS) + $(call cmd_link_o_target, $(OBJS) $(SOBJS)) + +######################################################################### + +# defines $(obj).depend target +include $(SRCTREE)/rules.mk + +sinclude $(obj).depend + +#########################################################################

Start with dockstar file and change for GoFlexHome
Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- board/Seagate/goflexhome/goflexhome.c | 187 +++++++++++++++++++++++++++++++++ 1 files changed, 187 insertions(+), 0 deletions(-) create mode 100644 board/Seagate/goflexhome/goflexhome.c
diff --git a/board/Seagate/goflexhome/goflexhome.c b/board/Seagate/goflexhome/goflexhome.c new file mode 100644 index 0000000..e990b16 --- /dev/null +++ b/board/Seagate/goflexhome/goflexhome.c @@ -0,0 +1,187 @@ +/* + * Copyright (C) 2013 Suriyan Ramasami suriyan.r@gmail.com + * + * Based on dockstar.c originally written by + * Copyright (C) 2010 Eric C. Cooper ecc@cmu.edu + * + * Based on sheevaplug.c originally written by + * Prafulla Wadaskar prafulla@marvell.com + * (C) Copyright 2009 + * Marvell Semiconductor <www.marvell.com> + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, + * MA 02110-1301 USA + */ + +#include <common.h> +#include <miiphy.h> +#include <asm/arch/kirkwood.h> +#include <asm/arch/mpp.h> +#include <asm/arch/cpu.h> +#include <asm/io.h> +#include "goflexhome.h" + + +DECLARE_GLOBAL_DATA_PTR; + +int board_early_init_f(void) +{ + /* + * default gpio configuration + * There are maximum 64 gpios controlled through 2 sets of registers + * the below configuration configures mainly initial LED status + */ + kw_config_gpio(GOFLEXHOME_OE_VAL_LOW, + GOFLEXHOME_OE_VAL_HIGH, + GOFLEXHOME_OE_LOW, GOFLEXHOME_OE_HIGH); + + /* Multi-Purpose Pins Functionality configuration */ + static const u32 kwmpp_config[] = { + MPP0_NF_IO2, + MPP1_NF_IO3, + MPP2_NF_IO4, + MPP3_NF_IO5, + MPP4_NF_IO6, + MPP5_NF_IO7, + MPP6_SYSRST_OUTn, + MPP7_GPO, + MPP8_UART0_RTS, + MPP9_UART0_CTS, + MPP10_UART0_TXD, + MPP11_UART0_RXD, + MPP12_SD_CLK, + MPP13_SD_CMD, + MPP14_SD_D0, + MPP15_SD_D1, + MPP16_SD_D2, + MPP17_SD_D3, + MPP18_NF_IO0, + MPP19_NF_IO1, + MPP20_GPIO, + MPP21_GPIO, + MPP22_GPIO, + MPP23_GPIO, + MPP24_GPIO, + MPP25_GPIO, + MPP26_GPIO, + MPP27_GPIO, + MPP28_GPIO, + MPP29_TSMP9, + MPP30_GPIO, + MPP31_GPIO, + MPP32_GPIO, + MPP33_GPIO, + MPP34_GPIO, + MPP35_GPIO, + MPP36_GPIO, + MPP37_GPIO, + MPP38_GPIO, + MPP39_GPIO, + MPP40_GPIO, + MPP41_GPIO, + MPP42_GPIO, + MPP43_GPIO, + MPP44_GPIO, + MPP45_GPIO, + MPP46_GPIO, + MPP47_GPIO, + MPP48_GPIO, + MPP49_GPIO, + 0 + }; + kirkwood_mpp_conf(kwmpp_config, NULL); + return 0; +} + +int board_init(void) +{ + /* + * arch number of board + */ + gd->bd->bi_arch_number = MACH_TYPE_GOFLEXHOME; + + /* address of boot parameters */ + gd->bd->bi_boot_params = kw_sdram_bar(0) + 0x100; + + return 0; +} + +#ifdef CONFIG_RESET_PHY_R +/* Configure and enable MV88E1116 PHY */ +void reset_phy(void) +{ + u16 reg; + u16 devadr; + char *name = "egiga0"; + + if (miiphy_set_current_dev(name)) + return; + + /* command to read PHY dev address */ + if (miiphy_read(name, 0xEE, 0xEE, (u16 *) &devadr)) { + printf("Err..%s could not read PHY dev address\n", + __FUNCTION__); + return; + } + + /* + * Enable RGMII delay on Tx and Rx for CPU port + * Ref: sec 4.7.2 of chip datasheet + */ + miiphy_write(name, devadr, MV88E1116_PGADR_REG, 2); + miiphy_read(name, devadr, MV88E1116_MAC_CTRL_REG, ®); + reg |= (MV88E1116_RGMII_RXTM_CTRL | MV88E1116_RGMII_TXTM_CTRL); + miiphy_write(name, devadr, MV88E1116_MAC_CTRL_REG, reg); + miiphy_write(name, devadr, MV88E1116_PGADR_REG, 0); + + /* reset the phy */ + miiphy_reset(name, devadr); + + printf("88E1116 Initialized on %s\n", name); +} +#endif /* CONFIG_RESET_PHY_R */ + +#define GREEN_LED (1 << 14) +#define ORANGE_LED (1 << 15) +#define BOTH_LEDS (GREEN_LED | ORANGE_LED) +#define NEITHER_LED 0 + +static void set_leds(u32 leds, u32 blinking) +{ + struct kwgpio_registers *r = (struct kwgpio_registers *)KW_GPIO1_BASE; + u32 oe = readl(&r->oe) | BOTH_LEDS; + writel(oe & ~leds, &r->oe); /* active low */ + u32 bl = readl(&r->blink_en) & ~BOTH_LEDS; + writel(bl | blinking, &r->blink_en); +} + +void show_boot_progress(int val) +{ + switch (val) { + case BOOTSTAGE_ID_RUN_OS: /* booting Linux */ + set_leds(BOTH_LEDS, NEITHER_LED); + break; + case BOOTSTAGE_ID_NET_ETH_START: /* Ethernet initialization */ + set_leds(GREEN_LED, GREEN_LED); + break; + default: + if (val < 0) /* error */ + set_leds(ORANGE_LED, ORANGE_LED); + break; + } +}

Start with dockstar file and change for GoFlexHome
Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- board/Seagate/goflexhome/goflexhome.h | 50 +++++++++++++++++++++++++++++++++ 1 files changed, 50 insertions(+), 0 deletions(-) create mode 100644 board/Seagate/goflexhome/goflexhome.h
diff --git a/board/Seagate/goflexhome/goflexhome.h b/board/Seagate/goflexhome/goflexhome.h new file mode 100644 index 0000000..730e4c9 --- /dev/null +++ b/board/Seagate/goflexhome/goflexhome.h @@ -0,0 +1,50 @@ +/* + * Copyright (C) 2013 Suriyan Ramasami suriyan.r@gmail.com + * + * Based on dockstar.h originally written by + * Copyright (C) 2010 Eric C. Cooper ecc@cmu.edu + * + * Based on sheevaplug.h originally written by + * Prafulla Wadaskar prafulla@marvell.com + * (C) Copyright 2009 + * Marvell Semiconductor <www.marvell.com> + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, + * MA 02110-1301 USA + */ + +#ifndef __GOFLEXHOME_H +#define __GOFLEXHOME_H + +#define GOFLEXHOME_OE_LOW (~(0)) +#define GOFLEXHOME_OE_HIGH (~(0)) +#define GOFLEXHOME_OE_VAL_LOW (1 << 29) /* USB_PWEN low */ +#define GOFLEXHOME_OE_VAL_HIGH (1 << 17) /* LED pin high */ + +/* PHY related */ +#define MV88E1116_LED_FCTRL_REG 10 +#define MV88E1116_CPRSP_CR3_REG 21 +#define MV88E1116_MAC_CTRL_REG 21 +#define MV88E1116_PGADR_REG 22 +#define MV88E1116_RGMII_TXTM_CTRL (1 << 4) +#define MV88E1116_RGMII_RXTM_CTRL (1 << 5) + +/* SATA related */ + + +#endif /* __GOFLEXHOME_H */

Start with dockstar file and change for GoFlexHome
Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- board/Seagate/goflexhome/kwbimage.cfg | 168 +++++++++++++++++++++++++++++++++ 1 files changed, 168 insertions(+), 0 deletions(-) create mode 100644 board/Seagate/goflexhome/kwbimage.cfg
diff --git a/board/Seagate/goflexhome/kwbimage.cfg b/board/Seagate/goflexhome/kwbimage.cfg new file mode 100644 index 0000000..e984d72 --- /dev/null +++ b/board/Seagate/goflexhome/kwbimage.cfg @@ -0,0 +1,168 @@ +# +# Copyright (C) 2013 Suriyan Ramasami suriyan.r@gmail.com +# +# Based on dockstar/kwbimage.cfg originally written by +# Copyright (C) 2010 Eric C. Cooper ecc@cmu.edu +# +# Based on sheevaplug/kwbimage.cfg originally written by +# Prafulla Wadaskar prafulla@marvell.com +# (C) Copyright 2009 +# Marvell Semiconductor <www.marvell.com> +# +# See file CREDITS for list of people who contributed to this +# project. +# +# This program is free software; you can redistribute it and/or +# modify it under the terms of the GNU General Public License as +# published by the Free Software Foundation; either version 2 of +# the License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, +# MA 02110-1301 USA +# +# Refer docs/README.kwimage for more details about how-to configure +# and create kirkwood boot image +# + +# Boot Media configurations +BOOT_FROM nand +NAND_ECC_MODE default +NAND_PAGE_SIZE 0x0800 + +# SOC registers configuration using bootrom header extension +# Maximum KWBIMAGE_MAX_CONFIG configurations allowed + +# Configure RGMII-0 interface pad voltage to 1.8V +DATA 0xFFD100e0 0x1b1b1b9b + +#Dram initalization for SINGLE x16 CL=5 @ 400MHz +DATA 0xFFD01400 0x43000c30 # DDR Configuration register +# bit13-0: 0xc30 (3120 DDR2 clks refresh rate) +# bit23-14: zero +# bit24: 1= enable exit self refresh mode on DDR access +# bit25: 1 required +# bit29-26: zero +# bit31-30: 01 + +DATA 0xFFD01404 0x37543000 # DDR Controller Control Low +# bit 4: 0=addr/cmd in smame cycle +# bit 5: 0=clk is driven during self refresh, we don't care for APX +# bit 6: 0=use recommended falling edge of clk for addr/cmd +# bit14: 0=input buffer always powered up +# bit18: 1=cpu lock transaction enabled +# bit23-20: 5=recommended value for CL=5 and STARTBURST_DEL disabled bit31=0 +# bit27-24: 7= CL+2, STARTBURST sample stages, for freqs 400MHz, unbuffered DIMM +# bit30-28: 3 required +# bit31: 0=no additional STARTBURST delay + +DATA 0xFFD01408 0x22125451 # DDR Timing (Low) (active cycles value +1) +# bit3-0: TRAS lsbs +# bit7-4: TRCD +# bit11- 8: TRP +# bit15-12: TWR +# bit19-16: TWTR +# bit20: TRAS msb +# bit23-21: 0x0 +# bit27-24: TRRD +# bit31-28: TRTP + +DATA 0xFFD0140C 0x00000a33 # DDR Timing (High) +# bit6-0: TRFC +# bit8-7: TR2R +# bit10-9: TR2W +# bit12-11: TW2W +# bit31-13: zero required + +DATA 0xFFD01410 0x0000000d # DDR Address Control +# bit1-0: 00, Cs0width=x8 +# bit3-2: 11, Cs0size=1Gb +# bit5-4: 00, Cs1width=nonexistent +# bit7-6: 00, Cs1size =nonexistent +# bit9-8: 00, Cs2width=nonexistent +# bit11-10: 00, Cs2size =nonexistent +# bit13-12: 00, Cs3width=nonexistent +# bit15-14: 00, Cs3size =nonexistent +# bit16: 0, Cs0AddrSel +# bit17: 0, Cs1AddrSel +# bit18: 0, Cs2AddrSel +# bit19: 0, Cs3AddrSel +# bit31-20: 0 required + +DATA 0xFFD01414 0x00000000 # DDR Open Pages Control +# bit0: 0, OpenPage enabled +# bit31-1: 0 required + +DATA 0xFFD01418 0x00000000 # DDR Operation +# bit3-0: 0x0, DDR cmd +# bit31-4: 0 required + +DATA 0xFFD0141C 0x00000C52 # DDR Mode +# bit2-0: 2, BurstLen=2 required +# bit3: 0, BurstType=0 required +# bit6-4: 4, CL=5 +# bit7: 0, TestMode=0 normal +# bit8: 0, DLL reset=0 normal +# bit11-9: 6, auto-precharge write recovery ???????????? +# bit12: 0, PD must be zero +# bit31-13: 0 required + +DATA 0xFFD01420 0x00000040 # DDR Extended Mode +# bit0: 0, DDR DLL enabled +# bit1: 0, DDR drive strenght normal +# bit2: 0, DDR ODT control lsd (disabled) +# bit5-3: 000, required +# bit6: 1, DDR ODT control msb, (disabled) +# bit9-7: 000, required +# bit10: 0, differential DQS enabled +# bit11: 0, required +# bit12: 0, DDR output buffer enabled +# bit31-13: 0 required + +DATA 0xFFD01424 0x0000F17F # DDR Controller Control High +# bit2-0: 111, required +# bit3 : 1 , MBUS Burst Chop disabled +# bit6-4: 111, required +# bit7 : 0 +# bit8 : 1 , add writepath sample stage, must be 1 for DDR freq >= 300MHz +# bit9 : 0 , no half clock cycle addition to dataout +# bit10 : 0 , 1/4 clock cycle skew enabled for addr/ctl signals +# bit11 : 0 , 1/4 clock cycle skew disabled for write mesh +# bit15-12: 1111 required +# bit31-16: 0 required + +DATA 0xFFD01428 0x00085520 # DDR2 ODT Read Timing (default values) +DATA 0xFFD0147C 0x00008552 # DDR2 ODT Write Timing (default values) + +DATA 0xFFD01500 0x00000000 # CS[0]n Base address to 0x0 +DATA 0xFFD01504 0x07FFFFF1 # CS[0]n Size +# bit0: 1, Window enabled +# bit1: 0, Write Protect disabled +# bit3-2: 00, CS0 hit selected +# bit23-4: ones, required +# bit31-24: 0x07, Size (i.e. 128MB) + +DATA 0xFFD01508 0x10000000 # CS[1]n Base address to 256Mb +DATA 0xFFD0150C 0x00000000 # CS[1]n Size, window disabled + +DATA 0xFFD01514 0x00000000 # CS[2]n Size, window disabled +DATA 0xFFD0151C 0x00000000 # CS[3]n Size, window disabled + +DATA 0xFFD01494 0x00030000 # DDR ODT Control (Low) +DATA 0xFFD01498 0x00000000 # DDR ODT Control (High) +# bit1-0: 00, ODT0 controlled by ODT Control (low) register above +# bit3-2: 01, ODT1 active NEVER! +# bit31-4: zero, required + +DATA 0xFFD0149C 0x0000E803 # CPU ODT Control +DATA 0xFFD01480 0x00000001 # DDR Initialization Control +#bit0=1, enable DDR init upon this register write + +# End of Header extension +DATA 0x0 0x0

Start with dockstar file and change for GoFlexHome
Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- include/configs/goflexhome.h | 140 ++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 140 insertions(+), 0 deletions(-) create mode 100644 include/configs/goflexhome.h
diff --git a/include/configs/goflexhome.h b/include/configs/goflexhome.h new file mode 100644 index 0000000..ce7ca41 --- /dev/null +++ b/include/configs/goflexhome.h @@ -0,0 +1,140 @@ +/* + * Copyright (C) 2013 Suriyan Ramasami suriyan.r@gmail.com + * + * Based on dockstar.h originally written by + * Copyright (C) 2010 Eric C. Cooper ecc@cmu.edu + * + * Based on sheevaplug.h originally written by + * Prafulla Wadaskar prafulla@marvell.com + * (C) Copyright 2009 + * Marvell Semiconductor <www.marvell.com> + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, + * MA 02110-1301 USA + */ + +#ifndef _CONFIG_GOFLEXHOME_H +#define _CONFIG_GOFLEXHOME_H + +/* + * Version number information + */ +#define CONFIG_IDENT_STRING "\nSeagate GoFlex Home" + +/* + * High Level Configuration Options (easy to change) + */ +#define CONFIG_FEROCEON_88FR131 1 /* CPU Core subversion */ +#define CONFIG_KIRKWOOD 1 /* SOC Family Name */ +#define CONFIG_KW88F6281 1 /* SOC Name */ +#define CONFIG_MACH_GOFLEXHOME /* Machine type */ +#define CONFIG_SKIP_LOWLEVEL_INIT /* disable board lowlevel_init */ + +/* + * Commands configuration + */ +#define CONFIG_SYS_NO_FLASH /* Declare no flash (NOR/SPI) */ +#define CONFIG_CONSOLE_MUX +#define CONFIG_SYS_CONSOLE_IS_IN_ENV + +#include <config_cmd_default.h> +#define CONFIG_CMD_DHCP +#define CONFIG_CMD_ENV +#define CONFIG_CMD_MII +#define CONFIG_CMD_NAND +#define CONFIG_CMD_PING +#define CONFIG_CMD_USB +#define CONFIG_CMD_IDE +#define CONFIG_CMD_DATE +#define CONFIG_CMD_EXT4 +#define CONFIG_SYS_MVFS /* Picks up Filesystem from mv-common.h */ + +/* + * mv-common.h should be defined after CMD configs since it used them + * to enable certain macros + */ +#include "mv-common.h" + +#undef CONFIG_SYS_PROMPT /* previously defined in mv-common.h */ +#define CONFIG_SYS_PROMPT "GoFlexHome> " /* Command Prompt */ + +/* + * Environment variables configurations + */ +#ifdef CONFIG_CMD_NAND +#define CONFIG_ENV_IS_IN_NAND 1 +#define CONFIG_ENV_SECT_SIZE 0x20000 /* 128K */ +#else +#define CONFIG_ENV_IS_NOWHERE 1 /* if env in SDRAM */ +#endif +/* + * max 4k env size is enough, but in case of nand + * it has to be rounded to sector size + */ +#define CONFIG_ENV_SIZE 0x20000 /* 128k */ +#define CONFIG_ENV_ADDR 0xC0000 +#define CONFIG_ENV_OFFSET 0xC0000 /* env starts here */ + +/* + * Default environment variables + */ +#define CONFIG_BOOTCOMMAND \ + "setenv bootargs ${console} ${mtdparts} ${bootargs_root}; " \ + "ubi part root; " \ + "ubifsmount ubi:root; " \ + "ubifsload 0x800000 ${kernel}; " \ + "bootm 0x800000" + +#define CONFIG_MTDPARTS "mtdparts=orion_nand:1m(uboot),6M(uImage),-(root)\0" + +#define CONFIG_EXTRA_ENV_SETTINGS \ + "console=console=ttyS0,115200\0" \ + "mtdids=nand0=orion_nand\0" \ + "mtdparts="CONFIG_MTDPARTS \ + "kernel=/boot/uImage\0" \ + "bootargs_root=ubi.mtd=root root=ubi0:root rootfstype=ubifs ro\0" + +/* + * Ethernet Driver configuration + */ +#ifdef CONFIG_CMD_NET +#define CONFIG_MVGBE_PORTS {1, 0} /* enable port 0 only */ +#define CONFIG_PHY_BASE_ADR 0 +#endif /* CONFIG_CMD_NET */ + +/* + * * SATA Driver configuration + * */ +#ifdef CONFIG_MVSATA_IDE +#define CONFIG_SYS_ATA_IDE0_OFFSET MV_SATA_PORT0_OFFSET +#endif /*CONFIG_MVSATA_IDE*/ + +/* + * * RTC driver configuration + * */ +#ifdef CONFIG_CMD_DATE +#define CONFIG_RTC_MV +#endif /* CONFIG_CMD_DATE */ + +/* Enable u-boot MENU */ +#define CONFIG_MENU + +/* Show the menu on boot */ +#define CONFIG_MENU_SHOW + +#endif /* _CONFIG_GOFLEXHOME_H */

Give the user a choosable menu of probable bootables to boot from.
Signed-off-by: Suriyan Ramasami suriyan.r@gmail.com --- board/Seagate/goflexhome/goflexhomemenu.c | 381 +++++++++++++++++++++++++++++ 1 files changed, 381 insertions(+), 0 deletions(-) create mode 100644 board/Seagate/goflexhome/goflexhomemenu.c
diff --git a/board/Seagate/goflexhome/goflexhomemenu.c b/board/Seagate/goflexhome/goflexhomemenu.c new file mode 100644 index 0000000..8866da6 --- /dev/null +++ b/board/Seagate/goflexhome/goflexhomemenu.c @@ -0,0 +1,381 @@ +/* + * Copyright (C) 2013 Suriyan Ramasami suriyan.r@gmail.com + * + * See file CREDITS for list of people who contributed to this + * project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of + * the License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, + * MA 02110-1301 USA + */ + +#include <common.h> + +#if defined(CONFIG_MENU) +/* Menu related code begins here */ + +/* Added to use the various usb/fat/ext4fs interfaces */ +#include <usb.h> +#include <ext4fs.h> +#include <menu.h> + +#define MENU_MAX_DEVICES 10 +#define MENU_MAX_PARTITIONS 10 +#define MENU_MAX_BOOTABLES 10 + +#define MENU_EXIT 1 +#define MENU_SHOW 2 + +typedef struct menu_bootables { + char interface[5]; + char drive; + int device; + int partition; + char filename[64]; + char fstype; /* f => fat, e => ext2/4 0 => invalid */ +} menu_bootables_t; + +static void goflexhome_menuprint(void *print_buffer) +{ + printf("%s\n", (char *) print_buffer); +} + +/* + * We shall use menu_<> variables to capture the state of past menu + * choices. + * menu_bootargs corresponds to bootargs + * menu_bootcmd corresponds to bootcmd + * menu_choice corresponds to the last choice that was picked + * menu_choice will be NULL the first time and also + * if a choice was never made. In that case we should pick + * to boot from the 1st bootable option if present. +*/ + +static int goflexhome_evaluate_env(void) +{ +char *s; + + run_command("run menu_bootargs", 0); + s = getenv("bootargs"); + printf("bootargs is %s\n", s); + run_command("run menu_bootcmd", 0); + s = getenv("bootcmd"); + printf("bootcmd is %s\n", s); + if (run_command("run bootcmd", 0) != 0) { + /* We failed to boot, present the menu */ + return MENU_SHOW; + } + if (strncmp(s, "echo", 4) == 0) { + /* User wants the u-boot prmpt */ + return MENU_EXIT; + } + run_command("bootm", 0); + + /* We are here, we failed to boot */ + return MENU_SHOW; +} + +static int goflexhome_handle_choice(menu_bootables_t menu_bootlist[], char *choice) +{ +char *s, *last_menu_choice; +char menu_command[128]; +char load_command[16]; +int index; +int call_saveenv = 0; + + if (choice == NULL) { + /* Exit menu and let it do its auto boot */ + return MENU_EXIT; + } + printf("\nYou chose: %s\n", choice); + + last_menu_choice = getenv("menu_choice"); + if (last_menu_choice == NULL) { + /* User has not yet chosen before */ + /* Lets default to boot from nand */ + setenv("menu_bootargs", "setenv bootargs ${console} ubi.mtd=2,2048 root=ubi0:root rootfstype=ubifs debug"); + setenv("menu_bootcmd", "setenv bootcmd nand read.e 0x800000 0x100000 0x600000"); + call_saveenv = 1; + } + if (choice[0] == '*') { + /* User wants same thing that was chosen the last time */ + return MENU_EXIT; + } + if (last_menu_choice && strcmp(choice, last_menu_choice) != 0) { + /* Save the choice chosen */ + setenv("menu_choice", choice); + call_saveenv = 1; + } + if (choice[0] == '+') { + /* User wants u-boot prompt */ + s = getenv("menu_bootcmd"); + if (strcmp(s, "setenv bootcmd echo Dropping you to u-boot prompt") != 0) { + setenv("menu_bootcmd", "setenv bootcmd echo Dropping you to u-boot prompt"); + saveenv(); + } + return MENU_EXIT; + } + + /* Steps to set the env variables to the chosen values */ + index = simple_strtoul(choice, NULL, 10); + sprintf(menu_command, "/dev/sd%c%d", menu_bootlist[index].drive, menu_bootlist[index].partition); + s = getenv("menu_root"); + if (strcmp(s, menu_command) != 0) { + setenv("menu_root", menu_command); + call_saveenv = 1; + } + s = getenv("menu_bootargs"); + if (strcmp(s, "setenv bootargs $console rootdelay=10 root=${menu_root} debug") != 0) { + setenv("menu_bootargs", "setenv bootargs ${console} rootdelay=10 root=${menu_root} debug"); + call_saveenv = 1; + } + switch (menu_bootlist[index].fstype) { + case 'e': + strcpy(load_command, "ext4load"); + break; + case 'f': + strcpy(load_command, "fatload"); + break; + default: + return MENU_EXIT; + } + + /* Lets try to load and check the image */ + sprintf(menu_command, "%s %s %d:%d %x %s", + load_command, + menu_bootlist[index].interface, + menu_bootlist[index].device, + menu_bootlist[index].partition, + CONFIG_SYS_LOAD_ADDR, + menu_bootlist[index].filename); + if (run_command(menu_command, 0) != 0) { + /* Could not load image */ + printf("Selected image could not be loaded ...\n"); + return MENU_SHOW; + } + sprintf(menu_command, "iminfo %x", CONFIG_SYS_LOAD_ADDR); + if (run_command(menu_command, 0) != 0) { + /* The image is not a valid image */ + printf("Selected image is not valid ...\n"); + return MENU_SHOW; + } + + sprintf(menu_command, "setenv bootcmd %s %s %d:%d %x %s", + load_command, + menu_bootlist[index].interface, + menu_bootlist[index].device, + menu_bootlist[index].partition, + CONFIG_SYS_LOAD_ADDR, + menu_bootlist[index].filename); + s = getenv("menu_bootcmd"); + if (strcmp(s, menu_command) != 0) { + setenv("menu_bootcmd", menu_command); + call_saveenv = 1; + } + if (call_saveenv) saveenv(); + return MENU_EXIT; +} + +static int goflexhome_menu(menu_bootables_t menu_bootlist[], int bootdelay) +{ +int index; +struct menu *m; +char menu_key[MENU_MAX_BOOTABLES][5]; +char menu_entry[MENU_MAX_BOOTABLES][64]; +char *menu_choice; +char *last_menu_choice; +char choice_menu_entry[64]; +char choice_menu[3]; + + m = menu_create("Bootables:\nChoice\tIntface\tDrive\tDevice\tPart\tFS\tFileName\n---------------------------------------------------------------", 60, 1, goflexhome_menuprint, NULL, NULL); + for (index = 0; index < MENU_MAX_BOOTABLES; index++) { + if (menu_bootlist[index].fstype == '0') break; + snprintf(menu_key[index], sizeof(menu_key[index]), "%d", index); + snprintf(menu_entry[index], sizeof(menu_entry[index]), "%d\t%s\t%c\t%d\t%d\t%c\t%s", index, menu_bootlist[index].interface, menu_bootlist[index].drive, menu_bootlist[index].device, menu_bootlist[index].partition, menu_bootlist[index].fstype, menu_bootlist[index].filename); + if (menu_item_add(m, menu_key[index], menu_entry[index]) != 1) { + menu_destroy(m); + return MENU_EXIT; + } + } + + /* Prep for what should be the default menu choice */ + /* If chosen before, choose the last boot options */ + /* If nothing chosen yet, then choose the first bootable option */ + /* If nothing chosen yet, and no first bootable option, then boot */ + /* from nand */ + + last_menu_choice = getenv("menu_choice"); + sprintf(choice_menu, "*"); + if (last_menu_choice) { + sprintf(choice_menu_entry, "* Last boot options (%s)", last_menu_choice); + } + else { + /* There was no last boot option */ + /* If there is at least 1 boot entry, make that the default */ + if (menu_bootlist[0].fstype != '0') { + setenv("menu_choice", menu_entry[0]); + sprintf(choice_menu_entry, menu_entry[0]); + } + else { + sprintf(choice_menu_entry, "* Last boot options (None, and no bootables found!)"); + } + } + if (menu_item_add(m, choice_menu, choice_menu_entry) != 1) { + menu_destroy(m); + return MENU_EXIT; + } + /* Mark this as the default choice. */ + menu_default_set(m, "*"); + if (menu_item_add(m, "+", "+ UBoot prompt") != 1) { + menu_destroy(m); + return MENU_EXIT; + } + + menu_get_choice(m, (void **) &menu_choice); + return(goflexhome_handle_choice(menu_bootlist, menu_choice)); +} + +static void goflexhome_filesearch(menu_bootables_t menu_bootlist[], int *bootindex) { +char *filenames[] = { "/uImage", "/boot/uImage", "" }; +int index = 0; + + while (filenames[index][0] != '\0') { + switch (menu_bootlist[*bootindex].fstype) { + case 'e': + if (ext4fs_open(filenames[index]) == -1) { + index++; + continue; + } + break; + + default: + break; + } + + /* Got a hit, record it */ + strcpy(menu_bootlist[*bootindex].filename, filenames[index]); + index++; + (*bootindex)++; + if (*bootindex >= MENU_MAX_BOOTABLES) break; + /* Prep next bootlist structure */ + memcpy(&menu_bootlist[*bootindex], &menu_bootlist[*bootindex - 1], sizeof(menu_bootables_t)); + } +} + +static void goflexhome_populate_partitions(menu_bootables_t menu_bootlist[], block_dev_desc_t *dev_desc, int *bootindex) +{ +int part; +disk_partition_t disk_part; + + part = menu_bootlist[*bootindex].partition; + + /* Get the partition structure */ + if (get_partition_info(dev_desc, part, &disk_part)) { + return; + } + + /* Try to check if its extX */ + if (ext4fs_probe(dev_desc, &disk_part) == 0) { + menu_bootlist[*bootindex].fstype = 'e'; + goflexhome_filesearch(menu_bootlist, bootindex); + ext4fs_close(); + return; + } +} + +static void goflexhome_populate_devices(menu_bootables_t menu_bootlist[], int *bootindex) +{ +block_dev_desc_t *dev_desc; +int device; +int part; + + /* Populate bootlist from each device and the partitions within */ + for (device = 0; device < MENU_MAX_DEVICES; device++) { + dev_desc = get_dev(menu_bootlist[*bootindex].interface, device); + if (dev_desc == NULL) continue; + menu_bootlist[*bootindex].device = device; + for (part = 0; part < MENU_MAX_PARTITIONS; part++) { + menu_bootlist[*bootindex].partition = part; + goflexhome_populate_partitions(menu_bootlist, dev_desc, bootindex); + } + } +} + +/* menu_bootlist[] can hold a max of MENU_MAX_BOOTABLES entries */ +static void goflexhome_populate_bootlist(menu_bootables_t menu_bootlist[]) +{ +/* ide is always first */ +char *interfaces[] = { "ide", "usb", "" }; +int bootindex = 0; +int i = 0; + + /* Lets initialize the usb sub system */ + usb_init(); + usb_stor_scan(0); + + /* This scans the partitions in the IDE storage */ + ide_init(); + + /* Populate bootlist from each interface */ + while ((interfaces[i][0] != '\0') && + (bootindex < MENU_MAX_BOOTABLES)) { + strcpy(menu_bootlist[bootindex].interface, interfaces[i]); + goflexhome_populate_devices(menu_bootlist, &bootindex); + i++; + } + if (bootindex < MENU_MAX_BOOTABLES) { + /* End marker of list */ + menu_bootlist[bootindex].fstype = '0'; + } + + /* Lets set the drive letter */ + menu_bootlist[0].drive = 'a'; + for (i = 1; i < bootindex; i++) { + if (menu_bootlist[i].fstype == '0') break; + /* Increase drive letter when interface changes */ + /* Or when device numbers change for same interface */ + menu_bootlist[i].drive = menu_bootlist[i - 1].drive; + if (strcmp(menu_bootlist[i].interface, menu_bootlist[i - 1].interface) != 0) { + menu_bootlist[i].drive++; + } + else { + if (menu_bootlist[i].device > menu_bootlist[i - 1].device) { + menu_bootlist[i].drive++; + } + } + } +} + +int menu_show(int bootdelay) +{ +menu_bootables_t menu_bootlist[MENU_MAX_BOOTABLES]; +int retval; + +#if 0 + set_default_env(); + saveenv(); +#endif + + goflexhome_populate_bootlist(menu_bootlist); + do { + retval = goflexhome_menu(menu_bootlist, bootdelay); + if (retval == MENU_EXIT) { + retval = goflexhome_evaluate_env(); + } + } while (retval == MENU_SHOW); + + return 0; +} + +#endif /* CONFIG_MENU */

Dear Suriyan Ramasami,
In message 1365392690-8668-1-git-send-email-suriyan.r@gmail.com you wrote:
Most of the files except goflexhomemenu.c are direct copies of the dockstar files with minor modifications.
goflexhomemenu.c has code which uses the menu functionality of u-boot and present the user with an option to choose the various bootable options.
Thank you very much for your patches, but there are a number of issues with these.
First, if you submit several patches, then please chose useful Subjects and Commit Messages for each patch. Something like this is not useful:
Suriyan Ramasami (8): Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support
Second, splitting your submission into several patches as you did it breaks bisectability of the code. Please squash all these poatches into one.
Third, there are a large number of coding style issues with your patches. Please make sure to read http://www.denx.de/wiki/U-Boot/Patches and run your patches through checkpatch before resubmitting:
CHECK: Alignment should match open parenthesis #161: FILE: board/Seagate/goflexhome/goflexhome.c:50: + kw_config_gpio(GOFLEXHOME_OE_VAL_LOW, + GOFLEXHOME_OE_VAL_HIGH,
CHECK: No space is necessary after a cast #247: FILE: board/Seagate/goflexhome/goflexhome.c:136: + /* command to read PHY dev address */ + if (miiphy_read(name, 0xEE, 0xEE, (u16 *) &devadr)) {
CHECK: Alignment should match open parenthesis #249: FILE: board/Seagate/goflexhome/goflexhome.c:138: + printf("Err..%s could not read PHY dev address\n", + __FUNCTION__);
WARNING: __func__ should be used instead of gcc specific __FUNCTION__ #249: FILE: board/Seagate/goflexhome/goflexhome.c:138: + __FUNCTION__);
total: 0 errors, 1 warnings, 3 checks, 187 lines checked
NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE USLEEP_RANGE
/home/wd/Mail/U-Boot/8248 has style problems, please review.
WARNING: line over 80 characters #214: FILE: include/configs/goflexhome.h:103: +#define CONFIG_MTDPARTS "mtdparts=orion_nand:1m(uboot),6M(uImage),-(root)\0"
total: 0 errors, 1 warnings, 0 checks, 140 lines checked
NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE USLEEP_RANGE
/home/wd/Mail/U-Boot/8251 has style problems, please review.
If any of these errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. WARNING: do not add new typedefs #151: FILE: board/Seagate/goflexhome/goflexhomemenu.c:40: +typedef struct menu_bootables {
ERROR: code indent should use tabs where possible #162: FILE: board/Seagate/goflexhome/goflexhomemenu.c:51: + printf("%s\n", (char *) print_buffer);$
CHECK: No space is necessary after a cast #162: FILE: board/Seagate/goflexhome/goflexhomemenu.c:51: +{ + printf("%s\n", (char *) print_buffer);
WARNING: please, no spaces at the start of a line #162: FILE: board/Seagate/goflexhome/goflexhomemenu.c:51: + printf("%s\n", (char *) print_buffer);$
WARNING: line over 80 characters #200: FILE: board/Seagate/goflexhome/goflexhomemenu.c:89: +static int goflexhome_handle_choice(menu_bootables_t menu_bootlist[], char *choice)
ERROR: do not initialise globals to 0 or NULL #206: FILE: board/Seagate/goflexhome/goflexhomemenu.c:95: +int call_saveenv = 0;
WARNING: line over 80 characters #218: FILE: board/Seagate/goflexhome/goflexhomemenu.c:107: + setenv("menu_bootargs", "setenv bootargs ${console} ubi.mtd=2,2048 root=ubi0:root rootfstype=ubifs debug");
WARNING: line over 80 characters #219: FILE: board/Seagate/goflexhome/goflexhomemenu.c:108: + setenv("menu_bootcmd", "setenv bootcmd nand read.e 0x800000 0x100000 0x600000");
WARNING: line over 80 characters #234: FILE: board/Seagate/goflexhome/goflexhomemenu.c:123: + if (strcmp(s, "setenv bootcmd echo Dropping you to u-boot prompt") != 0) {
WARNING: line over 80 characters #235: FILE: board/Seagate/goflexhome/goflexhomemenu.c:124: + setenv("menu_bootcmd", "setenv bootcmd echo Dropping you to u-boot prompt");
WARNING: line over 80 characters #243: FILE: board/Seagate/goflexhome/goflexhomemenu.c:132: + sprintf(menu_command, "/dev/sd%c%d", menu_bootlist[index].drive, menu_bootlist[index].partition);
WARNING: line over 80 characters #250: FILE: board/Seagate/goflexhome/goflexhomemenu.c:139: + if (strcmp(s, "setenv bootargs $console rootdelay=10 root=${menu_root} debug") != 0) {
WARNING: line over 80 characters #251: FILE: board/Seagate/goflexhome/goflexhomemenu.c:140: + setenv("menu_bootargs", "setenv bootargs ${console} rootdelay=10 root=${menu_root} debug");
CHECK: Alignment should match open parenthesis #267: FILE: board/Seagate/goflexhome/goflexhomemenu.c:156: + sprintf(menu_command, "%s %s %d:%d %x %s", + load_command,
CHECK: Alignment should match open parenthesis #286: FILE: board/Seagate/goflexhome/goflexhomemenu.c:175: + sprintf(menu_command, "setenv bootcmd %s %s %d:%d %x %s", + load_command,
ERROR: trailing statements should be on next line #297: FILE: board/Seagate/goflexhome/goflexhomemenu.c:186: + if (call_saveenv) saveenv();
WARNING: line over 80 characters #312: FILE: board/Seagate/goflexhome/goflexhomemenu.c:201: + m = menu_create("Bootables:\nChoice\tIntface\tDrive\tDevice\tPart\tFS\tFileName\n---------------------------------------------------------------", 60, 1, goflexhome_menuprint, NULL, NULL);
ERROR: trailing statements should be on next line #314: FILE: board/Seagate/goflexhome/goflexhomemenu.c:203: + if (menu_bootlist[index].fstype == '0') break;
WARNING: line over 80 characters #316: FILE: board/Seagate/goflexhome/goflexhomemenu.c:205: + snprintf(menu_entry[index], sizeof(menu_entry[index]), "%d\t%s\t%c\t%d\t%d\t%c\t%s", index, menu_bootlist[index].interface, menu_bootlist[index].drive, menu_bootlist[index].device, menu_bootlist[index].partition, menu_bootlist[index].fstype, menu_bootlist[index].filename);
ERROR: trailing whitespace #328: FILE: board/Seagate/goflexhome/goflexhomemenu.c:217: + $
WARNING: please, no spaces at the start of a line #328: FILE: board/Seagate/goflexhome/goflexhomemenu.c:217: + $
WARNING: line over 80 characters #332: FILE: board/Seagate/goflexhome/goflexhomemenu.c:221: + sprintf(choice_menu_entry, "* Last boot options (%s)", last_menu_choice);
ERROR: else should follow close brace '}' #334: FILE: board/Seagate/goflexhome/goflexhomemenu.c:223: + } + else {
ERROR: else should follow close brace '}' #341: FILE: board/Seagate/goflexhome/goflexhomemenu.c:230: + } + else {
WARNING: line over 80 characters #342: FILE: board/Seagate/goflexhome/goflexhomemenu.c:231: + sprintf(choice_menu_entry, "* Last boot options (None, and no bootables found!)");
ERROR: code indent should use tabs where possible #342: FILE: board/Seagate/goflexhome/goflexhomemenu.c:231: +^I ^I^Isprintf(choice_menu_entry, "* Last boot options (None, and no bootables found!)");$
WARNING: please, no space before tabs #342: FILE: board/Seagate/goflexhome/goflexhomemenu.c:231: +^I ^I^Isprintf(choice_menu_entry, "* Last boot options (None, and no bootables found!)");$
CHECK: No space is necessary after a cast #356: FILE: board/Seagate/goflexhome/goflexhomemenu.c:245: + + menu_get_choice(m, (void **) &menu_choice);
ERROR: return is not a function, parentheses are not required #357: FILE: board/Seagate/goflexhome/goflexhomemenu.c:246: + return(goflexhome_handle_choice(menu_bootlist, menu_choice));
WARNING: line over 80 characters #360: FILE: board/Seagate/goflexhome/goflexhomemenu.c:249: +static void goflexhome_filesearch(menu_bootables_t menu_bootlist[], int *bootindex) {
ERROR: open brace '{' following function declarations go on the next line #360: FILE: board/Seagate/goflexhome/goflexhomemenu.c:249: +static void goflexhome_filesearch(menu_bootables_t menu_bootlist[], int *bootindex) {
ERROR: do not initialise globals to 0 or NULL #362: FILE: board/Seagate/goflexhome/goflexhomemenu.c:251: +int index = 0;
ERROR: trailing statements should be on next line #381: FILE: board/Seagate/goflexhome/goflexhomemenu.c:270: + if (*bootindex >= MENU_MAX_BOOTABLES) break;
WARNING: line over 80 characters #383: FILE: board/Seagate/goflexhome/goflexhomemenu.c:272: + memcpy(&menu_bootlist[*bootindex], &menu_bootlist[*bootindex - 1], sizeof(menu_bootables_t));
WARNING: line over 80 characters #387: FILE: board/Seagate/goflexhome/goflexhomemenu.c:276: +static void goflexhome_populate_partitions(menu_bootables_t menu_bootlist[], block_dev_desc_t *dev_desc, int *bootindex)
WARNING: braces {} are not necessary for single statement blocks #395: FILE: board/Seagate/goflexhome/goflexhomemenu.c:284: + if (get_partition_info(dev_desc, part, &disk_part)) { + return; + }
WARNING: line over 80 characters #408: FILE: board/Seagate/goflexhome/goflexhomemenu.c:297: +static void goflexhome_populate_devices(menu_bootables_t menu_bootlist[], int *bootindex)
ERROR: trailing statements should be on next line #417: FILE: board/Seagate/goflexhome/goflexhomemenu.c:306: + if (dev_desc == NULL) continue;
WARNING: line over 80 characters #421: FILE: board/Seagate/goflexhome/goflexhomemenu.c:310: + goflexhome_populate_partitions(menu_bootlist, dev_desc, bootindex);
ERROR: do not initialise globals to 0 or NULL #431: FILE: board/Seagate/goflexhome/goflexhomemenu.c:320: +int bootindex = 0;
ERROR: do not initialise globals to 0 or NULL #432: FILE: board/Seagate/goflexhome/goflexhomemenu.c:321: +int i = 0;
ERROR: trailing whitespace #442: FILE: board/Seagate/goflexhome/goflexhomemenu.c:331: + while ((interfaces[i][0] != '\0') && $
ERROR: code indent should use tabs where possible #442: FILE: board/Seagate/goflexhome/goflexhomemenu.c:331: + while ((interfaces[i][0] != '\0') && $
WARNING: please, no spaces at the start of a line #442: FILE: board/Seagate/goflexhome/goflexhomemenu.c:331: + while ((interfaces[i][0] != '\0') && $
ERROR: code indent should use tabs where possible #443: FILE: board/Seagate/goflexhome/goflexhomemenu.c:332: + (bootindex < MENU_MAX_BOOTABLES)) {$
WARNING: please, no spaces at the start of a line #443: FILE: board/Seagate/goflexhome/goflexhomemenu.c:332: + (bootindex < MENU_MAX_BOOTABLES)) {$
ERROR: trailing statements should be on next line #456: FILE: board/Seagate/goflexhome/goflexhomemenu.c:345: + if (menu_bootlist[i].fstype == '0') break;
WARNING: line over 80 characters #460: FILE: board/Seagate/goflexhome/goflexhomemenu.c:349: + if (strcmp(menu_bootlist[i].interface, menu_bootlist[i - 1].interface) != 0) {
ERROR: else should follow close brace '}' #463: FILE: board/Seagate/goflexhome/goflexhomemenu.c:352: + } + else {
WARNING: line over 80 characters #464: FILE: board/Seagate/goflexhome/goflexhomemenu.c:353: + if (menu_bootlist[i].device > menu_bootlist[i - 1].device) {
WARNING: braces {} are not necessary for single statement blocks #464: FILE: board/Seagate/goflexhome/goflexhomemenu.c:353: + if (menu_bootlist[i].device > menu_bootlist[i - 1].device) { + menu_bootlist[i].drive++; + }
ERROR: trailing whitespace #469: FILE: board/Seagate/goflexhome/goflexhomemenu.c:358: +} $
CHECK: if this code is redundant consider removing it #476: FILE: board/Seagate/goflexhome/goflexhomemenu.c:365: +#if 0
ERROR: code indent should use tabs where possible #481: FILE: board/Seagate/goflexhome/goflexhomemenu.c:370: + goflexhome_populate_bootlist(menu_bootlist);$
WARNING: please, no spaces at the start of a line #481: FILE: board/Seagate/goflexhome/goflexhomemenu.c:370: + goflexhome_populate_bootlist(menu_bootlist);$
WARNING: braces {} are not necessary for single statement blocks #484: FILE: board/Seagate/goflexhome/goflexhomemenu.c:373: + if (retval == MENU_EXIT) { + retval = goflexhome_evaluate_env(); + }
total: 22 errors, 29 warnings, 5 checks, 381 lines checked
NOTE: whitespace errors detected, you may wish to use scripts/cleanpatch or scripts/cleanfile
NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE USLEEP_RANGE
/home/wd/Mail/U-Boot/8252 has style problems, please review.
Thanks.
Best regards,
Wolfgang Denk

-----Original Message----- From: Suriyan Ramasami [mailto:suriyan.r@gmail.com] Sent: 08 April 2013 09:15 To: u-boot@lists.denx.de Cc: ecc@cmu.edu; Prafulla Wadaskar; Suriyan Ramasami Subject: [PATCH 0/8] ARM: Add Seagate GoFlexHome support
Most of the files except goflexhomemenu.c are direct copies of the dockstar files with minor modifications.
goflexhomemenu.c has code which uses the menu functionality of u-boot and present the user with an option to choose the various bootable options.
Suriyan Ramasami (8): Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support Add Seagate GoFlexHome support
NACK: Each patch in the series must have meaning full discussion.
Also please explain in the each patch, the purpose and what exactly the patch is doing.
Please kindly maintain change log and versioning for the next updated patch series. you may follow the development documentation for the same.
Regards... Prafulla . . .
participants (3)
-
Prafulla Wadaskar
-
Suriyan Ramasami
-
Wolfgang Denk