[PATCH 5/6] xinput.sys: Create an internal PDO, on the XINPUT bus.

Zebediah Figura (she/her) zfigura at codeweavers.com
Mon Aug 30 11:23:52 CDT 2021


On 8/28/21 1:46 PM, Rémi Bernon wrote:
> On 8/27/21 10:55 PM, Rémi Bernon wrote:
>>>> -            if ((status = IoRegisterDeviceInterface(device,
>>>> &GUID_DEVINTERFACE_HID, NULL, &ext->u.pdo.link_name)))
>>>> +            if ((status = IoRegisterDeviceInterface(device,
>>>> ext->interface_guid, NULL, &ext->u.pdo.link_name)))
>>>>                {
>>>>                    ERR("Failed to register interface, status %#x.\n",
>>>> status);
>>>>                    break;
>>>
>>> Any chance we could put the hidclass hunks in a separate patch?
>>>
>>> ...
>>>
>>
>> That would cause the "internal" "xinput" PDO be momentarily listed in
>> the HID device interface class, as a duplicate device with the gamepad,
>> and although applications aren't supposed to see or use it. Not a big
>> deal, bug probably not very nice.
>>
> 
> I guess it could come first though.

Yeah, that's what I meant.

> BTW I used the "standard" XUSB interface class there, but now that I
> think of it and now that I'm moving everything to "winexinput" maybe
> it's not the best idea to do that.
> 
> Is there a general strategy on how to define Wine-specific GUIDs? Should
> I just go get some randomly generated GUID and that's it? Doesn't it
> have to have a specific marker to tell that it's some Wine-specific one?
> 

I don't think there is, no. Personally I like to use regular-looking 
numbers for Data1 so that it's clear it's not a Windows GUID and nobody 
needs to waste time looking it up (this is even more true in tests), but 
there's no requirement for it.



More information about the wine-devel mailing list