Re: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver

[Cousson, Benoit]

On 4/12/2012 9:15 PM, Paul Walmsley wrote:
> Hi Benoît,
>
> On Thu, 12 Apr 2012, Cousson, Benoit wrote:
>
> > The LPDDR2 spec does consider as well NVM (Non Volatile Memory), so I think we
> > should stick to driver/memory for EMIF.
>
> Hmm good point!
>
> > > So perhaps something like drivers/memory/dram/ for the SDRAM controllers,
> > > and maybe drivers/memory/ for the GPMC?
> >
> > In fact Felipe was considering something else for that kind of general purpose
> > bus driver like GMPC, C2C and LLI...
> >
> > ... But I do not remember the name :-)
>
> It would be nice if we could separate IP blocks that can drive an Ethernet
> controller or modem chip from the DRAM controllers; they are quite
> different beasts from a common API standpoint.

Yep, fully agree. In fact Felipe's suggestion was something like 
drivers/ocd for off-chip devices, but maybe something like drivers/gpbus 
will highlight a little bit more the bus controller aspect of such driver.

Regards,
Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/