RE: [PATCH v4 2/9] OMAP3: PM: Adding smartreflex driver support.

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

 




>>-----Original Message-----
>>From: Varadarajan, Charulatha
>>Sent: Friday, October 29, 2010 10:23 AM
>>To: Gopinath, Thara; linux-omap@xxxxxxxxxxxxxxx
>>Cc: paul@xxxxxxxxx; khilman@xxxxxxxxxxxxxxxxxxx; Cousson, Benoit; Sripathy,
>>Vishwanath; Sawant, Anand
>>Subject: RE: [PATCH v4 2/9] OMAP3: PM: Adding smartreflex driver support.
>>
>>
>>> >>> -----Original Message-----
>>> >>> From: linux-omap-owner@xxxxxxxxxxxxxxx
>>> >>> [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of Thara Gopinath
>>> >>> Sent: Wednesday, October 27, 2010 9:41 PM
>>> >>> To: linux-omap@xxxxxxxxxxxxxxx
>>> >>> Cc: paul@xxxxxxxxx; khilman@xxxxxxxxxxxxxxxxxxx; Cousson,
>>> >>> Benoit; Sripathy, Vishwanath; Sawant, Anand; Gopinath, Thara
>>> >>> Subject: [PATCH v4 2/9] OMAP3: PM: Adding smartreflex driver support.
>>> >>>
>>> >>> SmartReflex modules do adaptive voltage control for real-time
>>> >>> voltage adjustments. With Smartreflex the power supply voltage
>>> >>> can be adapted to the silicon performance(manufacturing process,
>>> >>> temperature induced performance, age induced performance etc).
>>> >>>
>>> >>> There are differnet classes of smartreflex implementation.
>>> >>> 	Class-0: Manufacturing Test Calibration
>>> >>> 	Class-1: Boot-Time Software Calibration
>>> >>> 	Class-2: Continuous Software Calibration
>>> >>> 	Class-3: Continuous Hardware Calibration
>>> >>> 	Class-4: Fully Integrated Power Management
>>> >>>
>>> >>> OMAP3 has two smartreflex modules one associated with VDD MPU and the
>>> >>> other associated with VDD CORE.
>>> >>> This patch adds support for  smartreflex driver. The driver
>>> >>> is designed
>>> >>> for Class-1 , Class-2 and Class-3 support and is  a platform driver.
>>> >>> Smartreflex driver can be enabled through a Kconfig option
>>> >>> "SmartReflex support" under "System type"->"TI OMAP
>>> >>> implementations" menu.
>>> >>>
>>> >>> Smartreflex autocompensation feature can be enabled runtime through
>>> >>> a debug fs option.
>>> >>> To enable smartreflex autocompensation feature
>>> >>> 	echo 1 > /debug/voltage/vdd_<X>/smartreflex/autocomp
>>> >>> To disable smartreflex autocompensation feature
>>> >>> 	echo 0 > /debug/voltage/vdd_<X>/smartreflex/autocomp
>>> >>>
>>> >>> where X can be mpu, core , iva etc.
>>> >>>
>>> >>> This patch contains code originally in linux omap pm branch.
>>> >>> Major contributors to this driver are
>>> >>> Lesly A M, Rajendra Nayak, Kalle Jokiniemi, Paul Walmsley,
>>> >>> Nishant Menon, Kevin Hilman.
>>> >>>
>>> >>> Signed-off-by: Thara Gopinath <thara@xxxxxx>
>>> >>> ---
>>> >>>  arch/arm/mach-omap2/Makefile                  |    1 +
>>> >>>  arch/arm/mach-omap2/smartreflex.c             |  975
>>> >>> +++++++++++++++++++++++++
>>> >>>  arch/arm/plat-omap/Kconfig                    |   36 +
>>> >>>  arch/arm/plat-omap/include/plat/smartreflex.h |  271 +++++++
>>> >>>  4 files changed, 1283 insertions(+), 0 deletions(-)
>>> >>>  create mode 100644 arch/arm/mach-omap2/smartreflex.c
>>> >>>  create mode 100644 arch/arm/plat-omap/include/plat/smartreflex.h
>>> >>>
>>> >>
>>> >><<snip>>
>>> >>
>>> >>> +static int __init omap_sr_probe(struct platform_device *pdev)
>>> >>> +{
>>> >>> +	struct omap_sr *sr_info = kzalloc(sizeof(struct
>>> >>> omap_sr), GFP_KERNEL);
>>> >>> +	struct omap_device *odev = to_omap_device(pdev);
>>> >>
>>> >>Patch3 should be ordered before patch2 in your series. Otherwise,
>>> >>this would fail during a git-bisect.
>>>
>>> Again why ?? All patches individually compile and boot.
>>
>>omap_device_build() for this device is being done in the next patch.
>>Hence I gave this input. But I believe that this does not harm because
>>the probe would not get called. Correct me if I am wrong.

Exactly probe will not get called. So I do not think the patch ordering
needs to change. 

Regards
Thara
--
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