[PATCH 01/14] xfree86: use udev to provide device enumeration for kms devices (v9)

Dave Airlie airlied at gmail.com
Mon Jun 25 11:17:55 PDT 2012


On Mon, Jun 25, 2012 at 4:56 PM, Michal Srb <msrb at suse.com> wrote:
> On Thursday 14 of June 2012 15:43:35 Dave Airlie wrote:
>> +int
>> +xf86platformProbeDev(DriverPtr drvp)
>> +{
>> +    Bool foundScreen = FALSE;
>> +    GDevPtr *devList;
>> +    const unsigned numDevs = xf86MatchDevice(drvp->driverName, &devList);
>> +    int i, j;
>> +
>> +    /* find the main device or any device specificed in xorg.conf */
>> +    for (i = 0; i < numDevs; i++) {
>> +        for (j = 0; j < xf86_num_platform_devices; j++) {
>> +            if (devList[i]->busID && *devList[i]->busID) {
>> +                if (xf86PlatformDeviceCheckBusID(&xf86_platform_devices[j], devList[i]->busID))
>> +                    break;
>> +            }
>> +            else {
>> +                if (xf86_platform_devices[j].pdev) {
>> +                    if (xf86IsPrimaryPlatform(&xf86_platform_devices[j]))
>> +                        break;
>> +                }
>> +            }
>> +        }
>> +
>> +        if (j == xf86_num_platform_devices)
>> +             continue;
>> +
>> +        foundScreen = probeSingleDevice(&xf86_platform_devices[j], drvp, devList[i], 0);
>> +        if (!foundScreen)
>> +            continue;
>
> Not sure if it is still relevant, but what purpose has this continue?
> Preparation for more code to be added bellow or was it supposed to be break?

Oh it was left over from before I made probeSingleDevice and I did bad
things in the loop,
so yeah I think I can drop that bit as well.

Thanks for noticing, will fix tomorrow.

Dave.


More information about the xorg-devel mailing list