On Thu, Jul 22, 2021 at 08:22:44AM +0200, Sam Ravnborg wrote: > The mode_valid implementation had a call to > drm_bridge_chain_mode_fixup() which would be wrong as the mode_valid is > not allowed to change anything - only to validate the mode. > > As the next bridge is often/always a connector the call had no effect > anyway. So drop it. > > From the git history I could see this call was included in the original > version of the driver so there was no help there to find out why it was > added in the first place. But a lot has changed since the initial driver > were added and is seems safe to remove the call now. > > Signed-off-by: Sam Ravnborg <s...@ravnborg.org> > Cc: Chun-Kuang Hu <chunkuang...@kernel.org> > Cc: Philipp Zabel <p.za...@pengutronix.de> > Cc: Matthias Brugger <matthias....@gmail.com> > Cc: Dafna Hirschfeld <dafna.hirschf...@collabora.com> > Cc: linux-media...@lists.infradead.org > Cc: linux-arm-ker...@lists.infradead.org
Reviewed-by: Maxime Ripard <max...@cerno.tech> Maxime
