From: Tomasz Figa <[email protected]> Current implementation of unbind dereferences the drvdata pointer assuming that it's its own data, however the ownership belongs to the analogix code, which means that the pointer is dereferenced with wrong type. Fix this by using the recently added platform data .cleanup() callback to do Rockchip-specific things at unbind.
Cc: Kristian H. Kristensen <[email protected]> Cc: Brian Norris <[email protected]> Signed-off-by: Tomasz Figa <[email protected]> [seanpaul fixed conflict keeping rockchip_dp_unbind() preserving clk_unprepare] Signed-off-by: Sean Paul <[email protected]> Signed-off-by: Thierry Escande <[email protected]> --- drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c index a6a3d6f1f127..eaa025730c28 100644 --- a/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c +++ b/drivers/gpu/drm/rockchip/analogix_dp-rockchip.c @@ -161,6 +161,13 @@ static int rockchip_dp_get_modes(struct analogix_dp_plat_data *plat_data, return 0; } +static void rockchip_dp_cleanup(struct analogix_dp_plat_data *plat_data) +{ + struct rockchip_dp_device *dp = to_dp(plat_data); + + rockchip_drm_psr_unregister(&dp->encoder); +} + static bool rockchip_dp_drm_encoder_mode_fixup(struct drm_encoder *encoder, const struct drm_display_mode *mode, @@ -333,6 +340,7 @@ static int rockchip_dp_bind(struct device *dev, struct device *master, dp->plat_data.power_on = rockchip_dp_poweron; dp->plat_data.power_off = rockchip_dp_powerdown; dp->plat_data.get_modes = rockchip_dp_get_modes; + dp->plat_data.cleanup = rockchip_dp_cleanup; ret = rockchip_drm_psr_register(&dp->encoder, analogix_dp_psr_set); if (ret < 0) @@ -358,7 +366,6 @@ static void rockchip_dp_unbind(struct device *dev, struct device *master, struct rockchip_dp_device *dp = dev_get_drvdata(dev); analogix_dp_unbind(dp->adp); - rockchip_drm_psr_unregister(&dp->encoder); dp->encoder.funcs->destroy(&dp->encoder); } -- 2.14.1 _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
