Make sure to drop the reference taken when looking up the companion
(ganged) device and its driver data during probe().

Note that holding a reference to a device does not prevent its driver
data from going away so there is no point in keeping the reference.

Fixes: e94236cde4d5 ("drm/tegra: dsi: Add ganged mode support")
Fixes: 221e3638feb8 ("drm/tegra: Fix reference leak in tegra_dsi_ganged_probe")
Cc: [email protected]      # 3.19: 221e3638feb8
Cc: Thierry Reding <[email protected]>
Signed-off-by: Johan Hovold <[email protected]>
---
 drivers/gpu/drm/tegra/dsi.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/tegra/dsi.c b/drivers/gpu/drm/tegra/dsi.c
index b5089b772267..b53038eb9699 100644
--- a/drivers/gpu/drm/tegra/dsi.c
+++ b/drivers/gpu/drm/tegra/dsi.c
@@ -1549,11 +1549,9 @@ static int tegra_dsi_ganged_probe(struct tegra_dsi *dsi)
                        return -EPROBE_DEFER;
 
                dsi->slave = platform_get_drvdata(gangster);
-
-               if (!dsi->slave) {
-                       put_device(&gangster->dev);
+               put_device(&gangster->dev);
+               if (!dsi->slave)
                        return -EPROBE_DEFER;
-               }
 
                dsi->slave->master = dsi;
        }
-- 
2.51.2

Reply via email to