Messages in this thread |  | | Date | Thu, 12 Apr 2012 12:58:55 -0700 | | Subject | Re: [PATCH v4 0/7] Add TI EMIF SDRAM controller driver | | From | "V, Aneesh" <> |
| |
Hi Greg,
On Wed, Apr 11, 2012 at 8:00 PM, Greg KH <greg@kroah.com> wrote: > On Wed, Apr 11, 2012 at 08:44:39PM -0600, Paul Walmsley wrote: >> Cc Mark Greer, Mark Salter >> >> Hi Greg, Aneesh, >> >> On Sat, 17 Mar 2012, Aneesh V wrote: >> >> > Add a driver for the EMIF SDRAM controller used in TI SoCs >> > >> > EMIF is an SDRAM controller that supports, based on its revision, >> > one or more of LPDDR2/DDR2/DDR3 protocols.This driver adds support >> > for LPDDR2. >> >> Just checking to see what the current state of this series is. Greg, are >> you considering this for merging, or are there remaining issues? Aneesh, >> do you have any remaining issues to resolve with this set? > > What about the review comment about devfreq?
I see that Santosh has already commented on this. My views are similar, that frequency update is only one of the many functions of the driver.
> >> This is useful not only for OMAP4 and AM3517/3505, but also will probably >> be useful for the C6x chips that Mark Salter is working on. > > It's still in my "to-review" queue, that I'm slowly making my way > through. So it's not lost, but I would like to get the devfreq > interface question cleared up first.
Thanks. I will wait for your review then. Once I have your comments I will re-work and submit in the new directory structure proposed.
thanks, Aneesh -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |