2016-04-17 20:53:37

by Rafał Miłecki

[permalink] [raw]
Subject: [PATCH V3 04/11] CRIS v32: nand: set ECC algorithm explicitly

This is part of process deprecating NAND_ECC_SOFT_BCH (and switching to
enum nand_ecc_algo).

Signed-off-by: Rafał Miłecki <[email protected]>
---
arch/cris/arch-v32/drivers/mach-a3/nandflash.c | 1 +
arch/cris/arch-v32/drivers/mach-fs/nandflash.c | 1 +
2 files changed, 2 insertions(+)

diff --git a/arch/cris/arch-v32/drivers/mach-a3/nandflash.c b/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
index 5aa3f51..3f646c7 100644
--- a/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
+++ b/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
@@ -157,6 +157,7 @@ struct mtd_info *__init crisv32_nand_flash_probe(void)
/* 20 us command delay time */
this->chip_delay = 20;
this->ecc.mode = NAND_ECC_SOFT;
+ this->ecc.algo = NAND_ECC_HAMMING;

/* Enable the following for a flash based bad block table */
/* this->bbt_options = NAND_BBT_USE_FLASH; */
diff --git a/arch/cris/arch-v32/drivers/mach-fs/nandflash.c b/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
index a7c17b0..a745405 100644
--- a/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
+++ b/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
@@ -148,6 +148,7 @@ struct mtd_info *__init crisv32_nand_flash_probe(void)
/* 20 us command delay time */
this->chip_delay = 20;
this->ecc.mode = NAND_ECC_SOFT;
+ this->ecc.algo = NAND_ECC_HAMMING;

/* Enable the following for a flash based bad block table */
/* this->bbt_options = NAND_BBT_USE_FLASH; */
--
1.8.4.5


2016-04-21 20:18:15

by Boris Brezillon

[permalink] [raw]
Subject: Re: [PATCH V3 04/11] CRIS v32: nand: set ECC algorithm explicitly

On Sun, 17 Apr 2016 22:53:00 +0200
Rafał Miłecki <[email protected]> wrote:

> This is part of process deprecating NAND_ECC_SOFT_BCH (and switching to
> enum nand_ecc_algo).
>
> Signed-off-by: Rafał Miłecki <[email protected]>

Mikael, Jesper, could you review/ack this patch? I'd like to take it
into the nand tree to avoid any dependency problems.

Thanks,

Boris

> ---
> arch/cris/arch-v32/drivers/mach-a3/nandflash.c | 1 +
> arch/cris/arch-v32/drivers/mach-fs/nandflash.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/arch/cris/arch-v32/drivers/mach-a3/nandflash.c b/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
> index 5aa3f51..3f646c7 100644
> --- a/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
> +++ b/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
> @@ -157,6 +157,7 @@ struct mtd_info *__init crisv32_nand_flash_probe(void)
> /* 20 us command delay time */
> this->chip_delay = 20;
> this->ecc.mode = NAND_ECC_SOFT;
> + this->ecc.algo = NAND_ECC_HAMMING;
>
> /* Enable the following for a flash based bad block table */
> /* this->bbt_options = NAND_BBT_USE_FLASH; */
> diff --git a/arch/cris/arch-v32/drivers/mach-fs/nandflash.c b/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
> index a7c17b0..a745405 100644
> --- a/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
> +++ b/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
> @@ -148,6 +148,7 @@ struct mtd_info *__init crisv32_nand_flash_probe(void)
> /* 20 us command delay time */
> this->chip_delay = 20;
> this->ecc.mode = NAND_ECC_SOFT;
> + this->ecc.algo = NAND_ECC_HAMMING;
>
> /* Enable the following for a flash based bad block table */
> /* this->bbt_options = NAND_BBT_USE_FLASH; */



--
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

2016-04-21 20:44:43

by Mikael Starvik

[permalink] [raw]
Subject: Re: [PATCH V3 04/11] CRIS v32: nand: set ECC algorithm explicitly

Acked-by: Mikael Starvik <[email protected]>

> 17 apr. 2016 kl. 22:53 skrev Rafa? Mi?ecki <[email protected]>:
>
> This is part of process deprecating NAND_ECC_SOFT_BCH (and switching to
> enum nand_ecc_algo).
>
> Signed-off-by: Rafa? Mi?ecki <[email protected]>
> ---
> arch/cris/arch-v32/drivers/mach-a3/nandflash.c | 1 +
> arch/cris/arch-v32/drivers/mach-fs/nandflash.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/arch/cris/arch-v32/drivers/mach-a3/nandflash.c b/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
> index 5aa3f51..3f646c7 100644
> --- a/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
> +++ b/arch/cris/arch-v32/drivers/mach-a3/nandflash.c
> @@ -157,6 +157,7 @@ struct mtd_info *__init crisv32_nand_flash_probe(void)
> /* 20 us command delay time */
> this->chip_delay = 20;
> this->ecc.mode = NAND_ECC_SOFT;
> + this->ecc.algo = NAND_ECC_HAMMING;
>
> /* Enable the following for a flash based bad block table */
> /* this->bbt_options = NAND_BBT_USE_FLASH; */
> diff --git a/arch/cris/arch-v32/drivers/mach-fs/nandflash.c b/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
> index a7c17b0..a745405 100644
> --- a/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
> +++ b/arch/cris/arch-v32/drivers/mach-fs/nandflash.c
> @@ -148,6 +148,7 @@ struct mtd_info *__init crisv32_nand_flash_probe(void)
> /* 20 us command delay time */
> this->chip_delay = 20;
> this->ecc.mode = NAND_ECC_SOFT;
> + this->ecc.algo = NAND_ECC_HAMMING;
>
> /* Enable the following for a flash based bad block table */
> /* this->bbt_options = NAND_BBT_USE_FLASH; */
> --
> 1.8.4.5
>

2016-04-22 07:26:32

by Jesper Nilsson

[permalink] [raw]
Subject: Re: [PATCH V3 04/11] CRIS v32: nand: set ECC algorithm explicitly

On Sun, Apr 17, 2016 at 10:53:00PM +0200, Rafał Miłecki wrote:
> This is part of process deprecating NAND_ECC_SOFT_BCH (and switching to
> enum nand_ecc_algo).
>
> Signed-off-by: Rafał Miłecki <[email protected]>

Acked-by: Jesper Nilsson <[email protected]>


/^JN - Jesper Nilsson
--
Jesper Nilsson -- [email protected]