[U-Boot] [PATCH 1/8] mtd: nand: sunxi: Fix strength minimum value

Miquel Raynal miquel.raynal at free-electrons.com
Wed Jan 24 22:21:29 UTC 2018


Hello Maxime,

On Wed, 24 Jan 2018 08:46:18 +0100
Maxime Ripard <maxime.ripard at free-electrons.com> wrote:

> Hi,
> 
> On Wed, Jan 24, 2018 at 01:44:47AM +0100, Miquel Raynal wrote:
> > When no requirement in Device Tree is given about the ECC strength and
> > step size, the engine should fallback on the minimal working case for
> > this engine (16b/1024B) instead of the NAND chip requirement which might
> > be simply unreachable.
> > 
> > Signed-off-by: Miquel Raynal <miquel.raynal at free-electrons.com>
> > ---
> >  drivers/mtd/nand/sunxi_nand.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/mtd/nand/sunxi_nand.c b/drivers/mtd/nand/sunxi_nand.c
> > index 8bc3828854..e8e7ad8ac5 100644
> > --- a/drivers/mtd/nand/sunxi_nand.c
> > +++ b/drivers/mtd/nand/sunxi_nand.c
> > @@ -1417,6 +1417,7 @@ static int sunxi_nand_hw_common_ecc_ctrl_init(struct mtd_info *mtd,
> >  		goto err;
> >  	}
> >  
> > +	ecc->strength = strengths[i];  
> 
> You should have a comment here as well mentionning why you're doing
> this (your commit log would be a good start :))

Sure, I also moved that line above, where the strength is selected
(with a comment). This way it seems more clear to me why this is needed.

Thanks,
Miquèl


More information about the U-Boot mailing list