RE: [PATCHV3 1/2] OMAP3: Set MPU and IVA bypass Clock Divider

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

 



> -----Original Message-----
> From: Sripathy, Vishwanath 
> Sent: Tuesday, April 06, 2010 6:45 PM
> To: Premi, Sanjeev; Paul Walmsley
> Cc: linux-omap@xxxxxxxxxxxxxxx
> Subject: RE: [PATCHV3 1/2] OMAP3: Set MPU and IVA bypass Clock Divider
> 

[snip]--[snip] 

> > I have been trying to find a good place for the same myself. Though,
> > my reason is different. The default kernel boots with the OPP3 for
> > OMAP34xx; but when mpurate is used to set 720; I feel sometimes the
> > boot may fail if the voltage isn't right.
> I thought uboot would have set the right voltage while 
> setting the MPU rate. Isn't that true?

"mpurate" is argument passed to kernel. u-boot is not doing much with it.

> > 
> > The voltage does stabilize when smartrelex reflex is initialized.
> Why do you have dependency on SR for voltage stabilization? 
> You can set the right voltage even w/o SR (via VP). SR is 
> only for optimizing Voltage for a given OPP. 

In the default flow, clock_arch_init() where mpurate is acted upon,
I didn't want to start changing the VP. Else some part of the code
in SR - related to VP - will be duplicated. OR needs to be exported.

Still not made my mind; as I have been away from this problem for a
while now :(

~sanjeev

> 
> Regards
> Vishwa
> > 
> > I was trying to move smartreflex above in the init sequence; after
> > i2c has been initialized.
> > 
> > Comments/ thoughts?
> > 
> > Best regards,
> > Sanjeev
> > 

[snip]--[snip]
--
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