[U-Boot] [PATCH 6/6] RFC: am335x: enable falcon boot mode for mmc (raw and fat)

[Tom Rini]

On Wed, Mar 27, 2013 at 08:57:05PM +0100, Peter Korsgaard wrote:
> >>>>> "Tom" == Tom Rini <trini at ti.com> writes:
> 
>  Tom> On Sun, Mar 24, 2013 at 10:51:33PM +0100, Peter Korsgaard wrote:
>  >> Jump into full u-boot mode if a 'c' character is received on the uart.
>  >> 
>  >> We need to adjust the spl bss/malloc area to not overlap with the
>  >> loadaddr of the kernel (sdram + 32k), so move it past u-boot instead.
>  >> 
>  >> Signed-off-by: Peter Korsgaard <peter.korsgaard at barco.com>
>  Tom> [snip]
>  >> +/* raw mmc */
>  >> +#define CONFIG_SYS_MMCSD_RAW_MODE_KERNEL_SECTOR	0xa00 /* address 0xa0000 */
>  >> +#define CONFIG_SYS_MMCSD_RAW_MODE_ARGS_SECTOR	0x8   /* address 0x1000 */
>  >> +#define CONFIG_SYS_MMCSD_RAW_MODE_ARGS_SECTORS	8     /* 4KB */
> 
>  Tom> Did you also test raw mode?
> 
> Yes, I did. I'm personally most interested in raw mode because of the 4x
> redundant MLO handling.

Note that you only get 3 on RAW mode because we place U-Boot (by
default) in the fourth slot.  I noticed this recently but was hesitant
to break possible deployed setups (in beagle land).  RAW mode isnt' well
documented, but it is in a few places.  For your device I imagine you
could fix things however.

>  Tom> Also, why 0xa00?  U-Boot is 0x300 -> 0x500,
>  Tom> and one might say throw a redundant copy at 0x500 -> 0x700.  But we
>  Tom> don't do 4 copy redundancy in U-Boot, just 2 usually.
> 
> No particular reason, 0x700 should work as well. As mentioned, this was
> just a proof of concept to be able to test it.

Right.  And I'd love to see bootcount or similar updated so that we can
try redundant copies of U-Boot.

>  >> +/* dummy defines to keep spl_nand.c happy */
>  >> +#define CONFIG_CMD_SPL_NAND_OFS			0
>  >> +#define CONFIG_SYS_NAND_SPL_KERNEL_OFFS		0
>  >> +#define CONFIG_CMD_SPL_WRITE_SIZE		0
> 
>  Tom> We should do some real defines here while at it, since the GP EVM has
>  Tom> NAND :)
> 
> Yes. It's a bit unfortunate that CONFIG_SPL_OS_BOOT is a global setting,
> so you need all the various falcon boot related defines even if you
> don't plan on using them.
> 
> I can try to come up with sensible NAND values, but I don't have a EVM
> to test.

Well, CONFIG_SYS_NAND_SPL_KERNEL_OFFS would be where we say nandimgsrc
is in env.  Stefano, looking at twister, the spl export looks like it
goes right at the start of the rootfs, isn't that bad?  Or at least,
luck?

-- 
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/20130327/9cacd03a/attachment.pgp>