Hi Mikel, > The involved values are all unsigned and thus unsigned int should be > used instead of signed int. Assigning ~0 to a signed int results in -1, > which is confusing and error-prone, while the code is trying to set the > maximum value possible. > > The code still works because the C standard defines that unsigned > comparison will be performed in these cases, when comparing an unsigned > int and a signed int. > > Signed-off-by: Mikel Astiz <mikel.astiz.oss@xxxxxxxxx> > --- > net/bluetooth/hci_core.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) Acked-by: Marcel Holtmann <marcel@xxxxxxxxxxxx> Regards Marcel -- 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