RE: [PATCH 0/2] Remove unused voltagedomain data for AM33xx

2013-06-16 Thread Hiremath, Vaibhav

 -Original Message-
 From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-
 ow...@vger.kernel.org] On Behalf Of Kevin Hilman
 Sent: Friday, June 14, 2013 7:31 PM
 To: Paul Walmsley
 Cc: Nayak, Rajendra; Hiremath, Vaibhav; linux-omap@vger.kernel.org;
 linux-arm-ker...@lists.infradead.org
 Subject: Re: [PATCH 0/2] Remove unused voltagedomain data for AM33xx
 
 Paul Walmsley p...@pwsan.com writes:
 
  cc Kevin, Vaibhav
 
  On Thu, 13 Jun 2013, Rajendra Nayak wrote:
 
  The powerdomain framework today expects to always have a
 voltagedomain
  associated with a given powerdomain. We already have AM33xx which
  has no Voltage Controller/Voltage Processor as part of PRCM.
  There are more SoCs' to follow starting with AM437x and DRA7xx
  which do not have VC/VP.
 
  Instead of adding dummy voltage domain data files, make the
 powerdomain
  framework aware of the fact that some SoCs' might not really have
  scalable voltage domains.
 
  Fine with me in principle if AM335x doesn't support voltage scaling.
  Vaibhav, if this is okay for you, please ack it.
 
  Then, in terms of merging, probably Kevin would be the right person
 for
  this since he's done much of the voltagedomain work.
 
 Yeah, I'll take this series after the minor issues I commented on are
 fixed, and Vaivhav ack's the AM33xx parts.
 

Yeup, Feel free to add my Ack here.


Thanks,
Vaibhav 

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/2] Remove unused voltagedomain data for AM33xx

2013-06-16 Thread Rajendra Nayak
On Monday 17 June 2013 10:38 AM, Hiremath, Vaibhav wrote:
 
 -Original Message-
 From: linux-omap-ow...@vger.kernel.org [mailto:linux-omap-
 ow...@vger.kernel.org] On Behalf Of Kevin Hilman
 Sent: Friday, June 14, 2013 7:31 PM
 To: Paul Walmsley
 Cc: Nayak, Rajendra; Hiremath, Vaibhav; linux-omap@vger.kernel.org;
 linux-arm-ker...@lists.infradead.org
 Subject: Re: [PATCH 0/2] Remove unused voltagedomain data for AM33xx

 Paul Walmsley p...@pwsan.com writes:

 cc Kevin, Vaibhav

 On Thu, 13 Jun 2013, Rajendra Nayak wrote:

 The powerdomain framework today expects to always have a
 voltagedomain
 associated with a given powerdomain. We already have AM33xx which
 has no Voltage Controller/Voltage Processor as part of PRCM.
 There are more SoCs' to follow starting with AM437x and DRA7xx
 which do not have VC/VP.

 Instead of adding dummy voltage domain data files, make the
 powerdomain
 framework aware of the fact that some SoCs' might not really have
 scalable voltage domains.

 Fine with me in principle if AM335x doesn't support voltage scaling.
 Vaibhav, if this is okay for you, please ack it.

 Then, in terms of merging, probably Kevin would be the right person
 for
 this since he's done much of the voltagedomain work.

 Yeah, I'll take this series after the minor issues I commented on are
 fixed, and Vaivhav ack's the AM33xx parts.

 
 Yeup, Feel free to add my Ack here.

Thanks Vaibhav, I will add your and Nishanths' ack for this series
and repost a v2 addressing Kevins comments.

regards,
Rajendra
 
 
 Thanks,
 Vaibhav 
 

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/2] Remove unused voltagedomain data for AM33xx

2013-06-14 Thread Nishanth Menon
On 02:46-20130614, Paul Walmsley wrote:
 cc Kevin, Vaibhav
 
 On Thu, 13 Jun 2013, Rajendra Nayak wrote:
 
  The powerdomain framework today expects to always have a voltagedomain
  associated with a given powerdomain. We already have AM33xx which
  has no Voltage Controller/Voltage Processor as part of PRCM.
  There are more SoCs' to follow starting with AM437x and DRA7xx
  which do not have VC/VP.
  
  Instead of adding dummy voltage domain data files, make the powerdomain
  framework aware of the fact that some SoCs' might not really have
  scalable voltage domains.
 
 Fine with me in principle if AM335x doesn't support voltage scaling.  
 Vaibhav, if this is okay for you, please ack it.  
just a nitpick :)
There is no VFSM auto scaling to retention voltage, nor is VC-VP IP
available here, but that does not mean DVFS voltage scaling does not
exist - it is done with traditional regulator framework using regular
shared I2C.

We now seem to be moving to a generation of SoCs where VC-VP has been
dumped in favor of allowing integration with PMIC which may talk SPI/I2C
over standard interfaces - this allows us to reach better product
options, but gives up on extreme low power scenarios we had seen in the
past on OMAP.
Overall, I love this idea.
Acked-by: Nishanth Menon n...@ti.com
 
 Then, in terms of merging, probably Kevin would be the right person for 
 this since he's done much of the voltagedomain work.
-- 
Regards,
Nishanth Menon
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/2] Remove unused voltagedomain data for AM33xx

2013-06-14 Thread Kevin Hilman
Paul Walmsley p...@pwsan.com writes:

 cc Kevin, Vaibhav

 On Thu, 13 Jun 2013, Rajendra Nayak wrote:

 The powerdomain framework today expects to always have a voltagedomain
 associated with a given powerdomain. We already have AM33xx which
 has no Voltage Controller/Voltage Processor as part of PRCM.
 There are more SoCs' to follow starting with AM437x and DRA7xx
 which do not have VC/VP.
 
 Instead of adding dummy voltage domain data files, make the powerdomain
 framework aware of the fact that some SoCs' might not really have
 scalable voltage domains.

 Fine with me in principle if AM335x doesn't support voltage scaling.  
 Vaibhav, if this is okay for you, please ack it.  

 Then, in terms of merging, probably Kevin would be the right person for 
 this since he's done much of the voltagedomain work.

Yeah, I'll take this series after the minor issues I commented on are
fixed, and Vaivhav ack's the AM33xx parts.

Kevin
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/2] Remove unused voltagedomain data for AM33xx

2013-06-13 Thread Paul Walmsley
cc Kevin, Vaibhav

On Thu, 13 Jun 2013, Rajendra Nayak wrote:

 The powerdomain framework today expects to always have a voltagedomain
 associated with a given powerdomain. We already have AM33xx which
 has no Voltage Controller/Voltage Processor as part of PRCM.
 There are more SoCs' to follow starting with AM437x and DRA7xx
 which do not have VC/VP.
 
 Instead of adding dummy voltage domain data files, make the powerdomain
 framework aware of the fact that some SoCs' might not really have
 scalable voltage domains.

Fine with me in principle if AM335x doesn't support voltage scaling.  
Vaibhav, if this is okay for you, please ack it.  

Then, in terms of merging, probably Kevin would be the right person for 
this since he's done much of the voltagedomain work.


- Paul
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html