Hi Lizardo, On Thu, Dec 05, 2013, Anderson Lizardo wrote: > On Thu, Dec 5, 2013 at 9:11 AM, <johan.hedberg@xxxxxxxxx> wrote: > > +static bool is_valid_psm(u16 psm, u8 dst_type) > > +{ > > + if (!psm) > > + return false; > > + > > + if (bdaddr_type_is_le(dst_type)) > > + return (psm < 0x00ff); > > Shouldn't be "psm <= 0x00ff" here? Yes, it should. I just sent two fix-up patches based on your feedback. Thanks. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html