diff mbox

Documentation: dt: mtd: drop "soft_bch" from example

Message ID a206ce4cabf9ba8d376241fc23e9973f6db2443b.1464147910.git.baruch@tkos.co.il
State Accepted
Commit 507e617440d70bc4269daac1b0ee96671bf2761a
Headers show

Commit Message

Baruch Siach May 25, 2016, 3:45 a.m. UTC
Commit 32698aafc9f0 (Documentation: devicetree: deprecate "soft_bch"
nand-ecc-mode value, 2016-04-22) deprecated "soft_bch". Update the example to
match.

Cc: Rafał Miłecki <zajec5@gmail.com>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
 Documentation/devicetree/bindings/mtd/nand.txt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Rafał Miłecki May 25, 2016, 7:26 a.m. UTC | #1
On 25 May 2016 at 05:45, Baruch Siach <baruch@tkos.co.il> wrote:
> Commit 32698aafc9f0 (Documentation: devicetree: deprecate "soft_bch"
> nand-ecc-mode value, 2016-04-22) deprecated "soft_bch". Update the example to
> match.
>
> Cc: Rafał Miłecki <zajec5@gmail.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Acked-by: Rafał Miłecki <zajec5@gmail.com>
Rob Herring (Arm) May 25, 2016, 7:02 p.m. UTC | #2
On Wed, May 25, 2016 at 06:45:10AM +0300, Baruch Siach wrote:
> Commit 32698aafc9f0 (Documentation: devicetree: deprecate "soft_bch"
> nand-ecc-mode value, 2016-04-22) deprecated "soft_bch". Update the example to
> match.
> 
> Cc: Rafał Miłecki <zajec5@gmail.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
>  Documentation/devicetree/bindings/mtd/nand.txt | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Acked-by: Rob Herring <robh@kernel.org>
Boris Brezillon May 27, 2016, 1:31 p.m. UTC | #3
Brian,

On Wed, 25 May 2016 06:45:10 +0300
Baruch Siach <baruch@tkos.co.il> wrote:

> Commit 32698aafc9f0 (Documentation: devicetree: deprecate "soft_bch"
> nand-ecc-mode value, 2016-04-22) deprecated "soft_bch". Update the example to
> match.
> 
> Cc: Rafał Miłecki <zajec5@gmail.com>
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>

I don't know if we should consider this as a bugfix. If that's the
case, can you take it directly in your tree with my

Acked-by: Boris Brezillon <boris.brezillon@free-electrons.com>

Thanks,

Boris

> ---
>  Documentation/devicetree/bindings/mtd/nand.txt | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mtd/nand.txt b/Documentation/devicetree/bindings/mtd/nand.txt
> index 68342eac2383..3733300de8dd 100644
> --- a/Documentation/devicetree/bindings/mtd/nand.txt
> +++ b/Documentation/devicetree/bindings/mtd/nand.txt
> @@ -53,7 +53,8 @@ Example:
>  
>  		nand@0 {
>  			reg = <0>;
> -			nand-ecc-mode = "soft_bch";
> +			nand-ecc-mode = "soft";
> +			nand-ecc-algo = "bch";
>  
>  			/* controller specific properties */
>  		};
Brian Norris May 27, 2016, 4:58 p.m. UTC | #4
On Fri, May 27, 2016 at 03:31:53PM +0200, Boris Brezillon wrote:
> On Wed, 25 May 2016 06:45:10 +0300
> Baruch Siach <baruch@tkos.co.il> wrote:
> 
> > Commit 32698aafc9f0 (Documentation: devicetree: deprecate "soft_bch"
> > nand-ecc-mode value, 2016-04-22) deprecated "soft_bch". Update the example to
> > match.
> > 
> > Cc: Rafał Miłecki <zajec5@gmail.com>
> > Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> 
> I don't know if we should consider this as a bugfix. If that's the
> case, can you take it directly in your tree with my

Well, I don't know if it's a "bugfix" per se, but it's obviously correct
and has approx 0 risk.

> Acked-by: Boris Brezillon <boris.brezillon@free-electrons.com>

Wow, thanks! Three acks! Y'all must really love documentation :)

Pushed to linux-mtd.git.

Brian
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/mtd/nand.txt b/Documentation/devicetree/bindings/mtd/nand.txt
index 68342eac2383..3733300de8dd 100644
--- a/Documentation/devicetree/bindings/mtd/nand.txt
+++ b/Documentation/devicetree/bindings/mtd/nand.txt
@@ -53,7 +53,8 @@  Example:
 
 		nand@0 {
 			reg = <0>;
-			nand-ecc-mode = "soft_bch";
+			nand-ecc-mode = "soft";
+			nand-ecc-algo = "bch";
 
 			/* controller specific properties */
 		};