Instead of detaching only the bridge directly connected to the encoder,
detach all bridges in the chain.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com>
---
 drivers/gpu/drm/drm_encoder.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_encoder.c b/drivers/gpu/drm/drm_encoder.c
index 0944f08abf58..5dde3c6b1467 100644
--- a/drivers/gpu/drm/drm_encoder.c
+++ b/drivers/gpu/drm/drm_encoder.c
@@ -159,8 +159,16 @@ void drm_encoder_cleanup(struct drm_encoder *encoder)
         * the indices on the drm_encoder after us in the encoder_list.
         */

-       if (encoder->bridge)
-               drm_bridge_detach(encoder->bridge);
+       if (encoder->bridge) {
+               struct drm_bridge *bridge = encoder->bridge;
+               struct drm_bridge *next;
+
+               while (bridge) {
+                       next = bridge->next;
+                       drm_bridge_detach(bridge);
+                       bridge = next;
+               }
+       }

        drm_mode_object_unregister(dev, &encoder->base);
        kfree(encoder->name);
-- 
Regards,

Laurent Pinchart

Reply via email to