Re: [U-Boot] [PATCH] env_onenand: set ONENAND_MAX_ENV_SIZE to CONFIG_ENV_SIZE
Dear David du Colombier, > > This fix prevents env_import() CRC to fail when CONFIG_ENV_SIZE > > is not equal to 4096 bytes > > It also prevents mtd->read and mtd->write to be incomplete when > > the environment is larger than 4096 bytes. > > > > Signed-off-by: David du Colombier <0in...@gmail.com> > > --- > > common/env_onenand.c |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/common/env_onenand.c b/common/env_onenand.c > > index 7197ab6..da35071 100644 > > --- a/common/env_onenand.c > > +++ b/common/env_onenand.c > > @@ -39,7 +39,7 @@ > > > > char *env_name_spec = "OneNAND"; > > > > -#define ONENAND_MAX_ENV_SIZE 4096 > > +#define ONENAND_MAX_ENV_SIZE CONFIG_ENV_SIZE > > #define ONENAND_ENV_SIZE(mtd) (ONENAND_MAX_ENV_SIZE - > > ENV_HEADER_SIZE) > > DECLARE_GLOBAL_DATA_PTR; > > Could you please take a look? It fixes environment > saving and restoring on IGEPv2. > > Thanks. > Tested-by: Lukasz Majewski Test HW: S5PC110 GONI Applied to u-boot-onenand repository, -- Best regards, Lukasz Majewski Samsung Poland R&D Center | Linux Platform Group ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] env_onenand: set ONENAND_MAX_ENV_SIZE to CONFIG_ENV_SIZE
> This fix prevents env_import() CRC to fail when CONFIG_ENV_SIZE > is not equal to 4096 bytes > It also prevents mtd->read and mtd->write to be incomplete when > the environment is larger than 4096 bytes. > > Signed-off-by: David du Colombier <0in...@gmail.com> > --- > common/env_onenand.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/common/env_onenand.c b/common/env_onenand.c > index 7197ab6..da35071 100644 > --- a/common/env_onenand.c > +++ b/common/env_onenand.c > @@ -39,7 +39,7 @@ > > char *env_name_spec = "OneNAND"; > > -#define ONENAND_MAX_ENV_SIZE 4096 > +#define ONENAND_MAX_ENV_SIZE CONFIG_ENV_SIZE > #define ONENAND_ENV_SIZE(mtd)(ONENAND_MAX_ENV_SIZE - > ENV_HEADER_SIZE) > DECLARE_GLOBAL_DATA_PTR; Could you please take a look? It fixes environment saving and restoring on IGEPv2. Thanks. -- David du Colombier ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] env_onenand: set ONENAND_MAX_ENV_SIZE to CONFIG_ENV_SIZE
This fix prevents env_import() CRC to fail when CONFIG_ENV_SIZE is not equal to 4096 bytes It also prevents mtd->read and mtd->write to be incomplete when the environment is larger than 4096 bytes. Signed-off-by: David du Colombier <0in...@gmail.com> --- common/env_onenand.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/common/env_onenand.c b/common/env_onenand.c index 7197ab6..da35071 100644 --- a/common/env_onenand.c +++ b/common/env_onenand.c @@ -39,7 +39,7 @@ char *env_name_spec = "OneNAND"; -#define ONENAND_MAX_ENV_SIZE 4096 +#define ONENAND_MAX_ENV_SIZE CONFIG_ENV_SIZE #define ONENAND_ENV_SIZE(mtd) (ONENAND_MAX_ENV_SIZE - ENV_HEADER_SIZE) DECLARE_GLOBAL_DATA_PTR; -- 1.7.6.5 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot