Re: [PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-07-26 Thread Felipe Balbi
On Tue, Mar 08, 2011 at 07:22:32AM +0530, Nishanth Menon wrote: On Mon, Mar 07, 2011 at 07:23:15PM +0300, Sergei Shtylyov wrote: Hello. On 07-03-2011 5:52, Nishanth Menon wrote: Blindly setting a 1.2V setting Hm, seems tautological... As a non native English speaker, I would

Re: [PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-03-17 Thread Kevin Hilman
Nishanth Menon n...@ti.com writes: Blindly setting a 1.2V setting in the initial structure may not even match the default voltages stored in the voltage table which are supported for the domain. For example, OMAP3430 core domain does not use 1.2V and ends up generating a warning on the first

Re: [PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-03-07 Thread Sergei Shtylyov
Hello. On 07-03-2011 5:52, Nishanth Menon wrote: Blindly setting a 1.2V setting Hm, seems tautological... As a non native English speaker, I would like to understand how we could improve this. I suppose you mean Blindly setting 1.2V is better here. Am i correct? Yes. WBR, Sergei --

Re: [PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-03-07 Thread Nishanth Menon
On Mon, Mar 07, 2011 at 07:23:15PM +0300, Sergei Shtylyov wrote: Hello. On 07-03-2011 5:52, Nishanth Menon wrote: Blindly setting a 1.2V setting Hm, seems tautological... As a non native English speaker, I would like to understand how we could improve this. I suppose you mean Blindly

Re: [PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-03-06 Thread Sergei Shtylyov
Hello. On 05-03-2011 18:29, Nishanth Menon wrote: Blindly setting a 1.2V setting Hm, seems tautological... in the initial structure may not even match the default voltages stored in the voltage table which are supported for the domain. For example, OMAP3430 core domain does not use 1.2V

Re: [PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-03-06 Thread Nishanth Menon
Sergei Shtylyov wrote, on 03/06/2011 07:07 PM: Blindly setting a 1.2V setting Hm, seems tautological... As a non native English speaker, I would like to understand how we could improve this. I suppose you mean Blindly setting 1.2V is better here. Am i correct? in the initial

[PATCH V3 03/19] OMAP3+: voltage: remove initial voltage

2011-03-05 Thread Nishanth Menon
Blindly setting a 1.2V setting in the initial structure may not even match the default voltages stored in the voltage table which are supported for the domain. For example, OMAP3430 core domain does not use 1.2V and ends up generating a warning on the first transition. Further, since