On Tue, 2016-08-02 at 13:32 -0700, Guenter Roeck wrote: > +static bool svdm_consume_svids(struct tcpm_port *port, const u32 > *payload, > + int cnt) > +{ > + struct pd_mode_data *pmdata = &port->mode_data; > + int i; > + > + for (i = 1; i < cnt; i++) { > + u16 svid; > + > + svid = (payload[i] >> 16) & 0xffff; > + if (!svid) > + return false; Hi, this looks like an endianness bug. Regards Oliver -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html