drm: kirin: Fix dev->driver_data setting
authorXu YiPing <xuyiping@hisilicon.com>
Tue, 20 Aug 2019 23:06:22 +0000 (23:06 +0000)
committerSam Ravnborg <sam@ravnborg.org>
Wed, 21 Aug 2019 17:15:44 +0000 (19:15 +0200)
As part of refactoring the kirin driver to better support
different hardware revisions, this patch changes the
dev->driver_data to point to a drm_device, not ade_data.

Thus we set the driver data to drm device after alloc.

Cc: Rongrong Zou <zourongrong@gmail.com>
Cc: Xinliang Liu <z.liuxinliang@hisilicon.com>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: dri-devel <dri-devel@lists.freedesktop.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Acked-by: Xinliang Liu <z.liuxinliang@hisilicon.com>
Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: Xu YiPing <xuyiping@hisilicon.com>
[jstultz: Reworded commit message]
Signed-off-by: John Stultz <john.stultz@linaro.org>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20190820230626.23253-22-john.stultz@linaro.org
drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c
drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c

index e390b1b..d8e40fc 100644 (file)
@@ -999,7 +999,6 @@ static int ade_drm_init(struct platform_device *pdev)
                DRM_ERROR("failed to alloc ade_data\n");
                return -ENOMEM;
        }
-       platform_set_drvdata(pdev, ade);
 
        ctx = ade_hw_ctx_alloc(pdev, &ade->crtc.base);
        if (IS_ERR(ctx)) {
index 68de883..7f8d453 100644 (file)
@@ -44,8 +44,6 @@ static int kirin_drm_kms_init(struct drm_device *dev)
 {
        int ret;
 
-       dev_set_drvdata(dev->dev, dev);
-
        /* dev->mode_config initialization */
        drm_mode_config_init(dev);
        dev->mode_config.min_width = 0;
@@ -140,6 +138,7 @@ static int kirin_drm_bind(struct device *dev)
        drm_dev = drm_dev_alloc(driver_data->driver, dev);
        if (IS_ERR(drm_dev))
                return PTR_ERR(drm_dev);
+       dev_set_drvdata(dev, drm_dev);
 
        ret = kirin_drm_kms_init(drm_dev);
        if (ret)