* [PATCH] staging: wlan-ng: fix checkpatch warnings
@ 2015-05-15 13:13 chaehyun lim
2015-05-15 13:36 ` walter harms
0 siblings, 1 reply; 2+ messages in thread
From: chaehyun lim @ 2015-05-15 13:13 UTC (permalink / raw)
To: kernel-janitors
clean up checkpatch.pl in prism2sta.c
WARNING : line over 80 characters
Signed-off-by: chaehyun lim <chaehyun.lim@gmail.com>
---
drivers/staging/wlan-ng/prism2sta.c | 23 ++++++++++++++---------
1 file changed, 14 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c
index ddb294e..0329c52 100644
--- a/drivers/staging/wlan-ng/prism2sta.c
+++ b/drivers/staging/wlan-ng/prism2sta.c
@@ -428,7 +428,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
result = hfa384x_drvr_start(hw);
if (result) {
netdev_err(wlandev->netdev,
- "hfa384x_drvr_start() failed,result=%d\n", (int)result);
+ "hfa384x_drvr_start() failed,result=%d\n",
+ (int)result);
result P80211ENUM_resultcode_implementation_failure;
wlandev->msdstate = WLAN_MSD_HWPRESENT;
@@ -471,7 +472,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
result = hfa384x_drvr_start(hw);
if (result) {
netdev_err(wlandev->netdev,
- "hfa384x_drvr_start() failed,result=%d\n", (int)result);
+ "hfa384x_drvr_start() failed,result=%d\n",
+ (int)result);
result P80211ENUM_resultcode_implementation_failure;
wlandev->msdstate = WLAN_MSD_HWPRESENT;
@@ -481,7 +483,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
result = prism2sta_getcardinfo(wlandev);
if (result) {
netdev_err(wlandev->netdev,
- "prism2sta_getcardinfo() failed,result=%d\n", (int)result);
+ "prism2sta_getcardinfo() failed,result=%d\n",
+ (int)result);
result P80211ENUM_resultcode_implementation_failure;
hfa384x_drvr_stop(hw);
@@ -491,7 +494,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
result = prism2sta_globalsetup(wlandev);
if (result) {
netdev_err(wlandev->netdev,
- "prism2sta_globalsetup() failed,result=%d\n", (int)result);
+ "prism2sta_globalsetup() failed,result=%d\n",
+ (int)result);
result P80211ENUM_resultcode_implementation_failure;
hfa384x_drvr_stop(hw);
@@ -1244,9 +1248,9 @@ void prism2sta_processing_defer(struct work_struct *data)
HFA384x_RID_CURRENTSSID, result);
return;
}
- prism2mgmt_bytestr2pstr((struct hfa384x_bytestr *) &ssid,
- (p80211pstrd_t *) &
- wlandev->ssid);
+ prism2mgmt_bytestr2pstr(
+ (struct hfa384x_bytestr *) &ssid,
+ (p80211pstrd_t *) &wlandev->ssid);
/* Collect the port status */
result = hfa384x_drvr_getconfig16(hw,
@@ -1658,8 +1662,9 @@ static void prism2sta_inf_authreq_defer(wlandevice_t *wlandev,
if (hw->authlist.cnt >= WLAN_AUTH_MAX) {
rec.status = P80211ENUM_status_ap_full;
} else {
- ether_addr_copy(hw->authlist.addr[hw->authlist.cnt],
- rec.address);
+ ether_addr_copy(
+ hw->authlist.addr[hw->authlist.cnt],
+ rec.address);
hw->authlist.cnt++;
added = 1;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] staging: wlan-ng: fix checkpatch warnings
2015-05-15 13:13 [PATCH] staging: wlan-ng: fix checkpatch warnings chaehyun lim
@ 2015-05-15 13:36 ` walter harms
0 siblings, 0 replies; 2+ messages in thread
From: walter harms @ 2015-05-15 13:36 UTC (permalink / raw)
To: kernel-janitors
Am 15.05.2015 15:13, schrieb chaehyun lim:
> clean up checkpatch.pl in prism2sta.c
> WARNING : line over 80 characters
>
> Signed-off-by: chaehyun lim <chaehyun.lim@gmail.com>
> ---
> drivers/staging/wlan-ng/prism2sta.c | 23 ++++++++++++++---------
> 1 file changed, 14 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/staging/wlan-ng/prism2sta.c b/drivers/staging/wlan-ng/prism2sta.c
> index ddb294e..0329c52 100644
> --- a/drivers/staging/wlan-ng/prism2sta.c
> +++ b/drivers/staging/wlan-ng/prism2sta.c
> @@ -428,7 +428,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
> result = hfa384x_drvr_start(hw);
> if (result) {
> netdev_err(wlandev->netdev,
> - "hfa384x_drvr_start() failed,result=%d\n", (int)result);
> + "hfa384x_drvr_start() failed,result=%d\n",
> + (int)result);
> result > P80211ENUM_resultcode_implementation_failure;
> wlandev->msdstate = WLAN_MSD_HWPRESENT;
> @@ -471,7 +472,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
> result = hfa384x_drvr_start(hw);
> if (result) {
> netdev_err(wlandev->netdev,
> - "hfa384x_drvr_start() failed,result=%d\n", (int)result);
> + "hfa384x_drvr_start() failed,result=%d\n",
> + (int)result);
> result > P80211ENUM_resultcode_implementation_failure;
> wlandev->msdstate = WLAN_MSD_HWPRESENT;
> @@ -481,7 +483,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
> result = prism2sta_getcardinfo(wlandev);
> if (result) {
> netdev_err(wlandev->netdev,
> - "prism2sta_getcardinfo() failed,result=%d\n", (int)result);
> + "prism2sta_getcardinfo() failed,result=%d\n",
> + (int)result);
> result > P80211ENUM_resultcode_implementation_failure;
> hfa384x_drvr_stop(hw);
> @@ -491,7 +494,8 @@ u32 prism2sta_ifstate(wlandevice_t *wlandev, u32 ifstate)
> result = prism2sta_globalsetup(wlandev);
> if (result) {
> netdev_err(wlandev->netdev,
> - "prism2sta_globalsetup() failed,result=%d\n", (int)result);
> + "prism2sta_globalsetup() failed,result=%d\n",
> + (int)result);
> result > P80211ENUM_resultcode_implementation_failure;
> hfa384x_drvr_stop(hw);
hello chaehyun lim,
my conclusion:
i do not believe that this improves readability.
An idea would be to detangle the case statements and create a helperfunctions
for thew states in prism2sta_ifstate.
just my 2 cents,
re,
wh
> @@ -1244,9 +1248,9 @@ void prism2sta_processing_defer(struct work_struct *data)
> HFA384x_RID_CURRENTSSID, result);
> return;
> }
> - prism2mgmt_bytestr2pstr((struct hfa384x_bytestr *) &ssid,
> - (p80211pstrd_t *) &
> - wlandev->ssid);
> + prism2mgmt_bytestr2pstr(
> + (struct hfa384x_bytestr *) &ssid,
> + (p80211pstrd_t *) &wlandev->ssid);
>
> /* Collect the port status */
> result = hfa384x_drvr_getconfig16(hw,
> @@ -1658,8 +1662,9 @@ static void prism2sta_inf_authreq_defer(wlandevice_t *wlandev,
> if (hw->authlist.cnt >= WLAN_AUTH_MAX) {
> rec.status = P80211ENUM_status_ap_full;
> } else {
> - ether_addr_copy(hw->authlist.addr[hw->authlist.cnt],
> - rec.address);
> + ether_addr_copy(
> + hw->authlist.addr[hw->authlist.cnt],
> + rec.address);
> hw->authlist.cnt++;
> added = 1;
> }
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-05-15 13:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-15 13:13 [PATCH] staging: wlan-ng: fix checkpatch warnings chaehyun lim
2015-05-15 13:36 ` walter harms
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.