From mboxrd@z Thu Jan 1 00:00:00 1970 From: Fabio Estevam Subject: Re: [PATCH v2] pwm: imx: Allow keeping the PWM active during suspend Date: Mon, 20 Nov 2017 20:41:06 -0200 Message-ID: References: <1511015768-14140-1-git-send-email-festevam@gmail.com> <20171120211932.ro3ybgo6ih2542m2@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <20171120211932.ro3ybgo6ih2542m2@rob-hp-laptop> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring Cc: Thierry Reding , Russell King , linux-pwm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org On Mon, Nov 20, 2017 at 7:19 PM, Rob Herring 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. > > You problem to me sounds like something that should be fixed in the pwm > and/or pwm-led drivers. pwm-leds does set brightness to 0 in suspend and the fix here is to make pwm-imx active during suspend, so that when pwm-leds sets the brightness to 0, then pwm-imx really goes to logic level 1. Please note that on imx6qdl-cuboxi the pwm is active low. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html