| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <1542363310.23489.22.camel@pengutronix.de>
Date: Fri, 16 Nov 2018 11:15:10 +0100
From: Lucas Stach <l.stach@...gutronix.de>
To: Fernando Ramos <greenfoo@...egarage.com>,
dri-devel@...ts.freedesktop.org
Cc: David Airlie <airlied@...ux.ie>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <maxime.ripard@...tlin.com>,
Sean Paul <sean@...rly.run>, Jonathan Corbet <corbet@....net>,
Alexey Brodkin <abrodkin@...opsys.com>,
Boris Brezillon <boris.brezillon@...tlin.com>,
Nicolas Ferre <nicolas.ferre@...rochip.com>,
Alexandre Belloni <alexandre.belloni@...tlin.com>,
Ludovic Desroches <ludovic.desroches@...rochip.com>,
Gerd Hoffmann <kraxel@...hat.com>,
Archit Taneja <architt@...eaurora.org>,
Andrzej Hajda <a.hajda@...sung.com>,
Laurent Pinchart <Laurent.pinchart@...asonboard.com>,
Russell King <linux+etnaviv@...linux.org.uk>,
Christian Gmeiner <christian.gmeiner@...il.com>,
Marek Vasut <marex@...x.de>, Stefan Agner <stefan@...er.ch>,
Kieran Bingham <kieran.bingham+renesas@...asonboard.com>,
Linus Walleij <linus.walleij@...aro.org>,
Shawn Guo <shawnguo@...nel.org>,
Julia Lawall <Julia.Lawall@...6.fr>,
Gilles Muller <Gilles.Muller@...6.fr>,
Nicolas Palix <nicolas.palix@...g.fr>,
Michal Marek <michal.lkml@...kovi.net>,
Haneen Mohammed <hamohammed.sa@...il.com>,
linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org,
virtualization@...ts.linux-foundation.org,
etnaviv@...ts.freedesktop.org, linux-renesas-soc@...r.kernel.org,
cocci@...teme.lip6.fr
Subject: Re: [PATCH 3/9] drm: replace "drm_dev_unref" function with
"drm_dev_put"
Am Donnerstag, den 15.11.2018, 23:16 +0100 schrieb Fernando Ramos:
> This patch unifies the naming of DRM functions for reference counting as
> requested on Documentation/gpu/todo.rst
>
> > Signed-off-by: Fernando Ramos <greenfoo@...egarage.com>
> ---
> drivers/gpu/drm/arc/arcpgu_drv.c | 4 ++--
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 4 ++--
> drivers/gpu/drm/etnaviv/etnaviv_drv.c | 4 ++--
At least the etnaviv part of the patch is already fixed in linux-next.
Regards,
Lucas
> drivers/gpu/drm/mxsfb/mxsfb_drv.c | 4 ++--
> drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 +-
> drivers/gpu/drm/shmobile/shmob_drm_drv.c | 4 ++--
> drivers/gpu/drm/tve200/tve200_drv.c | 4 ++--
> drivers/gpu/drm/zte/zx_drm_drv.c | 4 ++--
> 8 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/gpu/drm/arc/arcpgu_drv.c b/drivers/gpu/drm/arc/arcpgu_drv.c
> index f067de4e1e82..dcb06d4e9135 100644
> --- a/drivers/gpu/drm/arc/arcpgu_drv.c
> +++ b/drivers/gpu/drm/arc/arcpgu_drv.c
> @@ -216,7 +216,7 @@ static int arcpgu_probe(struct platform_device *pdev)
> > arcpgu_unload(drm);
>
> err_unref:
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
>
> > return ret;
> }
> @@ -227,7 +227,7 @@ static int arcpgu_remove(struct platform_device *pdev)
>
> > drm_dev_unregister(drm);
> > arcpgu_unload(drm);
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
>
> > return 0;
> }
> diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> index 843cac222e60..f8da51a63e2f 100644
> --- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> +++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c
> @@ -775,7 +775,7 @@ static int atmel_hlcdc_dc_drm_probe(struct platform_device *pdev)
> > atmel_hlcdc_dc_unload(ddev);
>
> err_unref:
> > - drm_dev_unref(ddev);
> > + drm_dev_put(ddev);
>
> > return ret;
> }
> @@ -786,7 +786,7 @@ static int atmel_hlcdc_dc_drm_remove(struct platform_device *pdev)
>
> > drm_dev_unregister(ddev);
> > atmel_hlcdc_dc_unload(ddev);
> > - drm_dev_unref(ddev);
> > + drm_dev_put(ddev);
>
> > return 0;
> }
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> index 83c1f46670bf..52802e6049e0 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> @@ -550,7 +550,7 @@ static int etnaviv_bind(struct device *dev)
> out_bind:
> > kfree(priv);
> out_unref:
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
>
> > return ret;
> }
> @@ -567,7 +567,7 @@ static void etnaviv_unbind(struct device *dev)
> > drm->dev_private = NULL;
> > kfree(priv);
>
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
> }
>
> static const struct component_master_ops etnaviv_master_ops = {
> diff --git a/drivers/gpu/drm/mxsfb/mxsfb_drv.c b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
> index 2393e6d16ffd..88ba003979e6 100644
> --- a/drivers/gpu/drm/mxsfb/mxsfb_drv.c
> +++ b/drivers/gpu/drm/mxsfb/mxsfb_drv.c
> @@ -417,7 +417,7 @@ static int mxsfb_probe(struct platform_device *pdev)
> err_unload:
> > mxsfb_unload(drm);
> err_free:
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
>
> > return ret;
> }
> @@ -428,7 +428,7 @@ static int mxsfb_remove(struct platform_device *pdev)
>
> > drm_dev_unregister(drm);
> > mxsfb_unload(drm);
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
>
> > return 0;
> }
> diff --git a/drivers/gpu/drm/rcar-du/rcar_du_drv.c b/drivers/gpu/drm/rcar-du/rcar_du_drv.c
> index 084f58df4a8c..c7fe2433ab9e 100644
> --- a/drivers/gpu/drm/rcar-du/rcar_du_drv.c
> +++ b/drivers/gpu/drm/rcar-du/rcar_du_drv.c
> @@ -454,7 +454,7 @@ static int rcar_du_remove(struct platform_device *pdev)
> > drm_kms_helper_poll_fini(ddev);
> > drm_mode_config_cleanup(ddev);
>
> > - drm_dev_unref(ddev);
> > + drm_dev_put(ddev);
>
> > return 0;
> }
> diff --git a/drivers/gpu/drm/shmobile/shmob_drm_drv.c b/drivers/gpu/drm/shmobile/shmob_drm_drv.c
> index 6ececad6f845..8554102a6ead 100644
> --- a/drivers/gpu/drm/shmobile/shmob_drm_drv.c
> +++ b/drivers/gpu/drm/shmobile/shmob_drm_drv.c
> @@ -194,7 +194,7 @@ static int shmob_drm_remove(struct platform_device *pdev)
> > drm_kms_helper_poll_fini(ddev);
> > drm_mode_config_cleanup(ddev);
> > drm_irq_uninstall(ddev);
> > - drm_dev_unref(ddev);
> > + drm_dev_put(ddev);
>
> > return 0;
> }
> @@ -290,7 +290,7 @@ static int shmob_drm_probe(struct platform_device *pdev)
> > drm_kms_helper_poll_fini(ddev);
> > drm_mode_config_cleanup(ddev);
> err_free_drm_dev:
> > - drm_dev_unref(ddev);
> > + drm_dev_put(ddev);
>
> > return ret;
> }
> diff --git a/drivers/gpu/drm/tve200/tve200_drv.c b/drivers/gpu/drm/tve200/tve200_drv.c
> index 72efcecb44f7..28e2d03c0ccf 100644
> --- a/drivers/gpu/drm/tve200/tve200_drv.c
> +++ b/drivers/gpu/drm/tve200/tve200_drv.c
> @@ -249,7 +249,7 @@ static int tve200_probe(struct platform_device *pdev)
> clk_disable:
> > clk_disable_unprepare(priv->pclk);
> dev_unref:
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
> > return ret;
> }
>
> @@ -263,7 +263,7 @@ static int tve200_remove(struct platform_device *pdev)
> > drm_panel_bridge_remove(priv->bridge);
> > drm_mode_config_cleanup(drm);
> > clk_disable_unprepare(priv->pclk);
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
>
> > return 0;
> }
> diff --git a/drivers/gpu/drm/zte/zx_drm_drv.c b/drivers/gpu/drm/zte/zx_drm_drv.c
> index 11ef17c2d1c1..5b6f1eda52e0 100644
> --- a/drivers/gpu/drm/zte/zx_drm_drv.c
> +++ b/drivers/gpu/drm/zte/zx_drm_drv.c
> @@ -114,7 +114,7 @@ static int zx_drm_bind(struct device *dev)
> > component_unbind_all(dev, drm);
> out_unregister:
> > dev_set_drvdata(dev, NULL);
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
> > return ret;
> }
>
> @@ -127,7 +127,7 @@ static void zx_drm_unbind(struct device *dev)
> > drm_mode_config_cleanup(drm);
> > component_unbind_all(dev, drm);
> > dev_set_drvdata(dev, NULL);
> > - drm_dev_unref(drm);
> > + drm_dev_put(drm);
> }
>
> static const struct component_master_ops zx_drm_master_ops = {
Powered by blists - more mailing lists