diff mbox

drm/stm: ltdc: fix duplicated arguments

Message ID 1495445421-20846-1-git-send-email-philippe.cornu@st.com (mailing list archive)
State New, archived
Headers show

Commit Message

Philippe CORNU May 22, 2017, 9:30 a.m. UTC
Fix COMPILE_TEST build issue detected with the
rule: "duplicated argument to & or |"

Signed-off-by: Philippe CORNU <philippe.cornu@st.com>
---
 drivers/gpu/drm/stm/ltdc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Benjamin Gaignard May 23, 2017, 1:44 p.m. UTC | #1
2017-05-22 11:30 GMT+02:00 Philippe CORNU <philippe.cornu@st.com>:
> Fix COMPILE_TEST build issue detected with the
> rule: "duplicated argument to & or |"
>

I have merged it in drm-misc-next

Thanks,
Benjamin

> Signed-off-by: Philippe CORNU <philippe.cornu@st.com>
> ---
>  drivers/gpu/drm/stm/ltdc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
> index a40418c..700cc08 100644
> --- a/drivers/gpu/drm/stm/ltdc.c
> +++ b/drivers/gpu/drm/stm/ltdc.c
> @@ -463,7 +463,7 @@ static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
>         clk_enable(ldev->pixel_clk);
>
>         /* Configures the HS, VS, DE and PC polarities. */
> -       val = HSPOL_AL | HSPOL_AL | DEPOL_AL | PCPOL_IPC;
> +       val = HSPOL_AL | VSPOL_AL | DEPOL_AL | PCPOL_IPC;
>
>         if (vm.flags & DISPLAY_FLAGS_HSYNC_HIGH)
>                 val |= HSPOL_AH;
> --
> 1.9.1
>
diff mbox

Patch

diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c
index a40418c..700cc08 100644
--- a/drivers/gpu/drm/stm/ltdc.c
+++ b/drivers/gpu/drm/stm/ltdc.c
@@ -463,7 +463,7 @@  static void ltdc_crtc_mode_set_nofb(struct drm_crtc *crtc)
 	clk_enable(ldev->pixel_clk);
 
 	/* Configures the HS, VS, DE and PC polarities. */
-	val = HSPOL_AL | HSPOL_AL | DEPOL_AL | PCPOL_IPC;
+	val = HSPOL_AL | VSPOL_AL | DEPOL_AL | PCPOL_IPC;
 
 	if (vm.flags & DISPLAY_FLAGS_HSYNC_HIGH)
 		val |= HSPOL_AH;