Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1690040 > unrolled thread

[PATCH v1 6/7] drm/stm: ltdc: Cleanup rename returned value

Started byPhilippe CORNU <philippe.cornu@st.com>
First post2017-07-18 12:30 +0200
Last post2017-07-20 11:40 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH v1 6/7] drm/stm: ltdc: Cleanup rename returned value Philippe CORNU <philippe.cornu@st.com> - 2017-07-18 12:30 +0200
    Re: [PATCH v1 6/7] drm/stm: ltdc: Cleanup rename returned value Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-07-20 11:40 +0200

#1690040 — [PATCH v1 6/7] drm/stm: ltdc: Cleanup rename returned value

FromPhilippe CORNU <philippe.cornu@st.com>
Date2017-07-18 12:30 +0200
Subject[PATCH v1 6/7] drm/stm: ltdc: Cleanup rename returned value
Message-ID<u4xVp-1aY-41@gated-at.bofh.it>
Signed-off-by: Philippe CORNU <philippe.cornu@st.com>
---
 drivers/gpu/drm/stm/ltdc.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index f4ed21a..8cd1b9b 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -760,7 +760,7 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
 	struct ltdc_device *ldev = ddev->dev_private;
 	struct drm_plane *primary, *overlay;
 	unsigned int i;
-	int res;
+	int ret;
 
 	primary = ltdc_plane_create(ddev, DRM_PLANE_TYPE_PRIMARY);
 	if (!primary) {
@@ -768,9 +768,9 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
 		return -EINVAL;
 	}
 
-	res = drm_crtc_init_with_planes(ddev, crtc, primary, NULL,
+	ret = drm_crtc_init_with_planes(ddev, crtc, primary, NULL,
 					&ltdc_crtc_funcs, NULL);
-	if (res) {
+	if (ret) {
 		DRM_ERROR("Can not initialize CRTC\n");
 		goto cleanup;
 	}
@@ -783,7 +783,7 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
 	for (i = 1; i < ldev->caps.nb_layers; i++) {
 		overlay = ltdc_plane_create(ddev, DRM_PLANE_TYPE_OVERLAY);
 		if (!overlay) {
-			res = -ENOMEM;
+			ret = -ENOMEM;
 			DRM_ERROR("Can not create overlay plane %d\n", i);
 			goto cleanup;
 		}
@@ -793,7 +793,7 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
 
 cleanup:
 	ltdc_plane_destroy_all(ddev);
-	return res;
+	return ret;
 }
 
 /*
-- 
1.9.1

[toc] | [next] | [standalone]


#1692693

FromBenjamin Gaignard <benjamin.gaignard@linaro.org>
Date2017-07-20 11:40 +0200
Message-ID<u5g65-5r9-5@gated-at.bofh.it>
In reply to#1690040
2017-07-18 12:20 GMT+02:00 Philippe CORNU <philippe.cornu@st.com>:

Commit message is missing here


> Signed-off-by: Philippe CORNU <philippe.cornu@st.com>
> ---
>  drivers/gpu/drm/stm/ltdc.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index f4ed21a..8cd1b9b 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -760,7 +760,7 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
>         struct ltdc_device *ldev = ddev->dev_private;
>         struct drm_plane *primary, *overlay;
>         unsigned int i;
> -       int res;
> +       int ret;
>
>         primary = ltdc_plane_create(ddev, DRM_PLANE_TYPE_PRIMARY);
>         if (!primary) {
> @@ -768,9 +768,9 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
>                 return -EINVAL;
>         }
>
> -       res = drm_crtc_init_with_planes(ddev, crtc, primary, NULL,
> +       ret = drm_crtc_init_with_planes(ddev, crtc, primary, NULL,
>                                         &ltdc_crtc_funcs, NULL);
> -       if (res) {
> +       if (ret) {
>                 DRM_ERROR("Can not initialize CRTC\n");
>                 goto cleanup;
>         }
> @@ -783,7 +783,7 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
>         for (i = 1; i < ldev->caps.nb_layers; i++) {
>                 overlay = ltdc_plane_create(ddev, DRM_PLANE_TYPE_OVERLAY);
>                 if (!overlay) {
> -                       res = -ENOMEM;
> +                       ret = -ENOMEM;
>                         DRM_ERROR("Can not create overlay plane %d\n", i);
>                         goto cleanup;
>                 }
> @@ -793,7 +793,7 @@ static int ltdc_crtc_init(struct drm_device *ddev, struct drm_crtc *crtc)
>
>  cleanup:
>         ltdc_plane_destroy_all(ddev);
> -       return res;
> +       return ret;
>  }
>
>  /*
> --
> 1.9.1
>



-- 
Benjamin Gaignard

Graphic Study Group

Linaro.org │ Open source software for ARM SoCs

Follow Linaro: Facebook | Twitter | Blog

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web