Commit 4ffab688 authored by Edgardo Hames's avatar Edgardo Hames Committed by Greg Kroah-Hartman

Staging: wlan-ng: rework code style after feedback

This patch includes the feedback received from Richard Kennedy.
Signed-off-by: default avatarEdgardo Hames <ehames@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 5813b624
...@@ -2612,19 +2612,18 @@ int hfa384x_drvr_start(hfa384x_t *hw) ...@@ -2612,19 +2612,18 @@ int hfa384x_drvr_start(hfa384x_t *hw)
if (result1 != 0) { if (result1 != 0) {
if (result2 != 0) { if (result2 != 0) {
printk(KERN_ERR printk(KERN_ERR
"cmd_initialize() failed on two attempts," "cmd_initialize() failed on two attempts, results %d and %d\n",
" results %d and %d\n", result1, result2); result1, result2);
usb_kill_urb(&hw->rx_urb); usb_kill_urb(&hw->rx_urb);
goto done; goto done;
} else { } else {
pr_debug("First cmd_initialize() failed (result %d),\n", pr_debug("First cmd_initialize() failed (result %d),\n",
result1); result1);
pr_debug("but second attempt succeeded." pr_debug("but second attempt succeeded. All should be ok\n");
" All should be ok\n");
} }
} else if (result2 != 0) { } else if (result2 != 0) {
printk(KERN_WARNING "First cmd_initialize() succeeded," printk(KERN_WARNING "First cmd_initialize() succeeded, but second attempt failed (result=%d)\n",
" but second attempt failed (result=%d)\n", result2); result2);
printk(KERN_WARNING printk(KERN_WARNING
"Most likely the card will be functional\n"); "Most likely the card will be functional\n");
goto done; goto done;
...@@ -3382,9 +3381,7 @@ static void hfa384x_usbin_ctlx(hfa384x_t *hw, hfa384x_usbin_t *usbin, ...@@ -3382,9 +3381,7 @@ static void hfa384x_usbin_ctlx(hfa384x_t *hw, hfa384x_usbin_t *usbin,
* our request has been acknowledged. Odd, * our request has been acknowledged. Odd,
* but our OUT URB is still alive... * but our OUT URB is still alive...
*/ */
pr_debug("Causality violation: " pr_debug("Causality violation: please reboot Universe\n");
"please reboot Universe, or email "
"linux-wlan-devel@lists.linux-wlan.com\n");
ctlx->state = CTLX_RESP_COMPLETE; ctlx->state = CTLX_RESP_COMPLETE;
break; break;
...@@ -3848,8 +3845,8 @@ static void hfa384x_ctlxout_callback(struct urb *urb) ...@@ -3848,8 +3845,8 @@ static void hfa384x_ctlxout_callback(struct urb *urb)
default: default:
/* This is NOT a valid CTLX "success" state! */ /* This is NOT a valid CTLX "success" state! */
printk(KERN_ERR "Illegal CTLX[%d]" printk(KERN_ERR
" success state(%s, %d) in OUT URB\n", "Illegal CTLX[%d] success state(%s, %d) in OUT URB\n",
le16_to_cpu(ctlx->outbuf.type), le16_to_cpu(ctlx->outbuf.type),
ctlxstr(ctlx->state), urb->status); ctlxstr(ctlx->state), urb->status);
break; break;
......
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