drm/vc4: hdmi: Rework the pre_crtc_configure error handling
authorMaxime Ripard <maxime@cerno.tech>
Thu, 19 Aug 2021 13:59:28 +0000 (15:59 +0200)
committerMaxime Ripard <maxime@cerno.tech>
Mon, 25 Oct 2021 13:33:40 +0000 (15:33 +0200)
commitcaa51a4c11f1cadba9bcf61ed9e0105711952ce7
tree3b034bdc5b01d2879b88743cdf9ecaecb2ca28f6
parent9c6e4f6ed1d61d5f46946e5c151ceb279eedadb1
drm/vc4: hdmi: Rework the pre_crtc_configure error handling

Since our pre_crtc_configure hook returned void, we didn't implement a
goto-based error path handling, leading to errors like failing to put
back the device in pm_runtime in all the error paths, but also failing
to disable the pixel clock if clk_set_min_rate on the HSM clock fails.

Move to a goto-based implementation to have an easier consitency.

Fixes: 4f6e3d66ac52 ("drm/vc4: Add runtime PM support to the HDMI encoder driver")
Link: https://patchwork.freedesktop.org/patch/msgid/20210819135931.895976-4-maxime@cerno.tech
Reviewed-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
drivers/gpu/drm/vc4/vc4_hdmi.c