* [PATCH] drm/stm: ltdc: fix various coding-style warnings
@ 2022-06-03 13:43 ` Yannick Fertre
0 siblings, 0 replies; 6+ messages in thread
From: Yannick Fertre @ 2022-06-03 13:43 UTC (permalink / raw)
To: Yannick Fertre, Philippe Cornu, Raphael Gallais-Pou,
David Airlie, Daniel Vetter, Maxime Coquelin, Alexandre Torgue,
dri-devel, linux-stm32, linux-arm-kernel, linux-kernel
Fix issues reported by checkpatch.pl:
- Braces {} should be used on all arms
- Blank lines
Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
---
drivers/gpu/drm/stm/ltdc.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index a4098aaff243..6a9f613839b5 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -908,9 +908,9 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
drm_connector_list_iter_end(&iter);
}
- if (bridge && bridge->timings)
+ if (bridge && bridge->timings) {
bus_flags = bridge->timings->input_bus_flags;
- else if (connector) {
+ } else if (connector) {
bus_flags = connector->display_info.bus_flags;
if (connector->display_info.num_bus_formats)
bus_formats = connector->display_info.bus_formats[0];
@@ -1917,7 +1917,6 @@ int ltdc_load(struct drm_device *ddev)
DRM_ERROR("Failed to register LTDC interrupt\n");
goto err;
}
-
}
crtc = devm_kzalloc(dev, sizeof(*crtc), GFP_KERNEL);
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH] drm/stm: ltdc: fix various coding-style warnings
@ 2022-06-03 13:43 ` Yannick Fertre
0 siblings, 0 replies; 6+ messages in thread
From: Yannick Fertre @ 2022-06-03 13:43 UTC (permalink / raw)
To: Yannick Fertre, Philippe Cornu, Raphael Gallais-Pou,
David Airlie, Daniel Vetter, Maxime Coquelin, Alexandre Torgue,
dri-devel, linux-stm32, linux-arm-kernel, linux-kernel
Fix issues reported by checkpatch.pl:
- Braces {} should be used on all arms
- Blank lines
Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
---
drivers/gpu/drm/stm/ltdc.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index a4098aaff243..6a9f613839b5 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -908,9 +908,9 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
drm_connector_list_iter_end(&iter);
}
- if (bridge && bridge->timings)
+ if (bridge && bridge->timings) {
bus_flags = bridge->timings->input_bus_flags;
- else if (connector) {
+ } else if (connector) {
bus_flags = connector->display_info.bus_flags;
if (connector->display_info.num_bus_formats)
bus_formats = connector->display_info.bus_formats[0];
@@ -1917,7 +1917,6 @@ int ltdc_load(struct drm_device *ddev)
DRM_ERROR("Failed to register LTDC interrupt\n");
goto err;
}
-
}
crtc = devm_kzalloc(dev, sizeof(*crtc), GFP_KERNEL);
--
2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/stm: ltdc: fix various coding-style warnings
2022-06-03 13:43 ` Yannick Fertre
@ 2022-06-09 15:36 ` Raphael Gallais-Pou
-1 siblings, 0 replies; 6+ messages in thread
From: Raphael Gallais-Pou @ 2022-06-09 15:36 UTC (permalink / raw)
To: Yannick Fertre, Philippe Cornu, David Airlie, Daniel Vetter,
Maxime Coquelin, Alexandre Torgue, dri-devel, linux-stm32,
linux-arm-kernel, linux-kernel
Hi,
Thanks
Acked-by: Raphael Gallais-Pou <raphael.gallais-pou@foss.st.com>
Cheers,
Raphaël
On 6/3/22 15:43, Yannick Fertre wrote:
> Fix issues reported by checkpatch.pl:
> - Braces {} should be used on all arms
> - Blank lines
>
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> ---
> drivers/gpu/drm/stm/ltdc.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index a4098aaff243..6a9f613839b5 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -908,9 +908,9 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
> drm_connector_list_iter_end(&iter);
> }
>
> - if (bridge && bridge->timings)
> + if (bridge && bridge->timings) {
> bus_flags = bridge->timings->input_bus_flags;
> - else if (connector) {
> + } else if (connector) {
> bus_flags = connector->display_info.bus_flags;
> if (connector->display_info.num_bus_formats)
> bus_formats = connector->display_info.bus_formats[0];
> @@ -1917,7 +1917,6 @@ int ltdc_load(struct drm_device *ddev)
> DRM_ERROR("Failed to register LTDC interrupt\n");
> goto err;
> }
> -
> }
>
> crtc = devm_kzalloc(dev, sizeof(*crtc), GFP_KERNEL);
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/stm: ltdc: fix various coding-style warnings
@ 2022-06-09 15:36 ` Raphael Gallais-Pou
0 siblings, 0 replies; 6+ messages in thread
From: Raphael Gallais-Pou @ 2022-06-09 15:36 UTC (permalink / raw)
To: Yannick Fertre, Philippe Cornu, David Airlie, Daniel Vetter,
Maxime Coquelin, Alexandre Torgue, dri-devel, linux-stm32,
linux-arm-kernel, linux-kernel
Hi,
Thanks
Acked-by: Raphael Gallais-Pou <raphael.gallais-pou@foss.st.com>
Cheers,
Raphaël
On 6/3/22 15:43, Yannick Fertre wrote:
> Fix issues reported by checkpatch.pl:
> - Braces {} should be used on all arms
> - Blank lines
>
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> ---
> drivers/gpu/drm/stm/ltdc.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index a4098aaff243..6a9f613839b5 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -908,9 +908,9 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
> drm_connector_list_iter_end(&iter);
> }
>
> - if (bridge && bridge->timings)
> + if (bridge && bridge->timings) {
> bus_flags = bridge->timings->input_bus_flags;
> - else if (connector) {
> + } else if (connector) {
> bus_flags = connector->display_info.bus_flags;
> if (connector->display_info.num_bus_formats)
> bus_formats = connector->display_info.bus_formats[0];
> @@ -1917,7 +1917,6 @@ int ltdc_load(struct drm_device *ddev)
> DRM_ERROR("Failed to register LTDC interrupt\n");
> goto err;
> }
> -
> }
>
> crtc = devm_kzalloc(dev, sizeof(*crtc), GFP_KERNEL);
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/stm: ltdc: fix various coding-style warnings
2022-06-03 13:43 ` Yannick Fertre
@ 2022-06-27 14:11 ` Philippe CORNU
-1 siblings, 0 replies; 6+ messages in thread
From: Philippe CORNU @ 2022-06-27 14:11 UTC (permalink / raw)
To: Yannick Fertre, Raphael Gallais-Pou, David Airlie, Daniel Vetter,
Maxime Coquelin, Alexandre Torgue, dri-devel, linux-stm32,
linux-arm-kernel, linux-kernel
On 6/3/22 15:43, Yannick Fertre wrote:
> Fix issues reported by checkpatch.pl:
> - Braces {} should be used on all arms
> - Blank lines
>
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> ---
> drivers/gpu/drm/stm/ltdc.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index a4098aaff243..6a9f613839b5 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -908,9 +908,9 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
> drm_connector_list_iter_end(&iter);
> }
>
> - if (bridge && bridge->timings)
> + if (bridge && bridge->timings) {
> bus_flags = bridge->timings->input_bus_flags;
> - else if (connector) {
> + } else if (connector) {
> bus_flags = connector->display_info.bus_flags;
> if (connector->display_info.num_bus_formats)
> bus_formats = connector->display_info.bus_formats[0];
> @@ -1917,7 +1917,6 @@ int ltdc_load(struct drm_device *ddev)
> DRM_ERROR("Failed to register LTDC interrupt\n");
> goto err;
> }
> -
> }
>
> crtc = devm_kzalloc(dev, sizeof(*crtc), GFP_KERNEL);
Dear Yannick,
Many thanks for your patch,
Applied on drm-misc-next.
Have a good day
Philippe :-)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/stm: ltdc: fix various coding-style warnings
@ 2022-06-27 14:11 ` Philippe CORNU
0 siblings, 0 replies; 6+ messages in thread
From: Philippe CORNU @ 2022-06-27 14:11 UTC (permalink / raw)
To: Yannick Fertre, Raphael Gallais-Pou, David Airlie, Daniel Vetter,
Maxime Coquelin, Alexandre Torgue, dri-devel, linux-stm32,
linux-arm-kernel, linux-kernel
On 6/3/22 15:43, Yannick Fertre wrote:
> Fix issues reported by checkpatch.pl:
> - Braces {} should be used on all arms
> - Blank lines
>
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> ---
> drivers/gpu/drm/stm/ltdc.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index a4098aaff243..6a9f613839b5 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -908,9 +908,9 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
> drm_connector_list_iter_end(&iter);
> }
>
> - if (bridge && bridge->timings)
> + if (bridge && bridge->timings) {
> bus_flags = bridge->timings->input_bus_flags;
> - else if (connector) {
> + } else if (connector) {
> bus_flags = connector->display_info.bus_flags;
> if (connector->display_info.num_bus_formats)
> bus_formats = connector->display_info.bus_formats[0];
> @@ -1917,7 +1917,6 @@ int ltdc_load(struct drm_device *ddev)
> DRM_ERROR("Failed to register LTDC interrupt\n");
> goto err;
> }
> -
> }
>
> crtc = devm_kzalloc(dev, sizeof(*crtc), GFP_KERNEL);
Dear Yannick,
Many thanks for your patch,
Applied on drm-misc-next.
Have a good day
Philippe :-)
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-06-27 14:13 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-03 13:43 [PATCH] drm/stm: ltdc: fix various coding-style warnings Yannick Fertre
2022-06-03 13:43 ` Yannick Fertre
2022-06-09 15:36 ` Raphael Gallais-Pou
2022-06-09 15:36 ` Raphael Gallais-Pou
2022-06-27 14:11 ` Philippe CORNU
2022-06-27 14:11 ` Philippe CORNU
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.