[PATCH 3/3 v3] P4080/mtd: Fix the freescale lbc issue with 36bit mode

Zang Roy-R61911 r61911 at freescale.com
Thu Sep 16 17:36:38 EST 2010



> -----Original Message-----
> From: Anton Vorontsov [mailto:cbouatmailru at gmail.com]
> Sent: Thursday, September 16, 2010 15:32 PM
> To: Zang Roy-R61911
> Cc: linux-mtd at lists.infradead.org; dwmw2 at infradead.org; dedekind1 at gmail.com;
> akpm at linux-foundation.org; Lan Chunhe-B25806; Wood Scott-B07421; Gala Kumar-
> B11780; linuxppc-dev at ozlabs.org
> Subject: Re: [PATCH 3/3 v3] P4080/mtd: Fix the freescale lbc issue with 36bit
> mode
> 
> On Thu, Sep 16, 2010 at 02:41:24PM +0800, Roy Zang wrote:
> > From: Lan Chunhe-B25806 <b25806 at freescale.com>
> >
> > When system uses 36bit physical address, res.start is 36bit
> > physical address. But the function of in_be32 returns 32bit
> > physical address. Then both of them compared each other is
> > wrong. So by converting the address of res.start into
> > the right format fixes this issue.
> >
> > Signed-off-by: Lan Chunhe-B25806 <b25806 at freescale.com>
> > Signed-off-by: Roy Zang <tie-fei.zang at freescale.com>
> > ---
> >  arch/powerpc/include/asm/fsl_lbc.h |    1 +
> >  arch/powerpc/sysdev/fsl_lbc.c      |   23 ++++++++++++++++++++++-
> >  drivers/mtd/nand/fsl_elbc_nand.c   |    2 +-
> >  3 files changed, 24 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/powerpc/include/asm/fsl_lbc.h
> b/arch/powerpc/include/asm/fsl_lbc.h
> > index db94698..5638b1e 100644
> > --- a/arch/powerpc/include/asm/fsl_lbc.h
> > +++ b/arch/powerpc/include/asm/fsl_lbc.h
> > @@ -246,6 +246,7 @@ struct fsl_upm {
> >  	int width;
> >  };
> >
> > +extern unsigned int fsl_lbc_addr(phys_addr_t addr_base);
> 
> u32 here.
> 
> Other than that, the patch looks good.
> 
> Reviewed-by: Anton Vorontsov <cbouatmailru at gmail.com>
I will correct this together with previous patches.
Do you have any more comments for the previous two patches?
Thanks.
Roy


More information about the Linuxppc-dev mailing list