staging: ks7010: clean up SDIO header comments

SDIO header file does not use kernel doc format struct
comments. Adding them aids readability and enables documentation to be
built from the source code. Other comments may be tidied up as we do this.

Add kernel format struct comments. Tidy up comments.

Signed-off-by: Tobin C. Harding <me@tobin.cc>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/ks7010/ks7010_sdio.h b/drivers/staging/ks7010/ks7010_sdio.h
index 5aa593af82ddf2e14d5e0c70bc47c2adca31156a..d04fccc606b68ba7f7b82f945ff94241a46a2284 100644 (file)
--- a/drivers/staging/ks7010/ks7010_sdio.h
+++ b/drivers/staging/ks7010/ks7010_sdio.h
@@ -76,8 +76,6 @@ enum gen_com_reg_b {
 };
 
 /* Wakeup Register */
-/* #define WAKEUP                      0x008104 */
-/* #define WAKEUP_REQ          0x00 */
 #define WAKEUP                 0x008018
 #define WAKEUP_REQ             0x5a
 
@@ -87,9 +85,6 @@ enum gen_com_reg_b {
 
 #define KS7010_IRAM_ADDRESS    0x06000000
 
-/*
- * struct define
- */
 struct hw_info_t {
        struct ks_sdio_card *sdio_card;
        struct workqueue_struct *ks7010sdio_wq;
@@ -105,35 +100,62 @@ struct ks_sdio_card {
 /* Tx Device struct */
 #define        TX_DEVICE_BUFF_SIZE     1024
 
+/**
+ * struct tx_device_buffer - Queue item for the tx queue.
+ * @sendp: Pointer to the send request data.
+ * @size: Size of @sendp data.
+ * @complete_handler: Function called once data write to device is complete.
+ * @arg1: First argument to @complete_handler.
+ * @arg2: Second argument to @complete_handler.
+ */
 struct tx_device_buffer {
-       unsigned char *sendp;   /* pointer of send req data */
+       unsigned char *sendp;
        unsigned int size;
        void (*complete_handler)(struct ks_wlan_private *priv,
                                 struct sk_buff *skb);
        struct sk_buff *skb;
 };
 
+/**
+ * struct tx_device - Tx buffer queue.
+ * @tx_device_buffer: Queue buffer.
+ * @qhead: Head of tx queue.
+ * @qtail: Tail of tx queue.
+ * @tx_dev_lock: Queue lock.
+ */
 struct tx_device {
        struct tx_device_buffer tx_dev_buff[TX_DEVICE_BUFF_SIZE];
-       unsigned int qhead;     /* tx buffer queue first pointer */
-       unsigned int qtail;     /* tx buffer queue last pointer */
-       spinlock_t tx_dev_lock;
+       unsigned int qhead;
+       unsigned int qtail;
+       spinlock_t tx_dev_lock; /* protect access to the queue */
 };
 
 /* Rx Device struct */
 #define        RX_DATA_SIZE    (2 + 2 + 2347 + 1)
 #define        RX_DEVICE_BUFF_SIZE     32
 
+/**
+ * struct rx_device_buffer - Queue item for the rx queue.
+ * @data: rx data.
+ * @size: Size of @data.
+ */
 struct rx_device_buffer {
        unsigned char data[RX_DATA_SIZE];
        unsigned int size;
 };
 
+/**
+ * struct rx_device - Rx buffer queue.
+ * @rx_device_buffer: Queue buffer.
+ * @qhead: Head of rx queue.
+ * @qtail: Tail of rx queue.
+ * @rx_dev_lock: Queue lock.
+ */
 struct rx_device {
        struct rx_device_buffer rx_dev_buff[RX_DEVICE_BUFF_SIZE];
-       unsigned int qhead;     /* rx buffer queue first pointer */
-       unsigned int qtail;     /* rx buffer queue last pointer */
-       spinlock_t rx_dev_lock;
+       unsigned int qhead;
+       unsigned int qtail;
+       spinlock_t rx_dev_lock; /* protect access to the queue */
 };
 
 #define        ROM_FILE "ks7010sd.rom"