From: Hans Verkuil <hverkuil@xs4all.nl>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Thierry Reding" <thierry.reding@gmail.com>,
"David Airlie" <airlied@gmail.com>,
"Daniel Vetter" <daniel@ffwll.ch>,
"Jonathan Hunter" <jonathanh@nvidia.com>,
"Sowjanya Komatineni" <skomatineni@nvidia.com>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
Cc: dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org,
kernel@pengutronix.de, linux-media@vger.kernel.org,
linux-staging@lists.linux.dev
Subject: Re: [PATCH 01/12] gpu: host1x: Make host1x_client_unregister() return void
Date: Mon, 27 Mar 2023 09:17:26 +0200 [thread overview]
Message-ID: <1c4e5eaf-8a38-41cf-1e4f-6006d575d5ea@xs4all.nl> (raw)
In-Reply-To: <20230322170223.410376-2-u.kleine-koenig@pengutronix.de>
On 22/03/2023 18:02, Uwe Kleine-König wrote:
> This function returned zero unconditionally. Make it return no value and
> simplify all callers accordingly.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
> drivers/gpu/drm/tegra/dc.c | 7 +------
> drivers/gpu/drm/tegra/dsi.c | 8 +-------
> drivers/gpu/drm/tegra/gr2d.c | 8 +-------
> drivers/gpu/drm/tegra/gr3d.c | 8 +-------
> drivers/gpu/drm/tegra/hdmi.c | 8 +-------
> drivers/gpu/drm/tegra/hub.c | 9 ++-------
> drivers/gpu/drm/tegra/nvdec.c | 8 +-------
> drivers/gpu/drm/tegra/sor.c | 8 +-------
> drivers/gpu/drm/tegra/vic.c | 8 +-------
> drivers/gpu/host1x/bus.c | 6 ++----
> drivers/staging/media/tegra-video/csi.c | 8 +-------
> drivers/staging/media/tegra-video/vi.c | 8 +-------
> include/linux/host1x.h | 2 +-
> 13 files changed, 15 insertions(+), 81 deletions(-)
Acked-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Regards,
Hans
>
> diff --git a/drivers/gpu/drm/tegra/dc.c b/drivers/gpu/drm/tegra/dc.c
> index a67453cee883..bd108159fc2c 100644
> --- a/drivers/gpu/drm/tegra/dc.c
> +++ b/drivers/gpu/drm/tegra/dc.c
> @@ -3268,12 +3268,7 @@ static int tegra_dc_remove(struct platform_device *pdev)
> struct tegra_dc *dc = platform_get_drvdata(pdev);
> int err;
>
> - err = host1x_client_unregister(&dc->client);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&dc->client);
>
> err = tegra_dc_rgb_remove(dc);
> if (err < 0) {
> diff --git a/drivers/gpu/drm/tegra/dsi.c b/drivers/gpu/drm/tegra/dsi.c
> index de1333dc0d86..c8a02a1024bc 100644
> --- a/drivers/gpu/drm/tegra/dsi.c
> +++ b/drivers/gpu/drm/tegra/dsi.c
> @@ -1662,16 +1662,10 @@ static int tegra_dsi_probe(struct platform_device *pdev)
> static int tegra_dsi_remove(struct platform_device *pdev)
> {
> struct tegra_dsi *dsi = platform_get_drvdata(pdev);
> - int err;
>
> pm_runtime_disable(&pdev->dev);
>
> - err = host1x_client_unregister(&dsi->client);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&dsi->client);
>
> tegra_output_remove(&dsi->output);
>
> diff --git a/drivers/gpu/drm/tegra/gr2d.c b/drivers/gpu/drm/tegra/gr2d.c
> index e3bb4c99ed39..49b8d4953e7f 100644
> --- a/drivers/gpu/drm/tegra/gr2d.c
> +++ b/drivers/gpu/drm/tegra/gr2d.c
> @@ -298,14 +298,8 @@ static int gr2d_probe(struct platform_device *pdev)
> static int gr2d_remove(struct platform_device *pdev)
> {
> struct gr2d *gr2d = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&gr2d->client.base);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&gr2d->client.base);
>
> return 0;
> }
> diff --git a/drivers/gpu/drm/tegra/gr3d.c b/drivers/gpu/drm/tegra/gr3d.c
> index a1fd3113ea96..cd3679d8eef9 100644
> --- a/drivers/gpu/drm/tegra/gr3d.c
> +++ b/drivers/gpu/drm/tegra/gr3d.c
> @@ -553,14 +553,8 @@ static int gr3d_probe(struct platform_device *pdev)
> static int gr3d_remove(struct platform_device *pdev)
> {
> struct gr3d *gr3d = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&gr3d->client.base);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&gr3d->client.base);
>
> return 0;
> }
> diff --git a/drivers/gpu/drm/tegra/hdmi.c b/drivers/gpu/drm/tegra/hdmi.c
> index 40ec3e6cf204..6798eea317df 100644
> --- a/drivers/gpu/drm/tegra/hdmi.c
> +++ b/drivers/gpu/drm/tegra/hdmi.c
> @@ -1877,14 +1877,8 @@ static int tegra_hdmi_probe(struct platform_device *pdev)
> static int tegra_hdmi_remove(struct platform_device *pdev)
> {
> struct tegra_hdmi *hdmi = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&hdmi->client);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&hdmi->client);
>
> tegra_output_remove(&hdmi->output);
>
> diff --git a/drivers/gpu/drm/tegra/hub.c b/drivers/gpu/drm/tegra/hub.c
> index b872527a123c..2c91dc444085 100644
> --- a/drivers/gpu/drm/tegra/hub.c
> +++ b/drivers/gpu/drm/tegra/hub.c
> @@ -1178,13 +1178,8 @@ static int tegra_display_hub_remove(struct platform_device *pdev)
> {
> struct tegra_display_hub *hub = platform_get_drvdata(pdev);
> unsigned int i;
> - int err;
>
> - err = host1x_client_unregister(&hub->client);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - }
> + host1x_client_unregister(&hub->client);
>
> for (i = 0; i < hub->soc->num_wgrps; i++) {
> struct tegra_windowgroup *wgrp = &hub->wgrps[i];
> @@ -1194,7 +1189,7 @@ static int tegra_display_hub_remove(struct platform_device *pdev)
>
> pm_runtime_disable(&pdev->dev);
>
> - return err;
> + return 0;
> }
>
> static const struct tegra_display_hub_soc tegra186_display_hub = {
> diff --git a/drivers/gpu/drm/tegra/nvdec.c b/drivers/gpu/drm/tegra/nvdec.c
> index 86c5818ac27b..077f3af0d0b6 100644
> --- a/drivers/gpu/drm/tegra/nvdec.c
> +++ b/drivers/gpu/drm/tegra/nvdec.c
> @@ -550,14 +550,8 @@ static int nvdec_probe(struct platform_device *pdev)
> static int nvdec_remove(struct platform_device *pdev)
> {
> struct nvdec *nvdec = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&nvdec->client.base);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&nvdec->client.base);
>
> falcon_exit(&nvdec->falcon);
>
> diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c
> index 8af632740673..d067b2d01b2e 100644
> --- a/drivers/gpu/drm/tegra/sor.c
> +++ b/drivers/gpu/drm/tegra/sor.c
> @@ -3981,14 +3981,8 @@ static int tegra_sor_probe(struct platform_device *pdev)
> static int tegra_sor_remove(struct platform_device *pdev)
> {
> struct tegra_sor *sor = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&sor->client);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&sor->client);
>
> pm_runtime_disable(&pdev->dev);
>
> diff --git a/drivers/gpu/drm/tegra/vic.c b/drivers/gpu/drm/tegra/vic.c
> index 531a71c72061..fd614756ecf8 100644
> --- a/drivers/gpu/drm/tegra/vic.c
> +++ b/drivers/gpu/drm/tegra/vic.c
> @@ -540,14 +540,8 @@ static int vic_probe(struct platform_device *pdev)
> static int vic_remove(struct platform_device *pdev)
> {
> struct vic *vic = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&vic->client.base);
> - if (err < 0) {
> - dev_err(&pdev->dev, "failed to unregister host1x client: %d\n",
> - err);
> - return err;
> - }
> + host1x_client_unregister(&vic->client.base);
>
> falcon_exit(&vic->falcon);
>
> diff --git a/drivers/gpu/host1x/bus.c b/drivers/gpu/host1x/bus.c
> index bc7271a00a94..4d16a3396c4a 100644
> --- a/drivers/gpu/host1x/bus.c
> +++ b/drivers/gpu/host1x/bus.c
> @@ -803,7 +803,7 @@ EXPORT_SYMBOL(__host1x_client_register);
> * Removes a host1x client from its host1x controller instance. If a logical
> * device has already been initialized, it will be torn down.
> */
> -int host1x_client_unregister(struct host1x_client *client)
> +void host1x_client_unregister(struct host1x_client *client)
> {
> struct host1x_client *c;
> struct host1x *host1x;
> @@ -815,7 +815,7 @@ int host1x_client_unregister(struct host1x_client *client)
> err = host1x_del_client(host1x, client);
> if (!err) {
> mutex_unlock(&devices_lock);
> - return 0;
> + return;
> }
> }
>
> @@ -832,8 +832,6 @@ int host1x_client_unregister(struct host1x_client *client)
> mutex_unlock(&clients_lock);
>
> host1x_bo_cache_destroy(&client->cache);
> -
> - return 0;
> }
> EXPORT_SYMBOL(host1x_client_unregister);
>
> diff --git a/drivers/staging/media/tegra-video/csi.c b/drivers/staging/media/tegra-video/csi.c
> index 426e653bd55d..36ca639622c9 100644
> --- a/drivers/staging/media/tegra-video/csi.c
> +++ b/drivers/staging/media/tegra-video/csi.c
> @@ -778,14 +778,8 @@ static int tegra_csi_probe(struct platform_device *pdev)
> static int tegra_csi_remove(struct platform_device *pdev)
> {
> struct tegra_csi *csi = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&csi->client);
> - if (err < 0) {
> - dev_err(&pdev->dev,
> - "failed to unregister host1x client: %d\n", err);
> - return err;
> - }
> + host1x_client_unregister(&csi->client);
>
> pm_runtime_disable(&pdev->dev);
>
> diff --git a/drivers/staging/media/tegra-video/vi.c b/drivers/staging/media/tegra-video/vi.c
> index 11dd142c98c5..26f7aedce718 100644
> --- a/drivers/staging/media/tegra-video/vi.c
> +++ b/drivers/staging/media/tegra-video/vi.c
> @@ -2041,14 +2041,8 @@ static int tegra_vi_probe(struct platform_device *pdev)
> static int tegra_vi_remove(struct platform_device *pdev)
> {
> struct tegra_vi *vi = platform_get_drvdata(pdev);
> - int err;
>
> - err = host1x_client_unregister(&vi->client);
> - if (err < 0) {
> - dev_err(&pdev->dev,
> - "failed to unregister host1x client: %d\n", err);
> - return err;
> - }
> + host1x_client_unregister(&vi->client);
>
> pm_runtime_disable(&pdev->dev);
>
> diff --git a/include/linux/host1x.h b/include/linux/host1x.h
> index 9a9de4b97a25..9c8119ed13a4 100644
> --- a/include/linux/host1x.h
> +++ b/include/linux/host1x.h
> @@ -443,7 +443,7 @@ int __host1x_client_register(struct host1x_client *client);
> __host1x_client_register(client); \
> })
>
> -int host1x_client_unregister(struct host1x_client *client);
> +void host1x_client_unregister(struct host1x_client *client);
>
> int host1x_client_suspend(struct host1x_client *client);
> int host1x_client_resume(struct host1x_client *client);
next prev parent reply other threads:[~2023-03-27 7:17 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-22 17:02 [PATCH 00/12] drm/tegra: Convert to platform remove callback returning void Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 01/12] gpu: host1x: Make host1x_client_unregister() return void Uwe Kleine-König
2023-03-27 7:17 ` Hans Verkuil [this message]
2023-03-22 17:02 ` [PATCH 02/12] drm/tegra: rgb: Make tegra_dc_rgb_remove() " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 03/12] drm/tegra: dc: Convert to platform remove callback returning void Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 04/12] drm/tegra: dpaux: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 05/12] drm/tegra: dsi: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 06/12] drm/tegra: gr2d: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 07/12] drm/tegra: gr3d: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 08/12] drm/tegra: hdmi: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 09/12] drm/tegra: hub: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 10/12] drm/tegra: nvdec: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 11/12] drm/tegra: sor: " Uwe Kleine-König
2023-03-22 17:02 ` [PATCH 12/12] drm/tegra: vic: " Uwe Kleine-König
2023-04-04 12:14 ` [PATCH 00/12] drm/tegra: " Thierry Reding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1c4e5eaf-8a38-41cf-1e4f-6006d575d5ea@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=airlied@gmail.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=jonathanh@nvidia.com \
--cc=kernel@pengutronix.de \
--cc=linux-media@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=skomatineni@nvidia.com \
--cc=thierry.reding@gmail.com \
--cc=u.kleine-koenig@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).