Hi Andrei, at least put the sparse warning in the commit message. > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > --- > net/bluetooth/mgmt.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c > index 06b5df3..5efca93 100644 > --- a/net/bluetooth/mgmt.c > +++ b/net/bluetooth/mgmt.c > @@ -287,7 +287,7 @@ static int read_commands(struct sock *sk, struct hci_dev *hdev, void *data, > struct mgmt_rp_read_commands *rp; > u16 num_commands = ARRAY_SIZE(mgmt_commands); > u16 num_events = ARRAY_SIZE(mgmt_events); > - u16 *opcode; > + __le16 *opcode; > size_t rp_size; > int i, err; > > @@ -2601,7 +2601,7 @@ static int load_long_term_keys(struct sock *sk, struct hci_dev *hdev, > return 0; > } > > -struct mgmt_handler { > +static struct mgmt_handler { > int (*func) (struct sock *sk, struct hci_dev *hdev, void *data, > u16 data_len); > bool var_len; And this should be static const. 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