| Message ID | 1478754560-27923-5-git-send-email-j-keerthy@ti.com |
|---|---|
| State | New |
| Headers | show |
* Keerthy <j-keerthy@ti.com> [161109 21:10]: > The PMICs have POWERHOLD set by default which prevents PMIC shutdown > even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. > So to enable pmic power off this property lets one over ride the default > value and enable pmic power off. This should not cause merge conflicts so probably best to merge along with the driver changes: Acked-by: Tony Lindgren <tony@atomide.com> If you guys want me to pick up this separately let me know. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tuesday 15 November 2016 05:38 AM, Tony Lindgren wrote: > * Keerthy <j-keerthy@ti.com> [161109 21:10]: >> The PMICs have POWERHOLD set by default which prevents PMIC shutdown >> even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. >> So to enable pmic power off this property lets one over ride the default >> value and enable pmic power off. > > This should not cause merge conflicts so probably best to merge along > with the driver changes: > > Acked-by: Tony Lindgren <tony@atomide.com> > > If you guys want me to pick up this separately let me know. Hi Lee Jones, Are you planning to pull DT and Documentation patches as well? Regards, Keerthy > > Regards, > > Tony > -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 23 November 2016 02:03 PM, Lee Jones wrote: > On Wed, 23 Nov 2016, Keerthy wrote: > >> >> >> On Tuesday 15 November 2016 05:38 AM, Tony Lindgren wrote: >>> * Keerthy <j-keerthy@ti.com> [161109 21:10]: >>>> The PMICs have POWERHOLD set by default which prevents PMIC shutdown >>>> even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. >>>> So to enable pmic power off this property lets one over ride the default >>>> value and enable pmic power off. >>> >>> This should not cause merge conflicts so probably best to merge along >>> with the driver changes: >>> >>> Acked-by: Tony Lindgren <tony@atomide.com> >>> >>> If you guys want me to pick up this separately let me know. >> >> Hi Lee Jones, >> >> Are you planning to pull DT and Documentation patches as well? > > No need. They can be safely applied to their own subsystems. Okay. Thanks for the response. Tony, Hope you can pull the DT patches. > -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, 23 Nov 2016, Keerthy wrote: > > > On Tuesday 15 November 2016 05:38 AM, Tony Lindgren wrote: > > * Keerthy <j-keerthy@ti.com> [161109 21:10]: > > > The PMICs have POWERHOLD set by default which prevents PMIC shutdown > > > even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. > > > So to enable pmic power off this property lets one over ride the default > > > value and enable pmic power off. > > > > This should not cause merge conflicts so probably best to merge along > > with the driver changes: > > > > Acked-by: Tony Lindgren <tony@atomide.com> > > > > If you guys want me to pick up this separately let me know. > > Hi Lee Jones, > > Are you planning to pull DT and Documentation patches as well? No need. They can be safely applied to their own subsystems.
* Keerthy <j-keerthy@ti.com> [161123 00:33]: > On Wednesday 23 November 2016 02:03 PM, Lee Jones wrote: > > On Wed, 23 Nov 2016, Keerthy wrote: > > > On Tuesday 15 November 2016 05:38 AM, Tony Lindgren wrote: > > > > * Keerthy <j-keerthy@ti.com> [161109 21:10]: > > > > > The PMICs have POWERHOLD set by default which prevents PMIC shutdown > > > > > even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. > > > > > So to enable pmic power off this property lets one over ride the default > > > > > value and enable pmic power off. > > > > > > > > This should not cause merge conflicts so probably best to merge along > > > > with the driver changes: > > > > > > > > Acked-by: Tony Lindgren <tony@atomide.com> > > > > > > > > If you guys want me to pick up this separately let me know. > > > > > > Hi Lee Jones, > > > > > > Are you planning to pull DT and Documentation patches as well? > > > > No need. They can be safely applied to their own subsystems. > > Okay. Thanks for the response. > > Tony, > > Hope you can pull the DT patches. Applying both into omap-for-v4.10/dt thanks. Please send dts changes seprately next time if there are no dependencies. This leaves out the second guessing who should apply what. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 23 November 2016 09:38 PM, Tony Lindgren wrote: > * Keerthy <j-keerthy@ti.com> [161123 00:33]: >> On Wednesday 23 November 2016 02:03 PM, Lee Jones wrote: >>> On Wed, 23 Nov 2016, Keerthy wrote: >>>> On Tuesday 15 November 2016 05:38 AM, Tony Lindgren wrote: >>>>> * Keerthy <j-keerthy@ti.com> [161109 21:10]: >>>>>> The PMICs have POWERHOLD set by default which prevents PMIC shutdown >>>>>> even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. >>>>>> So to enable pmic power off this property lets one over ride the default >>>>>> value and enable pmic power off. >>>>> >>>>> This should not cause merge conflicts so probably best to merge along >>>>> with the driver changes: >>>>> >>>>> Acked-by: Tony Lindgren <tony@atomide.com> >>>>> >>>>> If you guys want me to pick up this separately let me know. >>>> >>>> Hi Lee Jones, >>>> >>>> Are you planning to pull DT and Documentation patches as well? >>> >>> No need. They can be safely applied to their own subsystems. >> >> Okay. Thanks for the response. >> >> Tony, >> >> Hope you can pull the DT patches. > > Applying both into omap-for-v4.10/dt thanks. Please send dts changes > seprately next time if there are no dependencies. This leaves out > the second guessing who should apply what. Sure Tony. > > Regards, > > Tony > -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi b/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi index 6df7829..78bee26 100644 --- a/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi +++ b/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi @@ -204,6 +204,7 @@ interrupt-controller; ti,system-power-controller; + ti,palmas-override-powerhold; tps659038_pmic { compatible = "ti,tps659038-pmic";
The PMICs have POWERHOLD set by default which prevents PMIC shutdown even on DEV_CTRL On bit set to 0 as the Powerhold has higher priority. So to enable pmic power off this property lets one over ride the default value and enable pmic power off. Signed-off-by: Keerthy <j-keerthy@ti.com> --- arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi | 1 + 1 file changed, 1 insertion(+)