On Tue, May 23, 2017 at 10:47:58PM +0100, Kieran Bingham wrote: > On 23/05/17 22:43, Sakari Ailus wrote: > > On Wed, May 24, 2017 at 12:40:19AM +0300, Sakari Ailus wrote: > >>> * When all devices use endpoint matching, this code can be simplified, and the > >>> * parent comparisons can be removed. > > > > Oh, and this I'm not so sure about --- we'll need to match lens controllers > > and flash drivers. There are no endpoints in those devices. Let's see how it > > goes when we get there... > > > > Sure, would you like me to post a v4 of just this patch? Please. > > The parent checks should always be safe, so this feels like less of a workaround > now, and if it provides support for other use cases perhaps it could survive > longer term. Indeed. If you end up getting the second parent of a device node, the end result will be that you simply won't have a match. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx