Messages in this thread | | | Date | Mon, 22 Sep 2014 09:27:33 +0800 | From | Mark yao <> | Subject | Re: [PATCH v3 1/5] drm/rockchip: Add basic drm driver |
| |
On 2014年09月19日 21:04, David Herrmann wrote: > Hi > > On Fri, Sep 19, 2014 at 7:47 AM, Mark yao <mark.yao@rock-chips.com> wrote: > [snip] >> +static int rockchip_drm_bind(struct device *dev) >> +{ >> + return drm_platform_init(&rockchip_drm_driver, to_platform_device(dev)); > Please avoid drm_platform_*() usage. We're about to drop all the > drm_bus midlayers. See the tegra driver how to do it, but basically > just this: OK, I will do it. > struct drm_device *ddev; > > ddev = drm_dev_alloc(&rockchip_drm_driver, &to_platform_device(dev)->dev); > if (!ddev) > return -ENOMEM; > > r = drm_dev_set_unique(ddev, dev_name(&ddev->dev)); > if (r < 0) { > drm_dev_unref(ddev); > return r; > } > > r = drm_dev_register(ddev); > if (r < 0) { > drm_dev_unref(ddev); > return r; > } > >> +} >> + >> +static void rockchip_drm_unbind(struct device *dev) >> +{ >> + drm_put_dev(dev_get_drvdata(dev)); > Please use: > > struct drm_device *ddev = dev_get_drvdata(dev); > > drm_dev_unregister(ddev); > drm_dev_unref(ddev); > > > Thanks > David > > >
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |