[PATCH 09/36] xfree86: add platform bus hotplug support (v3)

Keith Packard keithp at keithp.com
Thu Jul 5 13:32:46 PDT 2012


Dave Airlie <airlied at gmail.com> writes:

> +
> +    for (layout = xf86ConfigLayout.screens; layout->screen != NULL;
> +         layout++) {
> +        xf86GPUScreens[i]->confScreen = layout->screen;
> +        break;
> +    }

This loop is insane.

Otherwise, this is all

Reviewed-by: Keith Packard <keithp at keithp.com>

-- 
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/20120705/00658c44/attachment.pgp>


More information about the xorg-devel mailing list