[PATCH v1 5/7] drm/mediatek: Drop chain_mode_fixup call in mode_valid()

Maxime Ripard maxime at cerno.tech
Thu Jul 22 00:31:48 PDT 2021


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 <sam at ravnborg.org>
> Cc: Chun-Kuang Hu <chunkuang.hu at kernel.org>
> Cc: Philipp Zabel <p.zabel at pengutronix.de>
> Cc: Matthias Brugger <matthias.bgg at gmail.com>
> Cc: Dafna Hirschfeld <dafna.hirschfeld at collabora.com>
> Cc: linux-mediatek at lists.infradead.org
> Cc: linux-arm-kernel at lists.infradead.org

Reviewed-by: Maxime Ripard <maxime at cerno.tech>

Maxime



More information about the Linux-mediatek mailing list