On Thu, Oct 25, 2018 at 10:27:20AM +0300, Heikki Krogerus wrote: > On Wed, Oct 24, 2018 at 08:07:09PM +0300, Sergei Shtylyov wrote: > > Hello! > > > > On 10/24/2018 06:05 PM, Heikki Krogerus wrote: > > > > > When the connections are defined in firmware, struct > > > device_connection will have the fwnode member pointing to > > > the device node (struct fwnode_handle) of the requested > > > device, and the endpoint will not be used at all in that > > > case. > > > > > > Signed-off-by: Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxx> > > > --- > > > drivers/usb/typec/class.c | 19 ++++++++++++++++--- > > > 1 file changed, 16 insertions(+), 3 deletions(-) > > > > > > diff --git a/drivers/usb/typec/class.c b/drivers/usb/typec/class.c > > > index 5db0593ca0bd..fe6f3a932a88 100644 > > > --- a/drivers/usb/typec/class.c > > > +++ b/drivers/usb/typec/class.c > > > @@ -204,15 +204,28 @@ static void typec_altmode_put_partner(struct altmode *altmode) > > > put_device(&adev->dev); > > > } > > > > > > -static int __typec_port_match(struct device *dev, const void *name) > > > +static int typec_port_fwnode_match(struct device *dev, const void *fwnode) > > > +{ > > > + return dev_name(fwnode) == fwnode; > > > > dev_name(dev), maybe? Actually: dev_fwnode(dev) == fwnode; Cheers, -- heikki