[PATCH] omap3_beagle: Change NAND ECC scheme back to OMAP_ECC_HAM1_CODE_HW

[Derald D. Woods]

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 <risca at 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
>