From: Douglas Anderson Date: Mon, 7 Mar 2016 22:00:53 +0000 (-0800) Subject: FROMLIST: drm/rockchip: dw_hdmi: Don't call platform_set_drvdata() X-Git-Tag: firefly_0821_release~2948 X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=7dcfa54210cddc4188b94ccc8102ff4019bf30e3;p=firefly-linux-kernel-4.4.55.git FROMLIST: drm/rockchip: dw_hdmi: Don't call platform_set_drvdata() The Rockchip dw_hdmi driver just called platform_set_drvdata() to get your hopes up that maybe, somehow, you'd be able to retrieve the 'struct rockchip_hdmi' from a pointer to the 'struct device'. You can't. When we call dw_hdmi_bind() the main driver calls dev_set_drvdata(), which clobbers our setting. Let's just remove the platform_set_drvdata() to avoid dashing people's hopes. (am from https://patchwork.kernel.org/patch/8523401/) Change-Id: I28c4dcff37f6800b841e0492eb2613dcff7d1c81 Signed-off-by: Douglas Anderson Signed-off-by: Mark Yao --- diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c index e5d5b18a841b..3ac8c4ee19d9 100644 --- a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c +++ b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c @@ -271,8 +271,6 @@ static int dw_hdmi_rockchip_bind(struct device *dev, struct device *master, if (!iores) return -ENXIO; - platform_set_drvdata(pdev, hdmi); - encoder->possible_crtcs = drm_of_find_possible_crtcs(drm, dev->of_node); /* * If we failed to find the CRTC(s) which this encoder is