From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@ti.com>
Date: Wed, 19 Feb 2014 08:52:35 -0500
Subject: [U-Boot] [RFC PATCH 2/3] move the beaglebones over to the
 generic configs
In-Reply-To: <1392659798-1665-3-git-send-email-dennis@ausil.us>
References: <1392659798-1665-1-git-send-email-dennis@ausil.us>
	<1392659798-1665-3-git-send-email-dennis@ausil.us>
Message-ID: <20140219135235.GL19081@bill-the-cat>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Mon, Feb 17, 2014 at 11:56:37AM -0600, Dennis Gilmore wrote:

[snip]
> -		"nand read ${fdtaddr} u-boot-spl-os; " \
> +		"nand read ${fdt_addr} u-boot-spl-os; " \
>  		"nand read ${loadaddr} kernel; " \
> -		"bootz ${loadaddr} - ${fdtaddr}\0"
> +		"bootz ${loadaddr} - ${fdt_addr}\0"

In all of these conversions you should switch loadaddr to kernel_addr_r
too.

[snip]
> -	"fdtfile=undefined\0" \

That's a problem.  Need this for 'findfdt' to work.

> -	"rootpath=/export/rootfs\0" \
> -	"nfsopts=nolock\0" \
> -	"static_ip=${ipaddr}:${serverip}:${gatewayip}:${netmask}:${hostname}" \
> -		"::off\0" \
[snip]
> -	"spiroot=/dev/mtdblock4 rw\0" \
> -	"spirootfstype=jffs2\0" \
> -	"spisrcaddr=0xe0000\0" \
> -	"spiimgsize=0x362000\0" \
> -	"spibusno=0\0" \
> -	"spiargs=setenv bootargs console=${console} " \
> -		"${optargs} " \
> -		"root=${spiroot} " \
> -		"rootfstype=${spirootfstype}\0" \
> -	"netargs=setenv bootargs console=${console} " \
> -		"${optargs} " \
> -		"root=/dev/nfs " \
> -		"nfsroot=${serverip}:${rootpath},${nfsopts} rw " \
> -		"ip=dhcp\0" \
[snip]
> -	"spiboot=echo Booting from spi ...; " \
> -		"run spiargs; " \
> -		"sf probe ${spibusno}:0; " \
> -		"sf read ${loadaddr} ${spisrcaddr} ${spiimgsize}; " \
> -		"bootz ${loadaddr}\0" \
> -	"netboot=echo Booting from network ...; " \
> -		"setenv autoload no; " \
> -		"dhcp; " \
> -		"tftp ${loadaddr} ${bootfile}; " \
> -		"tftp ${fdtaddr} ${fdtfile}; " \
> -		"run netargs; " \
> -		"bootz ${loadaddr} - ${fdtaddr}\0" \
> -	"ramboot=echo Booting from ramdisk ...; " \
> -		"run ramargs; " \
> -		"bootz ${loadaddr} ${rdaddr} ${fdtaddr}\0" \

And this is functionality I don't see in the generic distro config that
we still need.

[snip]
>  /*
> - * Common filesystems support.  When we have removable storage we
> - * enabled a number of useful commands and support.
> + * Include the generic config options and boot environment when not
> + * building our SPL
>   */
> -#if defined(CONFIG_MMC) || defined(CONFIG_USB_STORAGE)
> -#define CONFIG_DOS_PARTITION
> -#define CONFIG_CMD_FAT
> -#define CONFIG_FAT_WRITE
> -#define CONFIG_CMD_EXT2
> -#define CONFIG_CMD_EXT4
> -#define CONFIG_CMD_FS_GENERIC
> +#ifndef CONFIG_SPL_BUILD
> +#include <config_distro_defaults.h>
> +#include <config_distro_bootcmd.h>
>  #endif

This is fine, for now, but I'll have to shuffle things around later as
some of the good feedback I'd gotten off-list about ti_armv7_common.h is
that it's not as useful for non-Linux targets, so I need to adjust the
contents a bit more.

-- 
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/20140219/f3188ae9/attachment.pgp>