> > +static inline u32 map_swqe_size(u8 swqe_enc_size)
> > +{
> > +   return 128 << swqe_enc_size;
> > +}                ^
> > +                 |
> > +static inline u32|map_rwqe_size(u8 rwqe_enc_size)
> > +{                |
> > +   return 128 << rwqe_enc_size;
                      ^
> > +}                |
>                     |
> Snap!  These are ide|tical...
                      |
No, they aren't. -----+

> Name the function after what it does, not after the functions you expect
> to call it.

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to