Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964877AbdGTJ1r (ORCPT ); Thu, 20 Jul 2017 05:27:47 -0400 Received: from mail-qk0-f171.google.com ([209.85.220.171]:38252 "EHLO mail-qk0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935170AbdGTJ1o (ORCPT ); Thu, 20 Jul 2017 05:27:44 -0400 MIME-Version: 1.0 In-Reply-To: <1500373219-20233-3-git-send-email-philippe.cornu@st.com> References: <1500373219-20233-1-git-send-email-philippe.cornu@st.com> <1500373219-20233-3-git-send-email-philippe.cornu@st.com> From: Benjamin Gaignard Date: Thu, 20 Jul 2017 11:27:42 +0200 Message-ID: Subject: Re: [PATCH v1 2/7] drm/stm: ltdc: Cleanup signal polarity defines To: Philippe CORNU Cc: Yannick Fertre , Vincent Abriou , David Airlie , "dri-devel@lists.freedesktop.org" , Linux Kernel Mailing List , Fabien Dessenne , Mickael Reulier , Gabriel Fernandez , Ludovic Barre , Alexandre Torgue , Maxime Coquelin Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3375 Lines: 76 2017-07-18 12:20 GMT+02:00 Philippe CORNU : > The GCR_PCPOL/DEPOL/VSPOL/HSPOL defines are sufficient to > describe the HS, VS, DE & PC signal polarities. > > Signed-off-by: Philippe CORNU Reviewed-by: Benjamin Gaignard > --- > drivers/gpu/drm/stm/ltdc.c | 28 ++++++++++------------------ > 1 file changed, 10 insertions(+), 18 deletions(-) > > diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c > index e46b427..50e8a89 100644 > --- a/drivers/gpu/drm/stm/ltdc.c > +++ b/drivers/gpu/drm/stm/ltdc.c > @@ -104,10 +104,10 @@ > > #define GCR_LTDCEN BIT(0) /* LTDC ENable */ > #define GCR_DEN BIT(16) /* Dither ENable */ > -#define GCR_PCPOL BIT(28) /* Pixel Clock POLarity */ > -#define GCR_DEPOL BIT(29) /* Data Enable POLarity */ > -#define GCR_VSPOL BIT(30) /* Vertical Synchro POLarity */ > -#define GCR_HSPOL BIT(31) /* Horizontal Synchro POLarity */ > +#define GCR_PCPOL BIT(28) /* Pixel Clock POLarity-Inverted */ > +#define GCR_DEPOL BIT(29) /* Data Enable POLarity-High */ > +#define GCR_VSPOL BIT(30) /* Vertical Synchro POLarity-High */ > +#define GCR_HSPOL BIT(31) /* Horizontal Synchro POLarity-High */ > > #define GC1R_WBCH GENMASK(3, 0) /* Width of Blue CHannel output */ > #define GC1R_WGCH GENMASK(7, 4) /* Width of Green Channel output */ > @@ -174,14 +174,6 @@ > > #define LXCFBLNR_CFBLN GENMASK(10, 0) /* Color Frame Buffer Line Number */ > > -#define HSPOL_AL 0 /* Horizontal Sync POLarity Active Low */ > -#define VSPOL_AL 0 /* Vertical Sync POLarity Active Low */ > -#define DEPOL_AL 0 /* Data Enable POLarity Active Low */ > -#define PCPOL_IPC 0 /* Input Pixel Clock */ > -#define HSPOL_AH GCR_HSPOL /* Horizontal Sync POLarity Active High */ > -#define VSPOL_AH GCR_VSPOL /* Vertical Sync POLarity Active High */ > -#define DEPOL_AH GCR_DEPOL /* Data Enable POLarity Active High */ > -#define PCPOL_IIPC GCR_PCPOL /* Inverted Input Pixel Clock */ > #define CONSTA_MAX 0xFF /* CONSTant Alpha MAX= 1.0 */ > #define BF1_PAXCA 0x600 /* Pixel Alpha x Constant Alpha */ > #define BF1_CA 0x400 /* Constant Alpha */ > @@ -459,20 +451,20 @@ 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 | VSPOL_AL | DEPOL_AL | PCPOL_IPC; > + /* Configures the HS, VS, DE and PC polarities. Default Active Low */ > + val = 0; > > if (vm.flags & DISPLAY_FLAGS_HSYNC_HIGH) > - val |= HSPOL_AH; > + val |= GCR_HSPOL; > > if (vm.flags & DISPLAY_FLAGS_VSYNC_HIGH) > - val |= VSPOL_AH; > + val |= GCR_VSPOL; > > if (vm.flags & DISPLAY_FLAGS_DE_HIGH) > - val |= DEPOL_AH; > + val |= GCR_DEPOL; > > if (vm.flags & DISPLAY_FLAGS_PIXDATA_NEGEDGE) > - val |= PCPOL_IIPC; > + val |= GCR_PCPOL; > > reg_update_bits(ldev->regs, LTDC_GCR, > GCR_HSPOL | GCR_VSPOL | GCR_DEPOL | GCR_PCPOL, val); > -- > 1.9.1 >