Hi Gustavo, > Use of "__" usually means we need to call the function with a lock held, > which is not the case here. > > Signed-off-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > --- > net/bluetooth/l2cap_core.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Acked-by: Marcel Holtmann <marcel@xxxxxxxxxxxx> > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c > index 1d773b9..9060eec 100644 > --- a/net/bluetooth/l2cap_core.c > +++ b/net/bluetooth/l2cap_core.c > @@ -3389,7 +3389,7 @@ static inline int l2cap_command_rej(struct l2cap_conn *conn, > return 0; > } > > -static void __l2cap_connect(struct l2cap_conn *conn, struct l2cap_cmd_hdr *cmd, > +static void l2cap_connect(struct l2cap_conn *conn, struct l2cap_cmd_hdr *cmd, > u8 *data, u8 rsp_code, u8 amp_id) Don't you need to fix the coding style for the indentation of the second line. 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