Re: [PATCH 2/2] udevng: Fix detection of USB attached tty devices

[Denis Kenzior <denkenz@gmail.com>]

Hi Ivo,

>> -
>> +    /* Handle USB-connected devices in check_usb_device */
>> +    if (l_streq0(bus, "usb")) {
>> +        check_usb_device(device);
>>           return;
>>       }
>> -    if (l_streq0(subsystem, "usb") || l_streq0(subsystem, "usbmisc"))
> 
> Is it possible (in theory) that we have 'usb' subsystem device that is not on 
> USB bus? If yes, then you shouldn't drop l_streq0(subsystem, "usb") check.

I hope not.  What i've seen is subsystem 'usbmisc' devices not setting ID_BUS to 
'usb', which is a bit weird.  Hence the 'usbmisc' check below.

I think we do not care about usb subsystem devices, only the leafs:
	- usbmisc (cdc-acm)
	- tty (ttyUSB...)
	- net

Regards,
-Denis