diff mbox

[2/6] ARM: dts: am437x-gp-evm: Use BCH16 ECC scheme instead of BCH8

Message ID 1409666227-20622-3-git-send-email-rogerq@ti.com
State Accepted
Commit 6b8691100538427ad739fc8735451a30c73316ff
Headers show

Commit Message

Roger Quadros Sept. 2, 2014, 1:57 p.m. UTC
am437x-gp-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/am437x-gp-evm.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

pekon Sept. 2, 2014, 6:30 p.m. UTC | #1
On Tuesday 02 September 2014 07:27 PM, Roger Quadros wrote:
> am437x-gp-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/am437x-gp-evm.dts | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
> index 646a6ea..402249e 100644
> --- a/arch/arm/boot/dts/am437x-gp-evm.dts
> +++ b/arch/arm/boot/dts/am437x-gp-evm.dts
> @@ -424,7 +424,7 @@
>   	ranges = <0 0 0 0x01000000>;	/* minimum GPMC partition = 16MB */
>   	nand@0,0 {
>   		reg = <0 0 4>;		/* device IO registers */
> -		ti,nand-ecc-opt = "bch8";
> +		ti,nand-ecc-opt = "bch16";
>   		ti,elm-id = <&elm>;
>   		nand-bus-width = <8>;
>   		gpmc,device-width = <1>;
>
For already mentioned reason ..
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/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
index 646a6ea..402249e 100644
--- a/arch/arm/boot/dts/am437x-gp-evm.dts
+++ b/arch/arm/boot/dts/am437x-gp-evm.dts
@@ -424,7 +424,7 @@ 
 	ranges = <0 0 0 0x01000000>;	/* minimum GPMC partition = 16MB */
 	nand@0,0 {
 		reg = <0 0 4>;		/* device IO registers */
-		ti,nand-ecc-opt = "bch8";
+		ti,nand-ecc-opt = "bch16";
 		ti,elm-id = <&elm>;
 		nand-bus-width = <8>;
 		gpmc,device-width = <1>;