RE: [PATCH 0/2] Remove unused voltagedomain data for AM33xx
-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
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
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
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
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