Re: [PATCH net-next v4 02/13] mac802154: allocation of ieee802154 device
From: David Miller <davem@davemloft.net>
Date: 2012-05-14 22:28:27
From: Alexander Smirnov <redacted> Date: Sat, 12 May 2012 17:59:14 +0400
Add interface for device drivers to allocate and register ieee802154 compatible device. This code is based on net/mac80211/main.c file Signed-off-by: Alexander Smirnov <redacted>
Sorry, this needs to be fixed up. I can almost understand using a comment formatting style I disagree with, if you do so consistently. But formatting them in THREE completely different ways? Give me a break:
+ /* As in mac80211 slaves list is modified: + * 1) under the RTNL + * 2) protected by slaves_mtx; + * 3) in an RCU manner + * + * So atomic readers can use any of this protection methods + */
...
+ /* This one is used for scanning and other + * jobs not to be interfered with serial driver */
...
+ /* + * The following flag is also modified under slaves_mtx and RTNL, + * so you can read them using any of protection methods. + */ + /* + * SoftMAC device is registered and running. One can add + * subinterfaces. + */
That's just beyond silly, and the last two comments are right next to eachother just combine them to one. Go over your entire patch set and make sure that all of your comments are of the form: /* Like * this. */