RE: [PATCH] OMAP4 PM: To correct voltages in MPU OPP Table

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

 




>>-----Original Message-----
>>From: Menon, Nishanth
>>Sent: Thursday, January 06, 2011 8:29 PM
>>To: Gopinath, Thara
>>Cc: Gulati, Shweta; l-o
>>Subject: Re: [PATCH] OMAP4 PM: To correct voltages in MPU OPP Table
>>
>>Gopinath, Thara had written, on 01/06/2011 08:52 AM, the following:
>>>
>>>>> -----Original Message-----
>>>>> From: Menon, Nishanth
>>>>> Sent: Thursday, January 06, 2011 5:52 PM
>>>>> To: Gulati, Shweta
>>>>> Cc: linux-omap@xxxxxxxxxxxxxxx; Gopinath, Thara
>>>>> Subject: Re: [PATCH] OMAP4 PM: To correct voltages in MPU OPP Table
>>>>>
>>>>> it is pretty unfortunate that I have to NAK this patch in the public
>>ML
>>>>> as well.
>>>>>
>>>>> shweta.gulati@xxxxxx wrote, on 01/06/2011 12:27 AM:
>>>>>> From: Shweta Gulati<shweta.gulati@xxxxxx>
>>>>>>
>>>>>> There is a mismatch in voltages specified in OPP table of MPU
>>>>>> and voltage specified in voltage table 'omap44xx_vdd_mpu_volt_data'
>>>>>> This Patch corrects MPU OPP Table as
>>>>>> well as enable OPP-Turbo and OPP-SB  for MPU by default.
>>>>>>
>>>>>> Signed-off-by: Thara Gopinath<thara@xxxxxx>
>>>>>> Signed-off-by: Shweta Gulati<shweta.gulati@xxxxxx>
>>>>>> ---
>>>>>> The patch is generated on top of Kevin's PM branch. It's needed for
>>SR
>>>>>> functionality on the current pm branch. Have tested SR with this
>>patch
>>>>>> with different OPP configurations from boot loader.
>>>>>>
>>>>>>   arch/arm/mach-omap2/opp4xxx_data.c |    8 ++++----
>>>>>>   1 files changed, 4 insertions(+), 4 deletions(-)
>>>>>>
>>>>>> diff --git a/arch/arm/mach-omap2/opp4xxx_data.c b/arch/arm/mach-
>>>>> omap2/opp4xxx_data.c
>>>>>> index a11fa56..4f35361 100644
>>>>>> --- a/arch/arm/mach-omap2/opp4xxx_data.c
>>>>>> +++ b/arch/arm/mach-omap2/opp4xxx_data.c
>>>>>> @@ -25,13 +25,13 @@
>>>>>>
>>>>>>   static struct omap_opp_def __initdata omap44xx_opp_def_list[] = {
>>>>>>   	/* MPU OPP1 - OPP50 */
>>>>>> -	OPP_INITIALIZER("mpu", true, 300000000, 1100000),
>>>>>> +	OPP_INITIALIZER("mpu", true, 300000000, 930000),
>>>>>>   	/* MPU OPP2 - OPP100 */
>>>>>> -	OPP_INITIALIZER("mpu", true, 600000000, 1200000),
>>>>>> +	OPP_INITIALIZER("mpu", true, 600000000, 1100000),
>>>>>
>>>>> Did we finalize on the nominal voltages yet?
>>>>>
>>>>> As of yesterday's discussion, we were still debating about the actual
>>>>> voltage at OMAP ball level, while there is a secondary voltage called
>>>>> cap voltage - we have been discussing on this for some time. I suggest
>>>>> strongly that we dont touch this for the time being (the voltage in
>>>>> mainline is slightly higher - let it be so till the h/w folks finalize
>>>>> things).
>>>
>>> Actually no. The latest voltage layer pushed uses these voltages. Also
>>Arrgh... another reason to avoid messy duplicate tables!!

Oh there is a patch in my bag where we use a single macro for each voltage
across the voltage and opp layer!! Not yet posted because I am waiting for
voltage layer to be merged.

>>> We have been having this setting in the internal android code base for
>>> some time now without anybody having issues. So till the new voltages
>>> are conveyed officially, these remain the official voltage.
>>Funny,
>>how many versions of "internal" code bases are present?
>>
>>http://dev.omapzoom.org/?p=santosh/kernel-omap4-
>>base.git;a=blob;f=arch/arm/mach-
>>omap2/opp44xx_data.c;h=252e3d0cb6050a64f390b9311c1c4977d74f762a

What are you complaining about here? I thought Shweta's patch
was making the mpu entries in the opp table similar to the
ones in the link above. Anything I am missing?

>>
>>
>>>
>>>>>>   	/* MPU OPP3 - OPP-Turbo */
>>>>>> -	OPP_INITIALIZER("mpu", false, 800000000, 1260000),
>>>>>> +	OPP_INITIALIZER("mpu", true, 800000000, 1260000),
>>>>> I disagree.  This is not $subject. Also - not all boards will be
>>capable
>>>>> of supporting all higher frequencies rt? - remember the 3630
>>experience?
>>>>> is'nt it wiser to enable it based on board capabilities - e.g. similar
>>>>> to the patch I did for beagle XM yesterday - we wont be able to enable
>>>>> higher frequencies on SDP3630 as we have not guarenteed with PDN
>>>>> analysis that it is ok.
>>> I am not sure about this for OMAP4. Have you come across a board
>>> where these OPPs cannot be supported? We have been enabling these
>>> OPPs internally now for quite some time across all OMAP4 boards.
>>*all* as in how many? SDP/Blaze, Panda and....??? How many boards are
>>available which is in production?

All as in SDP, Blaze and Panda, today by default we boot up at 1 Ghz.
I have not heard of anybody asking to lower the frequencies. If you are
talking about any customer board, I am not the person to comment.

Regards
Thara

>>
>>> But still if you guys are paranoid about boards breaking, I am ok
>>> With keeping these OPPs disabled by default. But then anybody running
>>> the mainline code with a 800Mhz or 1GHz x-loader, will see a couple
>>> of warns in the kernel code.
>>You do have the option of enabling the frequency for specific boards
>>like SDP/Blaze (e.g. if you have h/w team's signoff that these can do
>>high frequencies - and I think they do).
>>
>>--
>>Regards,
>>Nishanth Menon
--
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