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

From: Paul Walmsley
Date: Thu Apr 12 2012 - 13:00:12 EST


Hi

On Thu, 12 Apr 2012, Mohammed, Afzal wrote:

> On Thu, Apr 12, 2012 at 18:40:45, Greg KH wrote:
> > On Thu, Apr 12, 2012 at 12:17:49PM +0530, Santosh Shilimkar wrote:
> > > I was hoping that we will have some thing like drivers/memory/*
> > > but since it doesn't exist, we used drivers/misc.
> >
> > Why not create it? I have no objection to that, it makes it more
> > obvious as to what this really is.
>
> There is another memory controller used in a few TI SoCs,
> namely GPMC [1], do you prefer having it too there.
>
> As of now it is not a driver, platform code handles GPMC, a patch
> series for converting it into a driver (but still residing in
> platform folder) was sent a few days back [2,3].

Probably the GPMC driver should go into a slightly different place than
SDRC/EMIF.

GPMC is actually a general-purpose parallel bus driver. It's used to
interface Ethernet controllers, UARTs, FPGAs, NAND/NOR flash, SRAM, etc.
It cannot be used to control DRAM, at least not without a separate DRAM
controller chip.

SDRC/EMIF are both DRAM controllers. That's all they do. They can't be
used to control anything else. They implement DRAM refresh, etc.

So perhaps something like drivers/memory/dram/ for the SDRAM controllers,
and maybe drivers/memory/ for the GPMC?


- Paul
--
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/