On Sat, Dec 21, 2019 at 05:18:22PM +0100, Patrik Dahlström wrote: > The omap3_beagle NAND ECC scheme was changed in 4b37928d357 for unknown > reasons, leading to uncorrectible ecc errors. This commit changes it > back to what it was before. >
Hello Patrick, Is there a setup/test that you are using for OMAP_ECC_HAM1_CODE_HW? I just want to give it a try. I have three OMAP3 boards, with NAND, that I would like to test. I also see that the SYS_NAND_ECC_BYTES should have been changed to '14' per the 'doc/README.nand' for OMAP_ECC_BCH8_CODE_HW_DETECTION_SW. This may be the issue with this particular ECC scheme. Derald > Signed-off-by: Patrik Dahlström <ri...@dalakolonin.se> > --- > include/configs/omap3_beagle.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h > index 4157d7614f..bc8aa7adf5 100644 > --- a/include/configs/omap3_beagle.h > +++ b/include/configs/omap3_beagle.h > @@ -37,7 +37,7 @@ > 10, 11, 12, 13} > #define CONFIG_SYS_NAND_ECCSIZE 512 > #define CONFIG_SYS_NAND_ECCBYTES 3 > -#define CONFIG_NAND_OMAP_ECCSCHEME OMAP_ECC_BCH8_CODE_HW_DETECTION_SW > +#define CONFIG_NAND_OMAP_ECCSCHEME OMAP_ECC_HAM1_CODE_HW > #define CONFIG_SYS_NAND_U_BOOT_OFFS 0x80000 > #define CONFIG_SYS_ENV_SECT_SIZE SZ_128K > #define CONFIG_ENV_OVERWRITE > -- > 2.17.1 >
