Hi Russell,

On Wed, May 18, 2011 at 10:21:40, Jean-Christophe PLAGNIOL-VILLARD wrote:

> On 17:41 Tue 17 May     , Ben Gardiner wrote:

> > diff --git a/arch/arm/mach-davinci/sram.c b/arch/arm/mach-davinci/sram.c
> > index 219d4c5..96026df 100644
> > --- a/arch/arm/mach-davinci/sram.c
> > +++ b/arch/arm/mach-davinci/sram.c
> > @@ -8,6 +8,7 @@
> >   * the Free Software Foundation; either version 2 of the License, or
> >   * (at your option) any later version.
> >   */
> > +#include <linux/io.h>
> >  #include <linux/module.h>
> >  #include <linux/init.h>
> >  #include <asm/pv-pool.h>
> > @@ -26,16 +27,23 @@ EXPORT_SYMBOL_GPL(davinci_pv_pool);
> >   */
> >  static int __init sram_init(void)
> >  {
> > +   void *addr;
> >     unsigned len = davinci_soc_info.sram_len;
> >     int status = 0;
> >  
> >     if (len) {
> >             len = min_t(unsigned, len, SRAM_SIZE);
> > -           davinci_pv_pool = pv_pool_create((void *)SRAM_VIRT,
> > +           addr = ioremap(davinci_soc_info.sram_phys, len);
> > +           if (!addr)
> > +                   return -EIO;
> > +
> > +           davinci_pv_pool = pv_pool_create(addr,
> >                                     davinci_soc_info.sram_phys, len,
> >                                     ilog2(SRAM_GRANULARITY));
> please use the gennalloc directly as discuss with Andrew and Russell my patch
> will present in the .40

With Jean-Christophe's patch getting merged, would you be dropping
your PV pool patch?

Thanks,
Sekhar

_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to