diff mbox

[1/6] ARM: dts: am43x-epos-evm: Use BCH16 ECC scheme instead of BCH8

Message ID 1409666227-20622-2-git-send-email-rogerq@ti.com
State Accepted
Commit db01e6c7fbe3b87b389f537b52a5d862cea498e1
Headers show

Commit Message

Roger Quadros Sept. 2, 2014, 1:57 p.m. UTC
am43x-epos-evm uses a NAND chip with page size 4096 bytes
and spare area of 225 bytes per page.

For such a setup it is preferrable to use BCH16 ECC scheme over
BCH8. This also makes it compatible with ROM code ECC scheme so
we can boot with NAND after flashing from kernel.

Signed-off-by: Roger Quadros <rogerq@ti.com>
---
 arch/arm/boot/dts/am43x-epos-evm.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

pekon Sept. 2, 2014, 6:29 p.m. UTC | #1
On Tuesday 02 September 2014 07:27 PM, Roger Quadros wrote:
> am43x-epos-evm uses a NAND chip with page size 4096 bytes
> and spare area of 225 bytes per page.
>
> For such a setup it is preferrable to use BCH16 ECC scheme over
> BCH8. This also makes it compatible with ROM code ECC scheme so
> we can boot with NAND after flashing from kernel.
>
> Signed-off-by: Roger Quadros <rogerq@ti.com>
> ---
>   arch/arm/boot/dts/am43x-epos-evm.dts | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
> index ed7dd23..f6c9898 100644
> --- a/arch/arm/boot/dts/am43x-epos-evm.dts
> +++ b/arch/arm/boot/dts/am43x-epos-evm.dts
> @@ -441,7 +441,7 @@
>   	ranges = <0 0 0x08000000 0x10000000>;	/* CS0: NAND */
>   	nand@0,0 {
>   		reg = <0 0 0>; /* CS0, offset 0 */
> -		ti,nand-ecc-opt = "bch8";
> +		ti,nand-ecc-opt = "bch16";
>   		ti,elm-id = <&elm>;
>   		nand-bus-width = <8>;
>   		gpmc,device-width = <1>;
>

As I have tested BCH16 on AM43x-EPOS board earlier So,
Reviewed-by: Pekon Gupta <pekon@pek-sem.com>


with regards, pekon

------------------------
Powered by BigRock.com

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index ed7dd23..f6c9898 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -441,7 +441,7 @@ 
 	ranges = <0 0 0x08000000 0x10000000>;	/* CS0: NAND */
 	nand@0,0 {
 		reg = <0 0 0>; /* CS0, offset 0 */
-		ti,nand-ecc-opt = "bch8";
+		ti,nand-ecc-opt = "bch16";
 		ti,elm-id = <&elm>;
 		nand-bus-width = <8>;
 		gpmc,device-width = <1>;