* Gupta, Pekon <[email protected]> [140713 22:43]:
> >From: Christoph Fritz [mailto:[email protected]]
> >
> >This patch adds bch8 ecc software fallback which is mostly used by
> >omap3s because they lack hardware elm support.
> >
> >Fixes: 0611c41934ab35ce84dea34ab291897ad3cbc7be (ARM: OMAP2+: gpmc:
> >update gpmc_hwecc_bch_capable() for new platforms and ECC schemes)
> >Cc: <[email protected]> # 3.15.x+
> >Signed-off-by: Christoph Fritz <[email protected]>
> >---
> > arch/arm/mach-omap2/gpmc-nand.c |   18 ++++++++++--------
> > 1 file changed, 10 insertions(+), 8 deletions(-)
> >
> >diff --git a/arch/arm/mach-omap2/gpmc-nand.c 
> >b/arch/arm/mach-omap2/gpmc-nand.c
> >index 17cd393..93914d2 100644
> >--- a/arch/arm/mach-omap2/gpmc-nand.c
> >+++ b/arch/arm/mach-omap2/gpmc-nand.c
> >@@ -50,6 +50,16 @@ static bool gpmc_hwecc_bch_capable(enum omap_ecc ecc_opt)
> >              soc_is_omap54xx() || soc_is_dra7xx())
> >             return 1;
> >
> >+    if (ecc_opt == OMAP_ECC_BCH4_CODE_HW_DETECTION_SW ||
> >+             ecc_opt == OMAP_ECC_BCH8_CODE_HW_DETECTION_SW) {
> >+            if (cpu_is_omap24xx())
> >+                    return 0;
> >+            else if (cpu_is_omap3630() && (GET_OMAP_REVISION() == 0))
> >+                    return 0;
> >+            else
> >+                    return 1;
> >+    }
> >+
> >     /* OMAP3xxx do not have ELM engine, so cannot support ECC schemes
> >      * which require H/W based ECC error detection */
> >     if ((cpu_is_omap34xx() || cpu_is_omap3630()) &&
> >@@ -57,14 +67,6 @@ static bool gpmc_hwecc_bch_capable(enum omap_ecc ecc_opt)
> >              (ecc_opt == OMAP_ECC_BCH8_CODE_HW)))
> >             return 0;
> >
> >-    /*
> >-     * For now, assume 4-bit mode is only supported on OMAP3630 ES1.x, x>=1
> >-     * and AM33xx derivates. Other chips may be added if confirmed to work.
> >-     */
> >-    if ((ecc_opt == OMAP_ECC_BCH4_CODE_HW_DETECTION_SW) &&
> >-        (!cpu_is_omap3630() || (GET_OMAP_REVISION() == 0)))
> >-            return 0;
> >-
> >     /* legacy platforms support only HAM1 (1-bit Hamming) ECC scheme */
> >     if (ecc_opt == OMAP_ECC_HAM1_CODE_HW)
> >             return 1;
> >--
> >1.7.10.4
> >
> Thanks much for this fix.
> Reviewed-by: Pekon Gupta <[email protected]>

Applying into omap-for-v3.16/fixes thanks.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to