Re: [PATCH] drm/omap: sdi: fix bridge enable/disable

[Tomi Valkeinen <tomi.valkeinen@ti.com>]

On 28/11/2020 19:45, Aaro Koskinen wrote:
> Hi,
> 
> On Fri, Nov 27, 2020 at 10:52:41AM +0200, Tomi Valkeinen wrote:
>> When the SDI output was converted to DRM bridge, the atomic versions of
>> enable and disable funcs were used. This was not intended, as that would
>> require implementing other atomic funcs too. This leads to:
>>
>> WARNING: CPU: 0 PID: 18 at drivers/gpu/drm/drm_bridge.c:708 drm_atomic_helper_commit_modeset_enables+0x134/0x268
>>
>> and display not working.
>>
>> Fix this by using the legacy enable/disable funcs.
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> Reported-by: Aaro Koskinen <aaro.koskinen@iki.fi>
>> Fixes: 8bef8a6d5da81b909a190822b96805a47348146f ("drm/omap: sdi: Register a drm_bridge")
>> Cc: stable@vger.kernel.org # v5.7+
>> Tested-by: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
> 
> Tested-by: Aaro Koskinen <aaro.koskinen@iki.fi>

Thanks! I pushed this to drm-misc-fixes. Hopefully with this and Sebastian's patch N900 display now
works.

 Tomi

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki