RE: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-17 Thread Ghorai, Sukumar
[PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC > virtual address > > On Mon, May 17, 2010 at 9:52 AM, Ghorai, Sukumar wrote: > [...] > >> > @@ -108,11 +108,27 @@ static u32 gpmc_read_reg(int idx) > >> >        return __raw_readl(gpmc_base + idx); >

Re: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-17 Thread Vimal Singh
On Mon, May 17, 2010 at 9:52 AM, Ghorai, Sukumar wrote: [...] >> > @@ -108,11 +108,27 @@ static u32 gpmc_read_reg(int idx) >> >        return __raw_readl(gpmc_base + idx); >> >  } >> > >> > +static void gpmc_cs_write_byte(int cs, int idx, u8 val) >> > +{ >> > +       void __iomem *reg_addr; >> > +

RE: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-16 Thread Ghorai, Sukumar
ct: Re: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC > virtual address > > Few more comments below. > > * Sukumar Ghorai [100514 08:18]: > > + > > +/** > > + * gmpc_ecc_init - Initialize the HW ECC for NAND flash in GPMC > controller > > + * @cs: Ch

RE: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-16 Thread Ghorai, Sukumar
ct: Re: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC > virtual address > > On Fri, May 14, 2010 at 8:53 PM, Sukumar Ghorai wrote: > [...] > > > diff --git a/arch/arm/mach-omap2/gpmc-onenand.c b/arch/arm/mach- > omap2/gpmc-onenand.c > > index 7bb6922..5

Re: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-14 Thread Tony Lindgren
Few more comments below. * Sukumar Ghorai [100514 08:18]: > + > +/** > + * gmpc_ecc_init - Initialize the HW ECC for NAND flash in GPMC controller > + * @cs: Chip select number > + * @ecc_size: bytes for which ECC will be generated > + */ > +void gpmc_ecc_init(int cs, int ecc_size) > +{ > + u

Re: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-14 Thread Vimal Singh
On Fri, May 14, 2010 at 8:53 PM, Sukumar Ghorai wrote: [...] > diff --git a/arch/arm/mach-omap2/gpmc-onenand.c > b/arch/arm/mach-omap2/gpmc-onenand.c > index 7bb6922..5d66817 > --- a/arch/arm/mach-omap2/gpmc-onenand.c > +++ b/arch/arm/mach-omap2/gpmc-onenand.c > @@ -301,7 +301,7 @@ static int om

Re: [PATCH v2 1/2] omap3 nand: cleanup for not to use GPMC virtual address

2010-05-14 Thread Tony Lindgren
* Sukumar Ghorai [100514 08:18]: > Necessary function added in GPMC module and used by nand driver. This is for > not to use GPMC address directly from nand driver. Also it was passing GPMC > base address from board files and that is removed. Great, this is starting to look good! I'll take a clos