Staging: batman-adv: checkpatch cleanup of comments
authorSven Eckelmann <sven.eckelmann@gmx.de>
Sat, 18 Sep 2010 19:01:11 +0000 (21:01 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 20 Sep 2010 23:29:47 +0000 (16:29 -0700)
checkpatch now detects the start of a comment and warns about usage of
multiple spaces at the beginning of a line. We have to replace the '   '
in multiple lines comments by ' * ' to fix it.

Checkpatch also wants a comment after a definition of a spinlock_t which
describes what it protects. It is currently not possible to add it
before the actual struct which includes the spinlock.

Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/batman-adv/packet.h
drivers/staging/batman-adv/types.h
drivers/staging/batman-adv/vis.c

index 44de1bfb0b0c4dcb89f59d9951d42f367f728992..2693383889a43875ab118d69b53d353a0f80e9e1 100644 (file)
@@ -79,7 +79,7 @@ struct icmp_packet {
 #define BAT_RR_LEN 16
 
 /* icmp_packet_rr must start with all fields from imcp_packet
  as this is assumed by code that handles ICMP packets */
* as this is assumed by code that handles ICMP packets */
 struct icmp_packet_rr {
        uint8_t  packet_type;
        uint8_t  version;  /* batman version field */
index 9d744d86402869ce66b007f6c0a36a2b0bf9813d..478277f6ad08b4c6d6dc36f7c8a115535e4d9030 100644 (file)
@@ -109,6 +109,7 @@ struct neigh_node {
        struct batman_if *if_incoming;
 };
 
+
 struct bat_priv {
        atomic_t mesh_state;
        struct net_device_stats stats;
@@ -133,13 +134,13 @@ struct bat_priv {
        struct hashtable_t *hna_local_hash;
        struct hashtable_t *hna_global_hash;
        struct hashtable_t *vis_hash;
-       spinlock_t orig_hash_lock;
-       spinlock_t forw_bat_list_lock;
-       spinlock_t forw_bcast_list_lock;
-       spinlock_t hna_lhash_lock;
-       spinlock_t hna_ghash_lock;
-       spinlock_t vis_hash_lock;
-       spinlock_t vis_list_lock;
+       spinlock_t orig_hash_lock; /* protects orig_hash */
+       spinlock_t forw_bat_list_lock; /* protects forw_bat_list */
+       spinlock_t forw_bcast_list_lock; /* protects  */
+       spinlock_t hna_lhash_lock; /* protects hna_local_hash */
+       spinlock_t hna_ghash_lock; /* protects hna_global_hash */
+       spinlock_t vis_hash_lock; /* protects vis_hash */
+       spinlock_t vis_list_lock; /* protects vis_info::recv_list */
        int16_t num_local_hna;
        atomic_t hna_local_changed;
        struct delayed_work hna_work;
@@ -152,7 +153,7 @@ struct socket_client {
        struct list_head queue_list;
        unsigned int queue_len;
        unsigned char index;
-       spinlock_t lock;
+       spinlock_t lock; /* protects queue_list, queue_len, index */
        wait_queue_head_t queue_wait;
        struct bat_priv *bat_priv;
 };
@@ -204,7 +205,7 @@ struct debug_log {
        char log_buff[LOG_BUF_LEN];
        unsigned long log_start;
        unsigned long log_end;
-       spinlock_t lock;
+       spinlock_t lock; /* protects log_buff, log_start and log_end */
        wait_queue_head_t queue_wait;
 };
 
index b2cec8e289693441e5a9354ec1754d52cb4296c8..6b102a3f4edc4fd838a91f65dd0a7ba3ec61e293 100644 (file)
 #define smallest_signed_int(x) (1u << (7u + 8u * (sizeof(x) - 1u)))
 
 /* Checks if a sequence number x is a predecessor/successor of y.
  they handle overflows/underflows and can correctly check for a
  predecessor/successor unless the variable sequence number has grown by
  more then 2**(bitwidth(x)-1)-1.
  This means that for a uint8_t with the maximum value 255, it would think:
   * when adding nothing - it is neither a predecessor nor a successor
   * before adding more than 127 to the starting value - it is a predecessor,
   * when adding 128 - it is neither a predecessor nor a successor,
   * after adding more than 127 to the starting value - it is a successor */
* they handle overflows/underflows and can correctly check for a
* predecessor/successor unless the variable sequence number has grown by
* more then 2**(bitwidth(x)-1)-1.
* This means that for a uint8_t with the maximum value 255, it would think:
*  - when adding nothing - it is neither a predecessor nor a successor
*  - before adding more than 127 to the starting value - it is a predecessor,
*  - when adding 128 - it is neither a predecessor nor a successor,
*  - after adding more than 127 to the starting value - it is a successor */
 #define seq_before(x, y) ({typeof(x) _dummy = (x - y); \
                        _dummy > smallest_signed_int(_dummy); })
 #define seq_after(x, y) seq_before(y, x)