From: Kevin Hilman <khilman@ti.com>
To: "Mohammed\, Afzal" <afzal@ti.com>
Cc: Greg KH <greg@kroah.com>, "Shilimkar\,
Santosh" <santosh.shilimkar@ti.com>,
Paul Walmsley <paul@pwsan.com>, "V\, Aneesh" <aneesh@ti.com>,
"linux-kernel\@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel\@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-omap\@vger.kernel.org" <linux-omap@vger.kernel.org>,
"mgreer\@animalcreek.com" <mgreer@animalcreek.com>,
"msalter\@redhat.com" <msalter@redhat.com>
Subject: Re: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver
Date: Thu, 12 Apr 2012 07:21:18 -0700 [thread overview]
Message-ID: <871untjajl.fsf@ti.com> (raw)
In-Reply-To: <C8443D0743D26F4388EA172BF4E2A7A93E95947E@DQHE04.ent.ti.com> (Afzal Mohammed's message of "Thu, 12 Apr 2012 13:34:15 +0000")
"Mohammed, Afzal" <afzal@ti.com> writes:
> Hi Greg,
>
> 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].
IMO, wherever EMIF ends up, GPMC should as well.
Kevin
> [1]
> GPMC (General Purpose Memory Controller) in brief:
> GPMC is an unified memory controller dedicated to interfacing external
> memory devices like
> Asynchronous SRAM like memories and application specific integrated circuit devices.
> Asynchronous, synchronous, and page mode burst NOR flash devices NAND flash
> Pseudo-SRAM devices
>
> GPMC has to be configured as required by timings of the connected
> peripheral. It needs to be configured only initially. Once it is
> configured it can be used to handle different protocols like NAND,
> NOR. Various kinds of devices like ethernet, uart, usb, fpga etc
> can work using GPMC interface. GPMC has a seperate additional
> functionality of NAND handling
>
> [2] https://lkml.org/lkml/2012/4/5/210
> [3] https://lkml.org/lkml/2012/4/5/212
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@ti.com>
To: "Mohammed, Afzal" <afzal@ti.com>
Cc: Greg KH <greg@kroah.com>,
"Shilimkar, Santosh" <santosh.shilimkar@ti.com>,
Paul Walmsley <paul@pwsan.com>, "V, Aneesh" <aneesh@ti.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"mgreer@animalcreek.com" <mgreer@animalcreek.com>,
"msalter@redhat.com" <msalter@redhat.com>
Subject: Re: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver
Date: Thu, 12 Apr 2012 07:21:18 -0700 [thread overview]
Message-ID: <871untjajl.fsf@ti.com> (raw)
In-Reply-To: <C8443D0743D26F4388EA172BF4E2A7A93E95947E@DQHE04.ent.ti.com> (Afzal Mohammed's message of "Thu, 12 Apr 2012 13:34:15 +0000")
"Mohammed, Afzal" <afzal@ti.com> writes:
> Hi Greg,
>
> 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].
IMO, wherever EMIF ends up, GPMC should as well.
Kevin
> [1]
> GPMC (General Purpose Memory Controller) in brief:
> GPMC is an unified memory controller dedicated to interfacing external
> memory devices like
> Asynchronous SRAM like memories and application specific integrated circuit devices.
> Asynchronous, synchronous, and page mode burst NOR flash devices NAND flash
> Pseudo-SRAM devices
>
> GPMC has to be configured as required by timings of the connected
> peripheral. It needs to be configured only initially. Once it is
> configured it can be used to handle different protocols like NAND,
> NOR. Various kinds of devices like ethernet, uart, usb, fpga etc
> can work using GPMC interface. GPMC has a seperate additional
> functionality of NAND handling
>
> [2] https://lkml.org/lkml/2012/4/5/210
> [3] https://lkml.org/lkml/2012/4/5/212
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: khilman@ti.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver
Date: Thu, 12 Apr 2012 07:21:18 -0700 [thread overview]
Message-ID: <871untjajl.fsf@ti.com> (raw)
In-Reply-To: <C8443D0743D26F4388EA172BF4E2A7A93E95947E@DQHE04.ent.ti.com> (Afzal Mohammed's message of "Thu, 12 Apr 2012 13:34:15 +0000")
"Mohammed, Afzal" <afzal@ti.com> writes:
> Hi Greg,
>
> 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].
IMO, wherever EMIF ends up, GPMC should as well.
Kevin
> [1]
> GPMC (General Purpose Memory Controller) in brief:
> GPMC is an unified memory controller dedicated to interfacing external
> memory devices like
> Asynchronous SRAM like memories and application specific integrated circuit devices.
> Asynchronous, synchronous, and page mode burst NOR flash devices NAND flash
> Pseudo-SRAM devices
>
> GPMC has to be configured as required by timings of the connected
> peripheral. It needs to be configured only initially. Once it is
> configured it can be used to handle different protocols like NAND,
> NOR. Various kinds of devices like ethernet, uart, usb, fpga etc
> can work using GPMC interface. GPMC has a seperate additional
> functionality of NAND handling
>
> [2] https://lkml.org/lkml/2012/4/5/210
> [3] https://lkml.org/lkml/2012/4/5/212
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2012-04-12 14:21 UTC|newest]
Thread overview: 84+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-16 21:51 [PATCH v4 0/7] Add TI EMIF SDRAM controller driver Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 1/7] misc: ddr: add LPDDR2 data from JESD209-2 Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 2/7] misc: emif: add register definitions for EMIF Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 3/7] misc: emif: add basic infrastructure for EMIF driver Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 4/7] misc: emif: handle frequency and voltage change events Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 5/7] misc: emif: add interrupt and temperature handling Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 6/7] misc: emif: add one-time settings Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-16 21:51 ` [PATCH v4 7/7] misc: emif: add debugfs entries for emif Aneesh V
2012-03-16 21:51 ` Aneesh V
2012-03-17 6:10 ` [PATCH v4 0/7] Add TI EMIF SDRAM controller driver Kyungmin Park
2012-03-17 6:10 ` Kyungmin Park
2012-03-17 6:10 ` Kyungmin Park
2012-04-12 2:44 ` Paul Walmsley
2012-04-12 2:44 ` Paul Walmsley
2012-04-12 2:57 ` Aneesh V
2012-04-12 2:57 ` Aneesh V
2012-04-12 2:57 ` Aneesh V
2012-04-12 3:00 ` Greg KH
2012-04-12 3:00 ` Greg KH
2012-04-12 3:12 ` Paul Walmsley
2012-04-12 3:12 ` Paul Walmsley
2012-04-12 6:47 ` Santosh Shilimkar
2012-04-12 6:47 ` Santosh Shilimkar
2012-04-12 13:10 ` Greg KH
2012-04-12 13:10 ` Greg KH
2012-04-12 13:34 ` Mohammed, Afzal
2012-04-12 13:34 ` Mohammed, Afzal
2012-04-12 13:34 ` Mohammed, Afzal
2012-04-12 14:10 ` Greg KH
2012-04-12 14:10 ` Greg KH
2012-04-12 14:10 ` Greg KH
2012-04-12 14:15 ` Mohammed, Afzal
2012-04-12 14:15 ` Mohammed, Afzal
2012-04-12 14:15 ` Mohammed, Afzal
2012-04-12 14:21 ` Kevin Hilman [this message]
2012-04-12 14:21 ` Kevin Hilman
2012-04-12 14:21 ` Kevin Hilman
2012-04-12 17:00 ` Paul Walmsley
2012-04-12 17:00 ` Paul Walmsley
2012-04-12 17:00 ` Paul Walmsley
2012-04-12 18:50 ` Cousson, Benoit
2012-04-12 18:50 ` Cousson, Benoit
2012-04-12 18:50 ` Cousson, Benoit
2012-04-12 19:15 ` Paul Walmsley
2012-04-12 19:15 ` Paul Walmsley
2012-04-12 19:15 ` Paul Walmsley
2012-04-12 19:24 ` Cousson, Benoit
2012-04-12 19:24 ` Cousson, Benoit
2012-04-12 19:24 ` Cousson, Benoit
2012-04-13 9:07 ` Felipe Balbi
2012-04-13 9:07 ` Felipe Balbi
2012-04-13 9:07 ` Felipe Balbi
2012-04-12 14:14 ` Shilimkar, Santosh
2012-04-12 14:14 ` Shilimkar, Santosh
2012-04-12 19:58 ` V, Aneesh
2012-04-12 19:58 ` V, Aneesh
2012-04-23 10:56 ` Mohammed, Afzal
2012-04-23 10:56 ` Mohammed, Afzal
2012-04-23 10:56 ` Mohammed, Afzal
2012-04-23 11:04 ` Shilimkar, Santosh
2012-04-23 11:04 ` Shilimkar, Santosh
2012-04-23 11:04 ` Shilimkar, Santosh
2012-04-23 11:09 ` Mohammed, Afzal
2012-04-23 11:09 ` Mohammed, Afzal
2012-04-23 11:09 ` Mohammed, Afzal
2012-04-23 14:27 ` Greg KH
2012-04-23 14:27 ` Greg KH
2012-04-23 14:27 ` Greg KH
2012-04-23 14:44 ` Shilimkar, Santosh
2012-04-23 14:44 ` Shilimkar, Santosh
2012-04-23 14:44 ` Shilimkar, Santosh
2012-04-27 13:16 ` Santosh Shilimkar
2012-04-27 13:16 ` Santosh Shilimkar
2012-04-27 13:16 ` Santosh Shilimkar
2012-04-13 14:15 ` Mark Salter
2012-04-13 14:15 ` Mark Salter
2012-03-19 1:08 함명주
2012-03-19 7:07 ` Shilimkar, Santosh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=871untjajl.fsf@ti.com \
--to=khilman@ti.com \
--cc=afzal@ti.com \
--cc=aneesh@ti.com \
--cc=greg@kroah.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=mgreer@animalcreek.com \
--cc=msalter@redhat.com \
--cc=paul@pwsan.com \
--cc=santosh.shilimkar@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.