From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965057Ab2DLOVb (ORCPT ); Thu, 12 Apr 2012 10:21:31 -0400 Received: from na3sys009aog105.obsmtp.com ([74.125.149.75]:44659 "EHLO na3sys009aog105.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934322Ab2DLOV3 (ORCPT ); Thu, 12 Apr 2012 10:21:29 -0400 From: Kevin Hilman To: "Mohammed\, Afzal" Cc: Greg KH , "Shilimkar\, Santosh" , Paul Walmsley , "V\, Aneesh" , "linux-kernel\@vger.kernel.org" , "linux-arm-kernel\@lists.infradead.org" , "linux-omap\@vger.kernel.org" , "mgreer\@animalcreek.com" , "msalter\@redhat.com" Subject: Re: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver Organization: Texas Instruments, Inc. References: <1331934707-29487-1-git-send-email-aneesh@ti.com> <20120412030044.GA26236@kroah.com> <4F867A95.9030104@ti.com> <20120412131045.GB31384@kroah.com> Date: Thu, 12 Apr 2012 07:21:18 -0700 In-Reply-To: (Afzal Mohammed's message of "Thu, 12 Apr 2012 13:34:15 +0000") Message-ID: <871untjajl.fsf@ti.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org "Mohammed, Afzal" 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver Date: Thu, 12 Apr 2012 07:21:18 -0700 Message-ID: <871untjajl.fsf@ti.com> References: <1331934707-29487-1-git-send-email-aneesh@ti.com> <20120412030044.GA26236@kroah.com> <4F867A95.9030104@ti.com> <20120412131045.GB31384@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from na3sys009aog137.obsmtp.com ([74.125.149.18]:57496 "EHLO psmtp.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S934184Ab2DLOV2 (ORCPT ); Thu, 12 Apr 2012 10:21:28 -0400 Received: by pbbrp16 with SMTP id rp16so460476pbb.3 for ; Thu, 12 Apr 2012 07:21:22 -0700 (PDT) In-Reply-To: (Afzal Mohammed's message of "Thu, 12 Apr 2012 13:34:15 +0000") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Mohammed, Afzal" Cc: Greg KH , "Shilimkar, Santosh" , Paul Walmsley , "V, Aneesh" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-omap@vger.kernel.org" , "mgreer@animalcreek.com" , "msalter@redhat.com" "Mohammed, Afzal" 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Thu, 12 Apr 2012 07:21:18 -0700 Subject: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver In-Reply-To: (Afzal Mohammed's message of "Thu, 12 Apr 2012 13:34:15 +0000") References: <1331934707-29487-1-git-send-email-aneesh@ti.com> <20120412030044.GA26236@kroah.com> <4F867A95.9030104@ti.com> <20120412131045.GB31384@kroah.com> Message-ID: <871untjajl.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org "Mohammed, Afzal" 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