Hi Rob, On Mon, Nov 20, 2017 at 7:19 PM, Rob Herring <robh@xxxxxxxxxx> wrote: > Seems to me the default behavior should be LEDs are off in suspend and > you'd add a property to enable them if desired. Are you suggesting that I set the MX3_PWMCR_STOPEN bit unconditionally? I see some old version of vendor driver that does exactly that: http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/tree/arch/arm/plat-mxc/pwm.c?h=imx_2.6.35_11.09.01 I can do that and then we do not need to add an extra device tree property. Thanks -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html