Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-11 Thread Tony Lindgren
* Tomi Valkeinen [191011 10:25]: > On 10/10/2019 16:24, Tony Lindgren wrote: > > > Hmm so what register does this clock actually change? > > > > I'm seeing an increase of few tens of extra mW, which means at > > least one day of standby time less for me :) It does not happen > > always, maybe

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-11 Thread Tomi Valkeinen
On 10/10/2019 16:24, Tony Lindgren wrote: Hmm so what register does this clock actually change? I'm seeing an increase of few tens of extra mW, which means at least one day of standby time less for me :) It does not happen always, maybe half of the time. I have no idea why this would affect

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-11 Thread Tony Lindgren
* Tomi Valkeinen [191010 06:48]: > On 08/10/2019 17:21, Tony Lindgren wrote: > > * Tomi Valkeinen [191008 14:17]: > > > On 08/10/2019 17:13, Tony Lindgren wrote: > > > > * Tomi Valkeinen [190930 10:38]: > > > > > If use_mclk is false, mclk_mode is written to a register without > > > > >

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-10 Thread Tomi Valkeinen
On 08/10/2019 17:21, Tony Lindgren wrote: * Tomi Valkeinen [191008 14:17]: On 08/10/2019 17:13, Tony Lindgren wrote: * Tomi Valkeinen [190930 10:38]: If use_mclk is false, mclk_mode is written to a register without initialization. This doesn't cause any ill effects as the written value is

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-09 Thread Tony Lindgren
* Tomi Valkeinen [191008 14:17]: > On 08/10/2019 17:13, Tony Lindgren wrote: > > * Tomi Valkeinen [190930 10:38]: > > > If use_mclk is false, mclk_mode is written to a register without > > > initialization. This doesn't cause any ill effects as the written value > > > is not used when use_mclk

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-09 Thread Tony Lindgren
* Tomi Valkeinen [190930 10:38]: > If use_mclk is false, mclk_mode is written to a register without > initialization. This doesn't cause any ill effects as the written value > is not used when use_mclk is false. > > To fix this, write use_mclk only when use_mclk is true. Hey nice catch. Based

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-08 Thread Tomi Valkeinen
On 08/10/2019 17:13, Tony Lindgren wrote: * Tomi Valkeinen [190930 10:38]: If use_mclk is false, mclk_mode is written to a register without initialization. This doesn't cause any ill effects as the written value is not used when use_mclk is false. To fix this, write use_mclk only when

Re: [PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-10-04 Thread Laurent Pinchart
Hi Tomi, Thank you for the patch. On Mon, Sep 30, 2019 at 01:38:40PM +0300, Tomi Valkeinen wrote: > If use_mclk is false, mclk_mode is written to a register without > initialization. This doesn't cause any ill effects as the written value > is not used when use_mclk is false. > > To fix this,

[PATCHv2 7/7] drm/omap: hdmi4: fix use of uninitialized var

2019-09-30 Thread Tomi Valkeinen
If use_mclk is false, mclk_mode is written to a register without initialization. This doesn't cause any ill effects as the written value is not used when use_mclk is false. To fix this, write use_mclk only when use_mclk is true. Signed-off-by: Tomi Valkeinen ---