From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vishwanath Sripathy Subject: RE: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage driver support Date: Fri, 17 Dec 2010 12:13:03 +0530 Message-ID: <9ff73171b066b99cc1ed88fded3e581b@mail.gmail.com> References: <1292004181-4804-1-git-send-email-thara@ti.com> <8739px9oe8.fsf@deeprootsystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: Received: from na3sys009aog102.obsmtp.com ([74.125.149.69]:33481 "EHLO na3sys009aog102.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752478Ab0LQGnG (ORCPT ); Fri, 17 Dec 2010 01:43:06 -0500 Received: by mail-fx0-f54.google.com with SMTP id 16so313514fxm.41 for ; Thu, 16 Dec 2010 22:43:05 -0800 (PST) In-Reply-To: <8739px9oe8.fsf@deeprootsystems.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Kevin Hilman , Thara Gopinath Cc: linux-omap@vger.kernel.org, paul@pwsan.com, Benoit Cousson , Anand Sawant , Nishanth Menon Kevin, > -----Original Message----- > From: Kevin Hilman [mailto:khilman@deeprootsystems.com] > Sent: Friday, December 17, 2010 7:01 AM > To: Thara Gopinath > Cc: linux-omap@vger.kernel.org; paul@pwsan.com; b-cousson@ti.com; > vishwanath.bs@ti.com; sawant@ti.com; nm@ti.com > Subject: Re: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage > driver support > > Hi Thara, > > Thara Gopinath writes: > > > This patch series introduces smartreflex and voltage driver support > > for OMAP3430 and OMAP3630. SmartReflex modules do adaptive > voltage > > control for real-time voltage adjustments. > > > [...] > > > This patch series has been tested on OMAP3430 SDP with > omap2plus_defconfig > > with the following menuconfig options enabled. > > Any testing on 3630? I tested this on ZOOM3 and it works fine. Vishwa > > > System type -> TI OMAP Implementations -> Smartreflex Support > > System type -> TI OMAP Implementations -> > > Class 3 mode of Smartreflex Implementation > > > > Major Changes in v5 > > > > - Rebased to k.org 2.6.37-rc3 > > - Rebased to Nishant Menon's latest opp patches > > - Voltage pmic info structure extended to include a > > vast set of PMIC dependent parameters. > > - Smartreflex software n-target values support > > removed from the kernel. Instead n-target > > values are exposed as debugfs entries which can > > be written into by the user if needed. > > - Introduced a new file arch/arm/mach-omap2/omap_twl.c > > for specifying OMAP and TWL related info for > > the voltage layer. > > - Remove default enabling of smartreflex autocompensation > > during boot on OMAP3430 ES3.1 chips. Instead > > an API is provided that can be called from > > board files in case autocompensation needs > > to be enabled during boot up itself. > > - Other review comments on v4 > > > > > > This series is looking good. I have a couple minor comments on specific > patches. > > Also, it needs (hopefully only) one more rebase/repost. > > First, a few things have changed at the PRCM API level that need to be > updated. I have a untested patch (below) that should fix this for you, > that you'll need to split and apply to the OMAP3 and OMAP4 voltage > driver patches. > > To test with those changes, you can rebase onto my pm-core branch, > which > contains not only the PRCM API changes, but also Nishanth's latest OPP > branch, and several other clock, hwmod, PM core changes queued for > 2.6.38 > > Second, please also Cc the linux-arm-kernel mailing list, so if/when we > merge this into linux-omap, it doesn't need a last minute repost. > > Thanks, > > Kevin