Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754223AbbLINZ7 (ORCPT ); Wed, 9 Dec 2015 08:25:59 -0500 Received: from down.free-electrons.com ([37.187.137.238]:36291 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751492AbbLINZ5 (ORCPT ); Wed, 9 Dec 2015 08:25:57 -0500 Date: Wed, 9 Dec 2015 14:25:54 +0100 From: Boris Brezillon To: Nicolas Iooss Cc: David Airlie , Jianwei Wang , Alison Wang , Mark Yao , Thierry Reding , Terje =?UTF-8?B?QmVyZ3N0csO2bQ==?= , Stephen Warren , dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] drm: use dev_name as default unique name in drm_dev_alloc() Message-ID: <20151209142554.40fe61d3@bbrezillon> In-Reply-To: <1449612732-32438-2-git-send-email-nicolas.iooss_linux@m4x.org> References: <1449612732-32438-1-git-send-email-nicolas.iooss_linux@m4x.org> <1449612732-32438-2-git-send-email-nicolas.iooss_linux@m4x.org> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.27; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5070 Lines: 146 On Tue, 8 Dec 2015 23:12:12 +0100 Nicolas Iooss wrote: > The following code pattern exists in some DRM drivers: > > ddev = drm_dev_alloc(&driver, parent_dev); > drm_dev_set_unique(ddev, dev_name(parent_dev)); > > (Sometimes dev_name(ddev->dev) is used, which is the same.) > > As suggested in > http://lists.freedesktop.org/archives/dri-devel/2015-December/096441.html, > the unique name of a new DRM device can be set as dev_name(parent_dev) > when parent_dev is not NULL (vgem is a special case). > > Signed-off-by: Nicolas Iooss [for atmel-hlcdc] Acked-by: Boris Brezillon > --- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 4 ---- > drivers/gpu/drm/drm_drv.c | 9 +++++++++ > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 1 - > drivers/gpu/drm/nouveau/nouveau_drm.c | 4 ---- > drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 4 ---- > drivers/gpu/drm/tegra/drm.c | 1 - > drivers/gpu/drm/vc4/vc4_drv.c | 2 -- > 7 files changed, 9 insertions(+), 16 deletions(-) > > diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c > index 244df0a440b7..fba4f72e7ae1 100644 > --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c > +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c > @@ -733,10 +733,6 @@ static int atmel_hlcdc_dc_drm_probe(struct platform_device *pdev) > if (!ddev) > return -ENOMEM; > > - ret = drm_dev_set_unique(ddev, dev_name(ddev->dev)); > - if (ret) > - goto err_unref; > - > ret = atmel_hlcdc_dc_load(ddev); > if (ret) > goto err_unref; > diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c > index 20eaa0aae205..df749a6156e0 100644 > --- a/drivers/gpu/drm/drm_drv.c > +++ b/drivers/gpu/drm/drm_drv.c > @@ -633,8 +633,17 @@ struct drm_device *drm_dev_alloc(struct drm_driver *driver, > } > } > > + if (parent) { > + ret = drm_dev_set_unique(dev, dev_name(parent)); > + if (ret) > + goto err_setunique; > + } > + > return dev; > > +err_setunique: > + if (drm_core_check_feature(dev, DRIVER_GEM)) > + drm_gem_destroy(dev); > err_ctxbitmap: > drm_legacy_ctxbitmap_cleanup(dev); > drm_ht_remove(&dev->map_hash); > diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c > index 1930234ba5f1..fca97d3fc846 100644 > --- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c > @@ -363,7 +363,6 @@ static int fsl_dcu_drm_probe(struct platform_device *pdev) > fsl_dev->np = dev->of_node; > drm->dev_private = fsl_dev; > dev_set_drvdata(dev, fsl_dev); > - drm_dev_set_unique(drm, dev_name(dev)); > > ret = drm_dev_register(drm, 0); > if (ret < 0) > diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c b/drivers/gpu/drm/nouveau/nouveau_drm.c > index 2d23f95f17ce..b3a563c44bcd 100644 > --- a/drivers/gpu/drm/nouveau/nouveau_drm.c > +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c > @@ -1046,10 +1046,6 @@ nouveau_platform_device_create(const struct nvkm_device_tegra_func *func, > goto err_free; > } > > - err = drm_dev_set_unique(drm, dev_name(&pdev->dev)); > - if (err < 0) > - goto err_free; > - > drm->platformdev = pdev; > platform_set_drvdata(pdev, drm); > > diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > index 215d6c44af55..afbb7407c44f 100644 > --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c > @@ -450,10 +450,6 @@ static int rockchip_drm_bind(struct device *dev) > if (!drm) > return -ENOMEM; > > - ret = drm_dev_set_unique(drm, dev_name(dev)); > - if (ret) > - goto err_free; > - > ret = drm_dev_register(drm, 0); > if (ret) > goto err_free; > diff --git a/drivers/gpu/drm/tegra/drm.c b/drivers/gpu/drm/tegra/drm.c > index 159ef515cab1..12e2d3ccbc9d 100644 > --- a/drivers/gpu/drm/tegra/drm.c > +++ b/drivers/gpu/drm/tegra/drm.c > @@ -991,7 +991,6 @@ static int host1x_drm_probe(struct host1x_device *dev) > if (!drm) > return -ENOMEM; > > - drm_dev_set_unique(drm, dev_name(&dev->dev)); > dev_set_drvdata(&dev->dev, drm); > > err = drm_dev_register(drm, 0); > diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c > index d5db9e0f3b73..647772305e8f 100644 > --- a/drivers/gpu/drm/vc4/vc4_drv.c > +++ b/drivers/gpu/drm/vc4/vc4_drv.c > @@ -168,8 +168,6 @@ static int vc4_drm_bind(struct device *dev) > vc4->dev = drm; > drm->dev_private = vc4; > > - drm_dev_set_unique(drm, dev_name(dev)); > - > drm_mode_config_init(drm); > if (ret) > goto unref; -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- 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/