On Mon, Oct 17, 2016 at 04:17:06PM -0600, Simon Glass wrote:
> Hi Peng,
> 
> On 8 October 2016 at 03:06, Peng Fan <peng....@nxp.com> wrote:
> > Add Kconfig entry for ENV storage
> >
> > Signed-off-by: Peng Fan <peng....@nxp.com>
> > Cc: Simon Glass <s...@chromium.org>
> > Cc: Heiko Schocher <h...@denx.de>
> > Cc: Tom Rini <tr...@konsulko.com>
> > Cc: Masahiro Yamada <yamada.masah...@socionext.com>
> > Cc: Stefan Roese <s...@denx.de>
> > Cc: Siva Durga Prasad Paladugu <siva.durga.palad...@xilinx.com>
> > Cc: Francois Retief <fgret...@spaceteq.co.za>
> > ---
> >  common/Kconfig | 73 
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 73 insertions(+)
> >
> > diff --git a/common/Kconfig b/common/Kconfig
> > index c69c141..a630924 100644
> > --- a/common/Kconfig
> > +++ b/common/Kconfig
> > @@ -158,6 +158,79 @@ config SPI_BOOT
> >
> >  endmenu
> >
> > +menu "ENV storage media"
> > +
> > +config ENV_IS_IN_DATAFLASH
> > +       bool "env stored in dataflash"
> > +       help
> > +         env stored in dataflash
> 
> Can you add a little more help detail on this? Also I suspect we
> should have CONFIG_ENV as well, to enable the actual environment
> (independent of where it is stored).

Yes, please add a bit more to the text.

> Also, can you migrate the existing configs with moveconfig? That would
> be a big step forward and may inspire me to move it to linker lists.

Actually, I bet this is going to be messy enough that I'm fine running
the moveconfig myself right before I merge it in.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to