linux-kernel - Re: [PATCH 7/7] drm/fsl-dcu: use mode flags for hsync/vsync pixelclk polarity

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160203140438.GB9650@ulmo>
Date:	Wed, 3 Feb 2016 15:04:38 +0100
From:	Thierry Reding <thierry.reding@...il.com>
To:	Stefan Agner <stefan@...er.ch>
Cc:	airlied@...il.com, alison.wang@...escale.com,
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
	airlied@...ux.ie, daniel.vetter@...ll.ch,
	jianwei.wang.chn@...il.com, Shawn Guo <shawnguo@...nel.org>
Subject: Re: [PATCH 7/7] drm/fsl-dcu: use mode flags for hsync/vsync pixelclk
 polarity

On Wed, Jan 27, 2016 at 06:46:50PM -0800, Stefan Agner wrote:
[...]
> On 2015-11-18 18:42, Stefan Agner wrote:
[...]
> > diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c
[...]
> > @@ -89,6 +91,15 @@ static void fsl_dcu_drm_crtc_mode_set_nofb(struct
> > drm_crtc *crtc)
> >  	vfp = mode->vsync_start - mode->vdisplay;
> >  	vsw = mode->vsync_end - mode->vsync_start;
> >  
> > +	if (!(mode->flags & DISPLAY_FLAGS_PIXDATA_POSEDGE))
> > +		pol |= DCU_SYN_POL_INV_PXCK_FALL;
> > +
> > +	if (mode->flags & DRM_MODE_FLAG_NHSYNC)
> > +		pol |= DCU_SYN_POL_INV_HS_LOW;
> > +
> > +	if (mode->flags & DRM_MODE_FLAG_NHSYNC)
> > +		pol |= DCU_SYN_POL_INV_VS_LOW;

I suspect that you want to check for DRM_MODE_FLAG_NVSYNC here instead.

> > diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > index 6413ac9..2a724f3 100644
> > --- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > @@ -47,8 +47,8 @@
> >  #define DCU_VSYN_PARA_FP(x)		(x)
> >  
> >  #define DCU_SYN_POL			0x0024
> > -#define DCU_SYN_POL_INV_PXCK_FALL	(0 << 6)
> > -#define DCU_SYN_POL_NEG_REMAIN		(0 << 5)
> > +#define DCU_SYN_POL_INV_PXCK_FALL	BIT(6)
> > +#define DCU_SYN_POL_NEG_REMAIN		BIT(5)

I don't understand these changes. You're in fact changing the values for
these defines, but it's not mentioned in the commit message. It also
seems like it should be a separate patch because it isn't related to the
mode flags changes in the remainder of the patch.

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)