* Emeltchenko Andrei <Andrei.Emeltchenko.news@xxxxxxxxx> [2010-12-01 16:58:24 +0200]: > From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > Do not initialize static vars to zero, macros with complex values > shall be enclosed with (), remove unneeded braces. > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > --- > include/net/bluetooth/l2cap.h | 10 +++++----- > net/bluetooth/l2cap.c | 7 +++---- > 2 files changed, 8 insertions(+), 9 deletions(-) > > diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h > index c819c8b..217bb91 100644 > --- a/include/net/bluetooth/l2cap.h > +++ b/include/net/bluetooth/l2cap.h > @@ -417,11 +417,11 @@ static inline int l2cap_tx_window_full(struct sock *sk) > return sub == pi->remote_tx_win; > } > > -#define __get_txseq(ctrl) ((ctrl) & L2CAP_CTRL_TXSEQ) >> 1 > -#define __get_reqseq(ctrl) ((ctrl) & L2CAP_CTRL_REQSEQ) >> 8 > -#define __is_iframe(ctrl) !((ctrl) & L2CAP_CTRL_FRAME_TYPE) > -#define __is_sframe(ctrl) (ctrl) & L2CAP_CTRL_FRAME_TYPE > -#define __is_sar_start(ctrl) ((ctrl) & L2CAP_CTRL_SAR) == L2CAP_SDU_START > +#define __get_txseq(ctrl) (((ctrl) & L2CAP_CTRL_TXSEQ) >> 1) > +#define __get_reqseq(ctrl) (((ctrl) & L2CAP_CTRL_REQSEQ) >> 8) > +#define __is_iframe(ctrl) (!((ctrl) & L2CAP_CTRL_FRAME_TYPE)) > +#define __is_sframe(ctrl) ((ctrl) & L2CAP_CTRL_FRAME_TYPE) > +#define __is_sar_start(ctrl) (((ctrl) & L2CAP_CTRL_SAR) == L2CAP_SDU_START) > > void l2cap_load(void); > > diff --git a/net/bluetooth/l2cap.c b/net/bluetooth/l2cap.c > index 12b4aa2..d4b4fbd 100644 > --- a/net/bluetooth/l2cap.c > +++ b/net/bluetooth/l2cap.c > @@ -57,7 +57,7 @@ > > #define VERSION "2.15" > > -static int disable_ertm = 0; > +static int disable_ertm; Same issue here. -- Gustavo F. Padovan http://profusion.mobi -- 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
