Re: [PATCH 2/3] staging: wlags49_h2: wl_if.h: fixes several coding style issues

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



100%, will do.

On 19 November 2012 09:17, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> On Fri, Nov 16, 2012 at 06:54:43PM +0200, Johan Meiring wrote:
>> This commit sorts out several coding style issues, including the use of C99 // comments, spaces at the start of lines, spaces before tabs, incorrect brace placement, and macros with complex values that have not been wrapped in parentheses.  The only remaining issue is the use of typedefs in two instances, but that's something that the original author might need to have a look at.
>>
>
> Could you line wrap your commit message at something like 72-73
> characters?  I think that's they rule on email, but I forget.
>
> Normally we ask that people break these kinds of changes into
> multiple patches.
> [patch 1/x] Staging: wlags49_h2: wl_if.h: use tab indents
> [patch 2/x] Staging: wlags49_h2: wl_if.h: fix comment style
> etc...
>
>> +/*
>> +#define HCF_MIN_COMM_QUALITY        0
>> +#define HCF_MAX_COMM_QUALITY        92
>> +#define HCF_MIN_SIGNAL_LEVEL        47
>> +#define HCF_MAX_SIGNAL_LEVEL        138
>> +#define HCF_MIN_NOISE_LEVEL         47
>> +#define HCF_MAX_NOISE_LEVEL         138
>> +#define HCF_0DBM_OFFSET             149
>> +*/
>
> Just delete this dead code.
>
>> +#define SIOCSIWNETNAME              (SIOCDEVPRIVATE+1)
>> +#define SIOCGIWNETNAME              (SIOCDEVPRIVATE+2)
>> +#define SIOCSIWSTANAME              (SIOCDEVPRIVATE+3)
>> +#define SIOCGIWSTANAME              (SIOCDEVPRIVATE+4)
>> +#define SIOCSIWPORTTYPE             (SIOCDEVPRIVATE+5)
>> +#define SIOCGIWPORTTYPE             (SIOCDEVPRIVATE+6)
>
> Could you put spaces around the '+' operator?
> #define SIOCSIWNETNAME          (SIOCDEVPRIVATE + 1)
>
> regards,
> dan carpenter
>
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux