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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




>>-----Original Message-----
>>From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx]
>>Sent: Friday, December 17, 2010 7:01 AM
>>To: Gopinath, Thara
>>Cc: linux-omap@xxxxxxxxxxxxxxx; paul@xxxxxxxxx; Cousson, Benoit; Sripathy,
>>Vishwanath; Sawant, Anand; Menon, Nishanth
>>Subject: Re: [PATCHv5 00/10] OMAP: Adding Smartreflex and Voltage driver
>>support
>>
>>Hi Thara,
>>
>>Thara Gopinath <thara@xxxxxx> 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux