[PATCH 09/36] xfree86: add autoAddGPU option

Keith Packard keithp at keithp.com
Mon Jul 2 11:40:19 PDT 2012


Dave Airlie <airlied at gmail.com> writes:

> +    /* if autoaddgpu devices is enabled then go find a few more and add them as GPU screens */
> +    if (xf86Info.autoAddGPU == FALSE)
> +        return foundScreen;
> +
> +    if (numDevs) {
> +        for (j = 0; j < xf86_num_platform_devices; j++) {
> +            probeSingleDevice(&xf86_platform_devices[j], drvp, devList[0], PLATFORM_PROBE_GPU_SCREEN);
> +        }
> +    }

Bikeshed -- seems like this would be more readable as:


	if (xf86Info.autoAddGPU) {
		if (numDevs) {
			for (j = 0; j < xf86_num_platform_devices; j++) {
				probeSingleDevice(&xf86_platform_devices[j], drvp, devList[0], PLATFORM_PROBE_GPU_SCREEN);
			}
		}
	}

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120702/111661e8/attachment.pgp>


More information about the xorg-devel mailing list