Messages in this thread | | | Date | Sat, 13 Mar 2010 17:40:21 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH] staging: winbond: mds.c whitspace, indentation etc. |
| |
First of all, thanks for the patch. I verified that the compiled version was the same before and after. It cleans up the code a lot.
> + } else { /*DSSS*/ > + /*CTS duration > + *2 SIFS + DATA transmit time + 1 ACK > + * Rate : ?? Mega bps > + *ACK frame length = 14 bytes > + */
There are a couple places like this. Could you put spaces between the '*' and the first character? In fact those two comments could be combined probably.
Once you fix that, check everything again and resend. Looks good.
regards, dan carpenter
| |