[U-Boot] [PATCH V2 2/3] mtd: nand: mxs check maximum ecc that platfrom supports

Scott Wood scottwood at freescale.com
Wed Aug 26 04:14:58 CEST 2015


On Wed, 2015-08-26 at 08:33 +0800, Peng Fan wrote:
> Hi Scott,
> 
> On Tue, Aug 25, 2015 at 04:05:56PM -0500, Scott Wood wrote:
> > On Tue, 2015-07-21 at 16:15 +0800, Peng Fan wrote:
> > > Check maximum ecc strength for each platfrom to avoid the calculated ecc
> > > exceed the limitation.
> > > 
> > > Signed-off-by: Peng Fan <Peng.Fan at freescale.com>
> > > Signed-off-by: Han Xu <b45815 at freescale.com>
> > > Reviewed-by: Marek Vasut <marex at denx.de>
> > > ---
> > > 
> > > Changes v2:
> > >  Add Marek's reviewed by.
> > > 
> > >  drivers/mtd/nand/mxs_nand.c | 9 ++++++++-
> > >  1 file changed, 8 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c
> > > index 33ce817..97011b2 100644
> > > --- a/drivers/mtd/nand/mxs_nand.c
> > > +++ b/drivers/mtd/nand/mxs_nand.c
> > > @@ -149,6 +149,13 @@ static inline uint32_t 
> > > mxs_nand_get_ecc_strength(uint32_t page_data_size,
> > >                                               uint32_t page_oob_size)
> > >  {
> > >       int ecc_strength;
> > > +     int max_ecc_strength_supported;
> > > +
> > > +     /* Refer to Chapter 17 for i.MX6DQ, Chapter 18 for i.MX6SX */
> > > +     if (is_cpu_type(MXC_CPU_MX6SX))
> > > +             max_ecc_strength_supported = 62;
> > > +     else
> > > +             max_ecc_strength_supported = 40;
> > 
> >       arm:  +   mx28evk_nand                  
> > +drivers/mtd/nand/mxs_nand.c:155:18: error: 'MXC_CPU_MX6SX' undeclared 
> > (first use in this function)
> > +drivers/mtd/nand/mxs_nand.c:155:18: note: each undeclared identifier is 
> > reported only once for each function it appears in
> > +make[2]: *** [drivers/mtd/nand/mxs_nand.o] Error 1
> > +make[1]: *** [drivers/mtd/nand] Error 2
> > +make: *** [sub-make] Error 2
> 
> My bad. I only take mx6 into consideration when I did this patch.
> 
> > 
> > (among other failed targets)
> > 
> > I tried to fix it by including asm/arch-imx/cpu.h, but then got undefined 
> > reference to is_cpu_type().
> 
> Now is_cpu_type() is common to all i.MXes, but need this patch 
> http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commitdiff;h=bf3303c98b23a29d99acc5e28865db604873b699 
> which is still in imx tree now.
> After applying the following patches, is_cpu_type should work.
> http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commit;h=7c015efa7f28911d538ddd7d12926edd95176791
> http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commit;h=8b647df9cae0ca65656cca19d4bd43239a9eca42
> http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commit;h=3e26a2224a828cd07d1d1df46cbea7e99cc87993
> http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commit;h=11caa3fa9dae79864726fcb763f100b4fd713884
> http://git.denx.de/?p=u-boot/u-boot-imx.git;a=commit;h=bf3303c98b23a29d99acc5e28865db604873b699
> 
> Then do you need me to repost this patch after the upper five patches in 
> imx tree merge into uboot master branch, or the 3 patches in this patchset 
> go through imx tree?
> https://patchwork.ozlabs.org/patch/498048/
> https://patchwork.ozlabs.org/patch/498049/
> https://patchwork.ozlabs.org/patch/498050/

Acked-by: Scott Wood <scottwood at freescale.com>

I already have patches 1 and 3 applied locally, and will probably send a pull 
request as soon as buildman finishes.  Let me know if you want me to send 
that, and then you can send patch 2 through imx, or if you want me to pull 
patches 1 and 3 out so all can go via imx.

-Scott



More information about the U-Boot mailing list