On Wed, Sep 25, 2024 at 10:31:33AM -0700, Abhishek Pandit-Subedi wrote: > On Wed, Sep 25, 2024 at 9:54 AM Dmitry Baryshkov > <dmitry.baryshkov@xxxxxxxxxx> wrote: > > > > On Wed, Sep 25, 2024 at 09:25:03AM GMT, Abhishek Pandit-Subedi wrote: > > > altmode_match is used when searching for the first port altmode that > > > matches the partner or plug altmode. If the port registered with mode > > > set to TYPEC_ANY_MODE, it should always match if the SVID matches. > > > > > > Signed-off-by: Abhishek Pandit-Subedi <abhishekpandit@xxxxxxxxxxxx> > > > > Fixes? > > This is new for Thunderbolt which registers as TYPEC_ANY_MODE so > there's no FIXES. I think Heikki may need to chime in on how the > `mode` is supposed to be used. > > IMO, it may be appropriate to get rid of the mode check entirely. It's probable okay to just drop it. > > > > > --- > > > > > > drivers/usb/typec/class.c | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/usb/typec/class.c b/drivers/usb/typec/class.c > > > index 9262fcd4144f..179856503d5d 100644 > > > --- a/drivers/usb/typec/class.c > > > +++ b/drivers/usb/typec/class.c > > > @@ -230,7 +230,8 @@ static int altmode_match(struct device *dev, void *data) > > > if (!is_typec_altmode(dev)) > > > return 0; > > > > > > - return ((adev->svid == id->svid) && (adev->mode == id->mode)); > > > + return ((adev->svid == id->svid) && > > > + (adev->mode == id->mode || adev->mode == TYPEC_ANY_MODE)); > > > } > > > > > > static void typec_altmode_set_partner(struct altmode *altmode) > > > -- > > > 2.46.0.792.g87dc391469-goog > > > > > > > -- > > With best wishes > > Dmitry -- heikki
