Hi Han,
On Wed, Mar 23, 2022 at 6:25 PM Han Xu <han...@nxp.com> wrote:
>
> Previous u-boot code changed the default bch setting behavior and caused
> backward compatible issue. This fix choose the legacy bch geometry back
> again as the default option. If the minimum ecc strength that NAND chips
> required need to be chosen, it can be enabled by either adding DT flag
> "fsl,use-minimum-ecc" or CONFIG_NAND_MXS_USE_MINIMUM_ECC in configs. The
> unused flag "fsl,legacy-bch-geometry" get removed.
>
> Fixes: 51cdf83eea mtd: gpmi: provide the option to use legacy bch geometry
> Fixes: 616f03daba mtd: gpmi: change the BCH layout setting for large oob NAND
The correct syntax for the Fixes tag is:
Fixes: 51cdf83eea45 ("mtd: gpmi: provide the option to use legacy bch geometry")
Fixes: 616f03dabacb ("mtd: gpmi: change the BCH layout setting for
large oob NAND")
> Tested-by: Tim Harvey <thar...@gateworks.com>
> Signed-off-by: Han Xu <han...@nxp.com>
>
> Changes in v2:
> -- change the commit log about backward compatible issue is in u-boot
> -- removed the unused flag
> -- add fixes and test tag
The "Changes in v2" part should go below the --- line.
