Commit e7d374e7 authored by Ben Chan's avatar Ben Chan Committed by Greg Kroah-Hartman

staging: gdm72xx: fix block comment style

This patch fixes the following checkpatch warnings, which are issued
when the gdm72xx driver is moved out of staging into drivers/net/wimax:

  WARNING: networking block comments don't use an empty /* line, use /* Comment...
  WARNING: networking block comments start with * on subsequent lines
  WARNING: networking block comments put the trailing */ on a separate line
Signed-off-by: default avatarBen Chan <benchan@chromium.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a29b1857
...@@ -439,9 +439,7 @@ static int gdm_sdio_send(void *priv_dev, void *data, int len, ...@@ -439,9 +439,7 @@ static int gdm_sdio_send(void *priv_dev, void *data, int len,
return 0; return 0;
} }
/* /* Handle the HCI, WIMAX_SDU_TX_FLOW. */
* Handle the HCI, WIMAX_SDU_TX_FLOW.
*/
static int control_sdu_tx_flow(struct sdiowm_dev *sdev, u8 *hci_data, int len) static int control_sdu_tx_flow(struct sdiowm_dev *sdev, u8 *hci_data, int len)
{ {
struct tx_cxt *tx = &sdev->tx; struct tx_cxt *tx = &sdev->tx;
...@@ -462,8 +460,7 @@ static int control_sdu_tx_flow(struct sdiowm_dev *sdev, u8 *hci_data, int len) ...@@ -462,8 +460,7 @@ static int control_sdu_tx_flow(struct sdiowm_dev *sdev, u8 *hci_data, int len)
tx->stop_sdu_tx = 0; tx->stop_sdu_tx = 0;
if (tx->can_send) if (tx->can_send)
schedule_work(&sdev->ws); schedule_work(&sdev->ws);
/* /* If free buffer for sdu tx doesn't exist, then tx queue
* If free buffer for sdu tx doesn't exist, then tx queue
* should not be woken. For this reason, don't pass the command, * should not be woken. For this reason, don't pass the command,
* START_SDU_TX. * START_SDU_TX.
*/ */
......
...@@ -338,8 +338,7 @@ static int gdm_usb_send(void *priv_dev, void *data, int len, ...@@ -338,8 +338,7 @@ static int gdm_usb_send(void *priv_dev, void *data, int len,
t->callback = cb; t->callback = cb;
t->cb_data = cb_data; t->cb_data = cb_data;
/* /* In some cases, USB Module of WiMax is blocked when data size is
* In some cases, USB Module of WiMax is blocked when data size is
* the multiple of 512. So, increment length by one in that case. * the multiple of 512. So, increment length by one in that case.
*/ */
if ((len % 512) == 0) if ((len % 512) == 0)
...@@ -439,8 +438,7 @@ static void gdm_usb_rcv_complete(struct urb *urb) ...@@ -439,8 +438,7 @@ static void gdm_usb_rcv_complete(struct urb *urb)
list_for_each_entry(t, &tx->sdu_list, list) { list_for_each_entry(t, &tx->sdu_list, list) {
usb_submit_urb(t->urb, GFP_ATOMIC); usb_submit_urb(t->urb, GFP_ATOMIC);
} }
/* /* If free buffer for sdu tx doesn't
* If free buffer for sdu tx doesn't
* exist, then tx queue should not be * exist, then tx queue should not be
* woken. For this reason, don't pass * woken. For this reason, don't pass
* the command, START_SDU_TX. * the command, START_SDU_TX.
......
...@@ -581,8 +581,9 @@ static int gdm_wimax_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) ...@@ -581,8 +581,9 @@ static int gdm_wimax_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
return ret; return ret;
} else if (req->cmd == SIOCS_DATA) { } else if (req->cmd == SIOCS_DATA) {
if (req->data_id == SIOC_DATA_FSM) { if (req->data_id == SIOC_DATA_FSM) {
/*NOTE: gdm_update_fsm should be called /* NOTE: gdm_update_fsm should be called
before gdm_wimax_ioctl_set_data is called*/ * before gdm_wimax_ioctl_set_data is called.
*/
gdm_update_fsm(dev, gdm_update_fsm(dev,
(struct fsm_s *)req->data.buf); (struct fsm_s *)req->data.buf);
} }
......
...@@ -112,13 +112,11 @@ ...@@ -112,13 +112,11 @@
#define W_SCAN_ALL_SUBSCRIPTION 1 #define W_SCAN_ALL_SUBSCRIPTION 1
#define W_SCAN_SPECIFIED_SUBSCRIPTION 2 #define W_SCAN_SPECIFIED_SUBSCRIPTION 2
/* /* TLV
* TLV
* *
* [31:31] indicates the type is composite. * [31:31] indicates the type is composite.
* [30:16] is the length of the type. 0 length means length is variable. * [30:16] is the length of the type. 0 length means length is variable.
* [15:0] is the actual type. * [15:0] is the actual type.
*
*/ */
#define TLV_L(x) (((x) >> 16) & 0xff) #define TLV_L(x) (((x) >> 16) & 0xff)
#define TLV_T(x) ((x) & 0xff) #define TLV_T(x) ((x) & 0xff)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment