[PATCH] Add check for link from output to crtc before optimizing out a CrtcSet call
Keith Packard
keithp at keithp.com
Sat Feb 8 01:16:57 CET 2014
David Sodman <dsodman at chromium.org> writes:
> The function RRCrtcSet call checks to see if the config being set is
> already configured, but, doesn't check that the selected outputs are
> connected to the crtc before skipping. This means that the following
> sequence will omit the final CrtcSet call to the driver:
>
> CRTC c1 connect to output o
> CRTC c2 connect to output o
> CRTC c1 connect to output o
>
> This change adds the check to ensure that each of the calls are made to
> the driver.
>
> Signed-off-by: David Sodman <dsodman at chromium.org>
> Reviewed-by: Stephane Marchesin <marcheu at chromium.org>
This generates a warning:
CC rrcrtc.lo
rrcrtc.c: In function 'RRCrtcSet':
rrcrtc.c:618:17: warning: declaration of 'o' shadows a previous local [-Wshadow]
int o;
^
rrcrtc.c:526:9: warning: shadowed declaration is here [-Wshadow]
int o;
^
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140207/431328ec/attachment.pgp>
More information about the xorg-devel
mailing list