All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support
@ 2013-04-10 13:12 Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 1/8] ARM: Add maintainer for the Seagate GoFlexHome Suriyan Ramasami
                   ` (9 more replies)
  0 siblings, 10 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Thank you Wolfgang and Prafulla for feedback on patch v1.
This is version 2.

I couldn't figure out how to squash all these changes in one
file with git. Do I just cat them? Hence, for this patch request
they are unsquashed and in separate emails. Please let me
know how I should squash for v3!

This adds support for the Seagate GoFlexHome.
It is based on the dockstar files for the most part.

Suriyan Ramasami (8):
  Add maintainer for the Seagate GoFlexHome
  Add a board entry for the Seagate GoFlexHome
  Present the user with a menu of bootable options on boot

 MAINTAINERS                               |    4 +
 board/Seagate/goflexhome/Makefile         |   51 ++++
 board/Seagate/goflexhome/goflexhome.c     |  187 +++++++++++++
 board/Seagate/goflexhome/goflexhome.h     |   47 ++++
 board/Seagate/goflexhome/goflexhomemenu.c |  415 +++++++++++++++++++++++++++++
 board/Seagate/goflexhome/kwbimage.cfg     |  168 ++++++++++++
 boards.cfg                                |    1 +
 include/configs/goflexhome.h              |  141 ++++++++++
 8 files changed, 1014 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

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 1/8] ARM: Add maintainer for the Seagate GoFlexHome
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 2/8] ARM: Add a board entry " Suriyan Ramasami
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Create an entry for the Seagate GoFlexHome as goflexhome

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes for v2:
	- Coding style changes

 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)
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 2/8] ARM: Add a board entry for the Seagate GoFlexHome
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 1/8] ARM: Add maintainer for the Seagate GoFlexHome Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 3/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Create an entry for the Seagate GoFlexHome as goflexhome
Its exactly like the dockstar entry.

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes for v2:
	- Coding style changes

 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
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 3/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 1/8] ARM: Add maintainer for the Seagate GoFlexHome Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 2/8] ARM: Add a board entry " Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 4/8] " Suriyan Ramasami
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Start with dockstar's dockstar.h
Add #define CONFIG_CMD_IDE, CONFIG_MVSATA_IDE for IDE support
Add #define CONFIG_CMD_DATE, CONFIG_RTC_MV for RTC support
Add #define CONFIG_CMD_EXT4 for supporting ext4 FS
Change CONFIG_SYS_PROMPT to be GoFlex specific
Change CONFIG_ENV_ADDR and CONFIG_ENV_OFFSET to default address
present in ArchLinuxArm install (/etc/fw_env.config)
Add #define CONFIG_MENU to enable u-boot MENU
Add #define CONFIG_MENU_SHOW to show the menu on boot

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes for v2:
	- Coding style changes

 include/configs/goflexhome.h |  141 ++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 141 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..03a7b22
--- /dev/null
+++ b/include/configs/goflexhome.h
@@ -0,0 +1,141 @@
+/*
+ * 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 */
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 4/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (2 preceding siblings ...)
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 3/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 5/8] " Suriyan Ramasami
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Start with the Dockstar Makefile
Add the two object files goflexhome.o and goflexhomemenu.o
that need to be compiled

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes for v2:
	- Coding style changes

 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
+
+#########################################################################
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 5/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (3 preceding siblings ...)
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 4/8] " Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 6/8] " Suriyan Ramasami
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Start with the Dockstar kwbimage.cfg
Absolutely no change from the dockstar file

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes for v2:
	- Coding style changes

 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
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 6/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (4 preceding siblings ...)
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 5/8] " Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 7/8] " Suriyan Ramasami
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Start with the dockstar file.
In all the #defines cosmetically change DOCKSTAR to GOFLEXHOME

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes in v2:
	- Coding style changes

 board/Seagate/goflexhome/goflexhome.h |   47 +++++++++++++++++++++++++++++++++
 1 files changed, 47 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..2af2596
--- /dev/null
+++ b/board/Seagate/goflexhome/goflexhome.h
@@ -0,0 +1,47 @@
+/*
+ * 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)
+
+#endif /* __GOFLEXHOME_H */
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 7/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (5 preceding siblings ...)
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 6/8] " Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot Suriyan Ramasami
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Start with the dockstar file.
Include goflexhome.h instead of dockstar.h
Change #define DOCKSTAR_* to #define GOFLEXHOME_*
Change the MACH_TYPE_DOCKSTAR to MACH_TYPE_GOFLEXHOME to reflect
that its a GoFlexHome.

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes in v2:
	Coding style changes

 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..c1314fb
--- /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",
+		       __func__);
+		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);
+	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;
+	}
+}
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (6 preceding siblings ...)
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 7/8] " Suriyan Ramasami
@ 2013-04-10 13:12 ` Suriyan Ramasami
  2013-04-11  1:43   ` Rob Herring
  2013-04-10 13:37 ` [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Albert ARIBAUD
  2013-04-10 13:45 ` Wolfgang Denk
  9 siblings, 1 reply; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-10 13:12 UTC (permalink / raw)
  To: u-boot

Initialize usb and ide.
Scan through the usb for storage and boot capable partitions.
Scan through the ide interface for boot capable partitions.
Present such bootable options to the user to choose to boot from
If the user does not choose any choose the default option
the default option is the option chosen by the user the last time
If no such default option exists, boot from the first possible
bootable option.

Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
---
Changes in v2:
	- Coding style changes

 board/Seagate/goflexhome/goflexhomemenu.c |  415 +++++++++++++++++++++++++++++
 1 files changed, 415 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..6169cf8
--- /dev/null
+++ b/board/Seagate/goflexhome/goflexhomemenu.c
@@ -0,0 +1,415 @@
+/*
+ * 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
+
+#define MENU_DEFAULT_BOOTARGS \
+	"setenv bootargs ${console} ubi.mtd=2,2048 " \
+	"root=ubi0:root rootfstype=ubifs debug"
+
+#define MENU_DEFAULT_BOOTCMD \
+	"setenv bootcmd nand read.e 0x800000 0x100000 0x600000"
+
+#define MENU_PROMPT_BOOTCMD \
+	"setenv bootcmd echo Dropping you to u-boot"
+
+#define MENU_CHOSEN_BOOTARGS \
+	"setenv bootargs $console rootdelay=10 root=${menu_root} debug"
+
+#define MENU_OPTIONS_HEADER \
+	"Bootables:\nChoice\tIntface\tDrive\tDevice\tPart\tFS\tFileName\n" \
+	"---------------------------------------------------------------"
+
+#define MENU_DEFAULT_NOBOOTABLES \
+	"* Last boot options (None, and no bootables found!"
+
+struct menu_bootables {
+	char	interface[5];
+	char	drive;
+	int	device;
+	int	partition;
+	char	filename[64];
+	char	fstype;		/* f => fat, e => ext2/4 0 => invalid */
+};
+
+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(struct menu_bootables menu_bootlist[],
+				    char *choice)
+{
+char *s, *last_menu_choice;
+char menu_command[128];
+char load_command[16];
+int index;
+int call_saveenv;
+
+	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", MENU_DEFAULT_BOOTARGS);
+		setenv("menu_bootcmd", MENU_DEFAULT_BOOTCMD);
+		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, MENU_PROMPT_BOOTCMD) !=  0) {
+			setenv("menu_bootcmd", MENU_PROMPT_BOOTCMD);
+			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, MENU_CHOSEN_BOOTARGS) != 0) {
+		setenv("menu_bootargs", MENU_CHOSEN_BOOTARGS);
+		call_saveenv = 1;
+	}
+	switch (menu_bootlist[index].fstype) {
+	case 'e':
+		strcpy(load_command, "ext4load");
+		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(struct menu_bootables 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(MENU_OPTIONS_HEADER, 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@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, MENU_DEFAULT_NOBOOTABLES);
+		}
+	}
+	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(struct menu_bootables menu_bootlist[],
+				  int *bootindex) {
+char *filenames[] = { "/uImage", "/boot/uImage", "" };
+int index;
+
+	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(struct menu_bootables));
+	}
+}
+
+static void goflexhome_populate_partitions(struct menu_bootables *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(struct menu_bootables 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(struct menu_bootables menu_bootlist[])
+{
+/* ide is always first */
+char *interfaces[] = { "ide", "usb", "" };
+int bootindex;
+int i;
+
+	bootindex = 0;
+	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)
+{
+struct menu_bootables menu_bootlist[MENU_MAX_BOOTABLES];
+int retval;
+
+	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 */
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (7 preceding siblings ...)
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot Suriyan Ramasami
@ 2013-04-10 13:37 ` Albert ARIBAUD
  2013-04-10 13:45 ` Wolfgang Denk
  9 siblings, 0 replies; 16+ messages in thread
From: Albert ARIBAUD @ 2013-04-10 13:37 UTC (permalink / raw)
  To: u-boot

Hi Suriyan,

On Wed, 10 Apr 2013 06:12:09 -0700, Suriyan Ramasami
<suriyan.r@gmail.com> wrote:

> Thank you Wolfgang and Prafulla for feedback on patch v1.
> This is version 2.
> 
> I couldn't figure out how to squash all these changes in one
> file with git. Do I just cat them? Hence, for this patch request
> they are unsquashed and in separate emails. Please let me
> know how I should squash for v3!
> 
> This adds support for the Seagate GoFlexHome.
> It is based on the dockstar files for the most part.
> 
> Suriyan Ramasami (8):
>   Add maintainer for the Seagate GoFlexHome
>   Add a board entry for the Seagate GoFlexHome
>   Present the user with a menu of bootable options on boot

Please do not spread board additon over 7 patches of which 6 are
necessarily useless if alone. Make this one single patch.

Also, patch 8 is unrelated to adding the board, and is not even related
to any hardware. Make it a separate submission.

Amicalement,
-- 
Albert.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support
  2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
                   ` (8 preceding siblings ...)
  2013-04-10 13:37 ` [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Albert ARIBAUD
@ 2013-04-10 13:45 ` Wolfgang Denk
  9 siblings, 0 replies; 16+ messages in thread
From: Wolfgang Denk @ 2013-04-10 13:45 UTC (permalink / raw)
  To: u-boot

Dear Suriyan Ramasami,

In message <1365599537-14138-1-git-send-email-suriyan.r@gmail.com> you wrote:
> 
> I couldn't figure out how to squash all these changes in one
> file with git. Do I just cat them? Hence, for this patch request
> they are unsquashed and in separate emails. Please let me
> know how I should squash for v3!

Use: "git rebase -i master.." or "git rebase -i HEAD~8" or similar.
Then, in your editor, make all commits except for the first one as "s"
(squash).

If you google, you will find a lot or git documentation that explains
the details; see for example
http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

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
As a general rule, the freedom of any people can  be  judged  by  the
volume of their laughter.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot
  2013-04-10 13:12 ` [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot Suriyan Ramasami
@ 2013-04-11  1:43   ` Rob Herring
  2013-04-11  4:27     ` Suriyan Ramasami
  2013-04-11 13:02     ` Mike Dunn
  0 siblings, 2 replies; 16+ messages in thread
From: Rob Herring @ 2013-04-11  1:43 UTC (permalink / raw)
  To: u-boot

On Wed, Apr 10, 2013 at 8:12 AM, Suriyan Ramasami <suriyan.r@gmail.com> wrote:
> Initialize usb and ide.
> Scan through the usb for storage and boot capable partitions.
> Scan through the ide interface for boot capable partitions.
> Present such bootable options to the user to choose to boot from
> If the user does not choose any choose the default option
> the default option is the option chosen by the user the last time
> If no such default option exists, boot from the first possible
> bootable option.

This all sounds very generic but...

>
> Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
> ---
> Changes in v2:
>         - Coding style changes
>
>  board/Seagate/goflexhome/goflexhomemenu.c |  415 +++++++++++++++++++++++++++++

but this is not a generic location. This feature would interest me and
probably Stephen as well.

Rob

>  1 files changed, 415 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..6169cf8
> --- /dev/null
> +++ b/board/Seagate/goflexhome/goflexhomemenu.c
> @@ -0,0 +1,415 @@
> +/*
> + * 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
> +
> +#define MENU_DEFAULT_BOOTARGS \
> +       "setenv bootargs ${console} ubi.mtd=2,2048 " \
> +       "root=ubi0:root rootfstype=ubifs debug"
> +
> +#define MENU_DEFAULT_BOOTCMD \
> +       "setenv bootcmd nand read.e 0x800000 0x100000 0x600000"
> +
> +#define MENU_PROMPT_BOOTCMD \
> +       "setenv bootcmd echo Dropping you to u-boot"
> +
> +#define MENU_CHOSEN_BOOTARGS \
> +       "setenv bootargs $console rootdelay=10 root=${menu_root} debug"
> +
> +#define MENU_OPTIONS_HEADER \
> +       "Bootables:\nChoice\tIntface\tDrive\tDevice\tPart\tFS\tFileName\n" \
> +       "---------------------------------------------------------------"
> +
> +#define MENU_DEFAULT_NOBOOTABLES \
> +       "* Last boot options (None, and no bootables found!"
> +
> +struct menu_bootables {
> +       char    interface[5];
> +       char    drive;
> +       int     device;
> +       int     partition;
> +       char    filename[64];
> +       char    fstype;         /* f => fat, e => ext2/4 0 => invalid */
> +};
> +
> +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(struct menu_bootables menu_bootlist[],
> +                                   char *choice)
> +{
> +char *s, *last_menu_choice;
> +char menu_command[128];
> +char load_command[16];
> +int index;
> +int call_saveenv;
> +
> +       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", MENU_DEFAULT_BOOTARGS);
> +               setenv("menu_bootcmd", MENU_DEFAULT_BOOTCMD);
> +               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, MENU_PROMPT_BOOTCMD) !=  0) {
> +                       setenv("menu_bootcmd", MENU_PROMPT_BOOTCMD);
> +                       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, MENU_CHOSEN_BOOTARGS) != 0) {
> +               setenv("menu_bootargs", MENU_CHOSEN_BOOTARGS);
> +               call_saveenv = 1;
> +       }
> +       switch (menu_bootlist[index].fstype) {
> +       case 'e':
> +               strcpy(load_command, "ext4load");
> +               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(struct menu_bootables 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(MENU_OPTIONS_HEADER, 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, MENU_DEFAULT_NOBOOTABLES);
> +               }
> +       }
> +       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(struct menu_bootables menu_bootlist[],
> +                                 int *bootindex) {
> +char *filenames[] = { "/uImage", "/boot/uImage", "" };
> +int index;
> +
> +       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(struct menu_bootables));
> +       }
> +}
> +
> +static void goflexhome_populate_partitions(struct menu_bootables *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(struct menu_bootables 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(struct menu_bootables menu_bootlist[])
> +{
> +/* ide is always first */
> +char *interfaces[] = { "ide", "usb", "" };
> +int bootindex;
> +int i;
> +
> +       bootindex = 0;
> +       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)
> +{
> +struct menu_bootables menu_bootlist[MENU_MAX_BOOTABLES];
> +int retval;
> +
> +       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 */
> --
> 1.7.1
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot
  2013-04-11  1:43   ` Rob Herring
@ 2013-04-11  4:27     ` Suriyan Ramasami
  2013-04-11 13:02     ` Mike Dunn
  1 sibling, 0 replies; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-11  4:27 UTC (permalink / raw)
  To: u-boot

I shall probably move it to a generic location with a more generic name.
Any reocmmendations?

- Suriyan


On Wed, Apr 10, 2013 at 6:43 PM, Rob Herring <robherring2@gmail.com> wrote:

> On Wed, Apr 10, 2013 at 8:12 AM, Suriyan Ramasami <suriyan.r@gmail.com>
> wrote:
> > Initialize usb and ide.
> > Scan through the usb for storage and boot capable partitions.
> > Scan through the ide interface for boot capable partitions.
> > Present such bootable options to the user to choose to boot from
> > If the user does not choose any choose the default option
> > the default option is the option chosen by the user the last time
> > If no such default option exists, boot from the first possible
> > bootable option.
>
> This all sounds very generic but...
>
> >
> > Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
> > ---
> > Changes in v2:
> >         - Coding style changes
> >
> >  board/Seagate/goflexhome/goflexhomemenu.c |  415
> +++++++++++++++++++++++++++++
>
> but this is not a generic location. This feature would interest me and
> probably Stephen as well.
>
> Rob
>
> >  1 files changed, 415 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..6169cf8
> > --- /dev/null
> > +++ b/board/Seagate/goflexhome/goflexhomemenu.c
> > @@ -0,0 +1,415 @@
> > +/*
> > + * 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
> > +
> > +#define MENU_DEFAULT_BOOTARGS \
> > +       "setenv bootargs ${console} ubi.mtd=2,2048 " \
> > +       "root=ubi0:root rootfstype=ubifs debug"
> > +
> > +#define MENU_DEFAULT_BOOTCMD \
> > +       "setenv bootcmd nand read.e 0x800000 0x100000 0x600000"
> > +
> > +#define MENU_PROMPT_BOOTCMD \
> > +       "setenv bootcmd echo Dropping you to u-boot"
> > +
> > +#define MENU_CHOSEN_BOOTARGS \
> > +       "setenv bootargs $console rootdelay=10 root=${menu_root} debug"
> > +
> > +#define MENU_OPTIONS_HEADER \
> > +
> "Bootables:\nChoice\tIntface\tDrive\tDevice\tPart\tFS\tFileName\n" \
> > +       "---------------------------------------------------------------"
> > +
> > +#define MENU_DEFAULT_NOBOOTABLES \
> > +       "* Last boot options (None, and no bootables found!"
> > +
> > +struct menu_bootables {
> > +       char    interface[5];
> > +       char    drive;
> > +       int     device;
> > +       int     partition;
> > +       char    filename[64];
> > +       char    fstype;         /* f => fat, e => ext2/4 0 => invalid */
> > +};
> > +
> > +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(struct menu_bootables
> menu_bootlist[],
> > +                                   char *choice)
> > +{
> > +char *s, *last_menu_choice;
> > +char menu_command[128];
> > +char load_command[16];
> > +int index;
> > +int call_saveenv;
> > +
> > +       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", MENU_DEFAULT_BOOTARGS);
> > +               setenv("menu_bootcmd", MENU_DEFAULT_BOOTCMD);
> > +               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, MENU_PROMPT_BOOTCMD) !=  0) {
> > +                       setenv("menu_bootcmd", MENU_PROMPT_BOOTCMD);
> > +                       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, MENU_CHOSEN_BOOTARGS) != 0) {
> > +               setenv("menu_bootargs", MENU_CHOSEN_BOOTARGS);
> > +               call_saveenv = 1;
> > +       }
> > +       switch (menu_bootlist[index].fstype) {
> > +       case 'e':
> > +               strcpy(load_command, "ext4load");
> > +               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(struct menu_bootables 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(MENU_OPTIONS_HEADER, 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,
> MENU_DEFAULT_NOBOOTABLES);
> > +               }
> > +       }
> > +       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(struct menu_bootables menu_bootlist[],
> > +                                 int *bootindex) {
> > +char *filenames[] = { "/uImage", "/boot/uImage", "" };
> > +int index;
> > +
> > +       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(struct menu_bootables));
> > +       }
> > +}
> > +
> > +static void goflexhome_populate_partitions(struct menu_bootables
> *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(struct menu_bootables
> 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(struct menu_bootables
> menu_bootlist[])
> > +{
> > +/* ide is always first */
> > +char *interfaces[] = { "ide", "usb", "" };
> > +int bootindex;
> > +int i;
> > +
> > +       bootindex = 0;
> > +       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)
> > +{
> > +struct menu_bootables menu_bootlist[MENU_MAX_BOOTABLES];
> > +int retval;
> > +
> > +       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 */
> > --
> > 1.7.1
> >
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot
  2013-04-11  1:43   ` Rob Herring
  2013-04-11  4:27     ` Suriyan Ramasami
@ 2013-04-11 13:02     ` Mike Dunn
  2013-04-11 19:51       ` Suriyan Ramasami
  1 sibling, 1 reply; 16+ messages in thread
From: Mike Dunn @ 2013-04-11 13:02 UTC (permalink / raw)
  To: u-boot

On 04/10/2013 06:43 PM, Rob Herring wrote:
> On Wed, Apr 10, 2013 at 8:12 AM, Suriyan Ramasami <suriyan.r@gmail.com> wrote:
>> Initialize usb and ide.
>> Scan through the usb for storage and boot capable partitions.
>> Scan through the ide interface for boot capable partitions.
>> Present such bootable options to the user to choose to boot from
>> If the user does not choose any choose the default option
>> the default option is the option chosen by the user the last time
>> If no such default option exists, boot from the first possible
>> bootable option.
> 
> This all sounds very generic but...
> 
>>
>> Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
>> ---
>> Changes in v2:
>>         - Coding style changes
>>
>>  board/Seagate/goflexhome/goflexhomemenu.c |  415 +++++++++++++++++++++++++++++
> 
> but this is not a generic location. This feature would interest me and
> probably Stephen as well.


I was thinking along these lines as well for the treo 680.  Suriyan, perhaps we
can collaborate, with the guidance of the maintainers here.

Mike

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot
  2013-04-11 13:02     ` Mike Dunn
@ 2013-04-11 19:51       ` Suriyan Ramasami
  2013-04-12 13:56         ` Tom Rini
  0 siblings, 1 reply; 16+ messages in thread
From: Suriyan Ramasami @ 2013-04-11 19:51 UTC (permalink / raw)
  To: u-boot

Sounds good to me.

I shall attempt a generic version and call it cmd_bootscan.c in the common
directory.

- Suriyan


On Thu, Apr 11, 2013 at 6:02 AM, Mike Dunn <mikedunn@newsguy.com> wrote:

> On 04/10/2013 06:43 PM, Rob Herring wrote:
> > On Wed, Apr 10, 2013 at 8:12 AM, Suriyan Ramasami <suriyan.r@gmail.com>
> wrote:
> >> Initialize usb and ide.
> >> Scan through the usb for storage and boot capable partitions.
> >> Scan through the ide interface for boot capable partitions.
> >> Present such bootable options to the user to choose to boot from
> >> If the user does not choose any choose the default option
> >> the default option is the option chosen by the user the last time
> >> If no such default option exists, boot from the first possible
> >> bootable option.
> >
> > This all sounds very generic but...
> >
> >>
> >> Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
> >> ---
> >> Changes in v2:
> >>         - Coding style changes
> >>
> >>  board/Seagate/goflexhome/goflexhomemenu.c |  415
> +++++++++++++++++++++++++++++
> >
> > but this is not a generic location. This feature would interest me and
> > probably Stephen as well.
>
>
> I was thinking along these lines as well for the treo 680.  Suriyan,
> perhaps we
> can collaborate, with the guidance of the maintainers here.
>
> Mike
>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot
  2013-04-11 19:51       ` Suriyan Ramasami
@ 2013-04-12 13:56         ` Tom Rini
  0 siblings, 0 replies; 16+ messages in thread
From: Tom Rini @ 2013-04-12 13:56 UTC (permalink / raw)
  To: u-boot

On Thu, Apr 11, 2013 at 12:51:03PM -0700, Suriyan Ramasami wrote:

> Sounds good to me.
> 
> I shall attempt a generic version and call it cmd_bootscan.c in the common
> directory.

A good first pass.  Please keep board/ait/cam_enc_4xx/cam_enc_4xx.c in
mind as that has a similar set of things going on.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130412/6181fd0b/attachment.pgp>

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2013-04-12 13:56 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-10 13:12 [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 1/8] ARM: Add maintainer for the Seagate GoFlexHome Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 2/8] ARM: Add a board entry " Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 3/8] ARM: Add Seagate GoFlexHome support Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 4/8] " Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 5/8] " Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 6/8] " Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 7/8] " Suriyan Ramasami
2013-04-10 13:12 ` [U-Boot] [PATCH v2 8/8] ARM: Present a menu of bootable options on boot Suriyan Ramasami
2013-04-11  1:43   ` Rob Herring
2013-04-11  4:27     ` Suriyan Ramasami
2013-04-11 13:02     ` Mike Dunn
2013-04-11 19:51       ` Suriyan Ramasami
2013-04-12 13:56         ` Tom Rini
2013-04-10 13:37 ` [U-Boot] [PATCH v2 0/8] ARM: Add Seagate GoFlexHome support Albert ARIBAUD
2013-04-10 13:45 ` Wolfgang Denk

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.