RE: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage driver support

2010-12-17 Thread Gopinath, Thara


-Original Message-
From: Kevin Hilman [mailto:khil...@deeprootsystems.com]
Sent: Friday, December 17, 2010 7:01 AM
To: Gopinath, Thara
Cc: linux-omap@vger.kernel.org; p...@pwsan.com; Cousson, Benoit; Sripathy,
Vishwanath; Sawant, Anand; Menon, Nishanth
Subject: Re: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage driver
support

Hi Thara,

Thara Gopinath th...@ti.com 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?

 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.
Hello Kevin,

Thanks for the review!


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.

You have forgotten to attach the patch! Can you please share it.


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.

I will CC linux-arm when I am posting the next version

Regards
Thaa
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage driver support

2010-12-16 Thread Kevin Hilman
Hi Thara,

Thara Gopinath th...@ti.com 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?

 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
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage driver support

2010-12-16 Thread Vishwanath Sripathy
Kevin,

 -Original Message-
 From: Kevin Hilman [mailto:khil...@deeprootsystems.com]
 Sent: Friday, December 17, 2010 7:01 AM
 To: Thara Gopinath
 Cc: linux-omap@vger.kernel.org; p...@pwsan.com; b-cous...@ti.com;
 vishwanath...@ti.com; saw...@ti.com; n...@ti.com
 Subject: Re: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage
 driver support

 Hi Thara,

 Thara Gopinath th...@ti.com 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
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html