[U-Boot] [PATCH] arm: am57xx: Keep environment in eMMC

[Tom Rini]

On Fri, Jun 09, 2017 at 05:24:16PM +0300, Sam Protsenko wrote:
> On 9 June 2017 at 17:17, Tom Rini <trini at konsulko.com> wrote:
> > On Fri, Jun 09, 2017 at 05:12:56PM +0300, Sam Protsenko wrote:
> >
> >> Use eMMC (instead of SD card) to store U-Boot environment. Use
> >> "reserved" partition for U-Boot environment.
> >>
> >> Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
> >> ---
> >>  include/configs/am57xx_evm.h | 14 +++++++++-----
> >>  1 file changed, 9 insertions(+), 5 deletions(-)
> >>
> >> diff --git a/include/configs/am57xx_evm.h b/include/configs/am57xx_evm.h
> >> index af19098727..7a42d79647 100644
> >> --- a/include/configs/am57xx_evm.h
> >> +++ b/include/configs/am57xx_evm.h
> >> @@ -13,6 +13,7 @@
> >>  #define __CONFIG_AM57XX_EVM_H
> >>
> >>  #include <environment/ti/dfu.h>
> >> +#include <linux/sizes.h>
> >>
> >>  #ifdef CONFIG_SPL_BUILD
> >>  #define CONFIG_IODELAY_RECALIBRATION
> >> @@ -20,11 +21,14 @@
> >>
> >>  #define CONFIG_NR_DRAM_BANKS         2
> >>
> >> -#define CONFIG_ENV_SIZE                      (64 << 10)
> >> -#define CONFIG_ENV_IS_IN_FAT
> >> -#define FAT_ENV_INTERFACE            "mmc"
> >> -#define FAT_ENV_DEVICE_AND_PART              "0:1"
> >> -#define FAT_ENV_FILE                 "uboot.env"
> >> +/* MMC ENV related defines */
> >> +#define CONFIG_ENV_IS_IN_MMC
> >> +#define CONFIG_SYS_MMC_ENV_DEV               1               /* eMMC */
> >> +#define CONFIG_SYS_MMC_ENV_PART              0
> >> +#define CONFIG_ENV_SIZE                      SZ_128K
> >> +#define CONFIG_ENV_OFFSET            0x260000
> >> +#define CONFIG_ENV_OFFSET_REDUND     (CONFIG_ENV_OFFSET + CONFIG_ENV_SIZE)
> >> +#define CONFIG_SYS_REDUNDAND_ENVIRONMENT
> >
> > Why are we doing this?
> 
> This is exactly how it's done for DRA7 EVM board. And since those two
> boards are basically the same family, we should stick to one way of
> doing things here. And we agreed with Praneeth that this policy is
> booting Android from eMMC.

Right, but Android isn't the only use-case here.  If Lokesh acks it, I'm
fine with it.  But, to be clear, are there am57xx_evm targets that lack
eMMC?  I don't want to get into the case we used to have on am335x_evm
where we assume NAND for env, and run on many targets without NAND.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170609/2d2153f2/attachment.sig>