Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1692682
| From | Benjamin Gaignard <benjamin.gaignard@linaro.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v1 2/7] drm/stm: ltdc: Cleanup signal polarity defines |
| Date | 2017-07-20 11:30 +0200 |
| Message-ID | <u5fWp-5m0-1@gated-at.bofh.it> (permalink) |
| References | <u4xVo-1aY-25@gated-at.bofh.it> <u4xVp-1aY-57@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
2017-07-18 12:20 GMT+02:00 Philippe CORNU <philippe.cornu@st.com>: > The GCR_PCPOL/DEPOL/VSPOL/HSPOL defines are sufficient to > describe the HS, VS, DE & PC signal polarities. > > Signed-off-by: Philippe CORNU <philippe.cornu@st.com> Reviewed-by: Benjamin Gaignard <benjamin.gaignard@linaro.org> > --- > 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 >
Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread
[PATCH v1 0/7] drm/stm: Various cleanups Philippe CORNU <philippe.cornu@st.com> - 2017-07-18 12:30 +0200
[PATCH v1 7/7] drm/stm: dsi: Constify phy ops structure Philippe CORNU <philippe.cornu@st.com> - 2017-07-18 12:30 +0200
Re: [PATCH v1 7/7] drm/stm: dsi: Constify phy ops structure Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-07-20 11:40 +0200
[PATCH v1 3/7] drm/stm: ltdc: Lindent and minor cleanups Philippe CORNU <philippe.cornu@st.com> - 2017-07-18 12:30 +0200
Re: [PATCH v1 3/7] drm/stm: ltdc: Lindent and minor cleanups Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-07-20 11:30 +0200
[PATCH v1 1/7] drm/stm: drv: Rename platform driver name Philippe CORNU <philippe.cornu@st.com> - 2017-07-18 12:30 +0200
Re: [PATCH v1 1/7] drm/stm: drv: Rename platform driver name Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-07-20 11:30 +0200
[PATCH v1 2/7] drm/stm: ltdc: Cleanup signal polarity defines Philippe CORNU <philippe.cornu@st.com> - 2017-07-18 12:30 +0200
Re: [PATCH v1 2/7] drm/stm: ltdc: Cleanup signal polarity defines Benjamin Gaignard <benjamin.gaignard@linaro.org> - 2017-07-20 11:30 +0200
csiph-web