Message ID | 1419901957-22787-11-git-send-email-sjg@chromium.org |
---|---|
State | Superseded |
Delegated to: | Simon Glass |
Headers | show |
Hi Simon, On Tue, Dec 30, 2014 at 9:12 AM, Simon Glass <sjg@chromium.org> wrote: > Enable an environment area. > > Signed-off-by: Simon Glass <sjg@chromium.org> > --- > > include/configs/chromebook_link.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h > index 22cb134..9b70435 100644 > --- a/include/configs/chromebook_link.h > +++ b/include/configs/chromebook_link.h > @@ -68,6 +68,11 @@ > #define CONFIG_CMD_CROS_EC > #define CONFIG_ARCH_EARLY_INIT_R > > +#undef CONFIG_ENV_IS_NOWHERE > +#define CONFIG_ENV_IS_IN_SPI_FLASH I don't see CONFIG_ENV_SIZE is defined anywhere. Will that work without CONFIG_ENV_SIZE? > +#define CONFIG_ENV_OFFSET 0x003f8000 > +#define CONFIG_ENV_SECT_SIZE 256 I think the CONFIG_ENV_SECT_SIZE should match the SPI flash sector size. > #define CONFIG_STD_DEVICES_SETTINGS "stdin=usbkbd,vga,serial\0" \ > "stdout=vga,serial\0" \ > "stderr=vga,serial\0" > -- Regards, Bin
Hi Bin, On 4 January 2015 at 01:08, Bin Meng <bmeng.cn@gmail.com> wrote: > Hi Simon, > > On Tue, Dec 30, 2014 at 9:12 AM, Simon Glass <sjg@chromium.org> wrote: >> Enable an environment area. >> >> Signed-off-by: Simon Glass <sjg@chromium.org> >> --- >> >> include/configs/chromebook_link.h | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h >> index 22cb134..9b70435 100644 >> --- a/include/configs/chromebook_link.h >> +++ b/include/configs/chromebook_link.h >> @@ -68,6 +68,11 @@ >> #define CONFIG_CMD_CROS_EC >> #define CONFIG_ARCH_EARLY_INIT_R >> >> +#undef CONFIG_ENV_IS_NOWHERE >> +#define CONFIG_ENV_IS_IN_SPI_FLASH > > I don't see CONFIG_ENV_SIZE is defined anywhere. Will that work > without CONFIG_ENV_SIZE? That's in x86_common.h > >> +#define CONFIG_ENV_OFFSET 0x003f8000 >> +#define CONFIG_ENV_SECT_SIZE 256 > > I think the CONFIG_ENV_SECT_SIZE should match the SPI flash sector size. Ah yes I think this should be 4096. Regards, Simon
Hi Simon, On Mon, Jan 5, 2015 at 9:54 AM, Simon Glass <sjg@chromium.org> wrote: > Hi Bin, > > On 4 January 2015 at 01:08, Bin Meng <bmeng.cn@gmail.com> wrote: >> Hi Simon, >> >> On Tue, Dec 30, 2014 at 9:12 AM, Simon Glass <sjg@chromium.org> wrote: >>> Enable an environment area. >>> >>> Signed-off-by: Simon Glass <sjg@chromium.org> >>> --- >>> >>> include/configs/chromebook_link.h | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h >>> index 22cb134..9b70435 100644 >>> --- a/include/configs/chromebook_link.h >>> +++ b/include/configs/chromebook_link.h >>> @@ -68,6 +68,11 @@ >>> #define CONFIG_CMD_CROS_EC >>> #define CONFIG_ARCH_EARLY_INIT_R >>> >>> +#undef CONFIG_ENV_IS_NOWHERE >>> +#define CONFIG_ENV_IS_IN_SPI_FLASH >> >> I don't see CONFIG_ENV_SIZE is defined anywhere. Will that work >> without CONFIG_ENV_SIZE? > > That's in x86_common.h Ah, I see, thanks. Maybe we need just redefine it here since I believe CONFIG_ENV_SIZE should match to multiple times of CONFIG_ENV_SECT_SIZE. >> >>> +#define CONFIG_ENV_OFFSET 0x003f8000 >>> +#define CONFIG_ENV_SECT_SIZE 256 >> >> I think the CONFIG_ENV_SECT_SIZE should match the SPI flash sector size. > > Ah yes I think this should be 4096. Regards, Bin
diff --git a/include/configs/chromebook_link.h b/include/configs/chromebook_link.h index 22cb134..9b70435 100644 --- a/include/configs/chromebook_link.h +++ b/include/configs/chromebook_link.h @@ -68,6 +68,11 @@ #define CONFIG_CMD_CROS_EC #define CONFIG_ARCH_EARLY_INIT_R +#undef CONFIG_ENV_IS_NOWHERE +#define CONFIG_ENV_IS_IN_SPI_FLASH +#define CONFIG_ENV_OFFSET 0x003f8000 +#define CONFIG_ENV_SECT_SIZE 256 + #define CONFIG_STD_DEVICES_SETTINGS "stdin=usbkbd,vga,serial\0" \ "stdout=vga,serial\0" \ "stderr=vga,serial\0"
Enable an environment area. Signed-off-by: Simon Glass <sjg@chromium.org> --- include/configs/chromebook_link.h | 5 +++++ 1 file changed, 5 insertions(+)