On 26/03/2019 17:14, Tony Lindgren wrote: > If CONFIG_OMAP4_DSS_HDMI_CEC is enabled in .config, deeper SoC idle > states are blocked because the CEC clock gets always enabled on init. > > Let's fix the issue by moving the CEC clock handling to happen later in > hdmi_cec_adap_enable() as suggested by Hans Verkuil <hverk...@xs4all.nl>. > This way the CEC clock gets only enabled when needed. This can be tested > by doing cec-ctl --playback to enable the CEC, and doing cec-ctl --clear > to disable it. > > Let's also fix the typo for "divider" in the comments while at it. > > Fixes: 8d7f934df8d8 ("omapdrm: hdmi4_cec: add OMAP4 HDMI CEC support") > Suggested-by: Hans Verkuil <hverk...@xs4all.nl> > Cc: Hans Verkuil <hverk...@xs4all.nl> > Cc: Jyri Sarha <jsa...@ti.com> > Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > Signed-off-by: Tony Lindgren <t...@atomide.com> > ---
Thanks! I'll pick this up. Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel