[PATCH] staging: rtl8188eu: core: rtw_recv.c: Remove Unnecessary parentheses

[Saurav Girepunje]

Remove Unnecessary parentheses around precvframe->list,
pmlmeext->mlmext_info and precvpriv->free_recv_queue

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8188eu/core/rtw_recv.c | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/rtl8188eu/core/rtw_recv.c 
b/drivers/staging/rtl8188eu/core/rtw_recv.c
index 9caf7041ad60..620da6c003d8 100644
--- a/drivers/staging/rtl8188eu/core/rtw_recv.c
+++ b/drivers/staging/rtl8188eu/core/rtw_recv.c
@@ -145,8 +145,8 @@ int rtw_free_recvframe(struct recv_frame *precvframe,
 
 int _rtw_enqueue_recvframe(struct recv_frame *precvframe, struct __queue 
*queue)
 {
-   list_del_init(&(precvframe->list));
-   list_add_tail(&(precvframe->list), get_list_head(queue));
+   list_del_init(&precvframe->list);
+   list_add_tail(&precvframe->list, get_list_head(queue));
 
return _SUCCESS;
 }
@@ -219,7 +219,7 @@ static int recvframe_chkmic(struct adapter *adapter,
struct  security_priv   *psecuritypriv = &adapter->securitypriv;
 
struct mlme_ext_priv*pmlmeext = &adapter->mlmeextpriv;
-   struct mlme_ext_info*pmlmeinfo = &(pmlmeext->mlmext_info);
+   struct mlme_ext_info*pmlmeinfo = &pmlmeext->mlmext_info;
 
stainfo = rtw_get_stainfo(&adapter->stapriv, &prxattrib->ta[0]);
 
@@ -1500,7 +1500,7 @@ static int amsdu_to_msdu(struct adapter *padapter, struct 
recv_frame *prframe)
struct rx_pkt_attrib *pattrib;
struct sk_buff *sub_skb, *subframes[MAX_SUBFRAME_COUNT];
struct recv_priv *precvpriv = &padapter->recvpriv;
-   struct __queue *pfree_recv_queue = &(precvpriv->free_recv_queue);
+   struct __queue *pfree_recv_queue = &precvpriv->free_recv_queue;
 
nr_subframes = 0;
pattrib = &prframe->attrib;
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8188eu: core: add spaces around '-', '+', '>>','<<' and '*' in rtw_efuse.c

[Saurav Girepunje]

Add spaces around '-', '+', '>>','<<' and '*' to improve readability
and follow kernel coding style.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8188eu/core/rtw_efuse.c | 128 ++---
 1 file changed, 64 insertions(+), 64 deletions(-)

diff --git a/drivers/staging/rtl8188eu/core/rtw_efuse.c 
b/drivers/staging/rtl8188eu/core/rtw_efuse.c
index 533ca1ddffb0..02c476f45b33 100644
--- a/drivers/staging/rtl8188eu/core/rtw_efuse.c
+++ b/drivers/staging/rtl8188eu/core/rtw_efuse.c
@@ -108,7 +108,7 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 
_size_byte, u8  *pbuf)
/*  1. Read the first byte to check if efuse is empty!!! */
/*  */
/*  */
-   rtemp8 = *(phymap+eFuse_Addr);
+   rtemp8 = *(phymap + eFuse_Addr);
if (rtemp8 != 0xFF) {
efuse_utilized++;
eFuse_Addr++;
@@ -124,10 +124,10 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 
_size_byte, u8  *pbuf)
/*  Check PG header for section num. */
if ((rtemp8 & 0x1F) == 0x0F) {  /* extended header */
u1temp = (rtemp8 & 0xE0) >> 5;
-   rtemp8 = *(phymap+eFuse_Addr);
+   rtemp8 = *(phymap + eFuse_Addr);
if ((rtemp8 & 0x0F) == 0x0F) {
eFuse_Addr++;
-   rtemp8 = *(phymap+eFuse_Addr);
+   rtemp8 = *(phymap + eFuse_Addr);
 
if (rtemp8 != 0xFF && (eFuse_Addr < 
EFUSE_REAL_CONTENT_LEN_88E))
eFuse_Addr++;
@@ -147,13 +147,13 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 
_size_byte, u8  *pbuf)
for (i = 0; i < EFUSE_MAX_WORD_UNIT; i++) {
/*  Check word enable condition in the section 
*/
if (!(wren & 0x01)) {
-   rtemp8 = *(phymap+eFuse_Addr);
+   rtemp8 = *(phymap + eFuse_Addr);
eFuse_Addr++;
efuse_utilized++;
eFuseWord[offset][i] = (rtemp8 & 0xff);
if (eFuse_Addr >= 
EFUSE_REAL_CONTENT_LEN_88E)
break;
-   rtemp8 = *(phymap+eFuse_Addr);
+   rtemp8 = *(phymap + eFuse_Addr);
eFuse_Addr++;
efuse_utilized++;
eFuseWord[offset][i] |= (((u16)rtemp8 
<< 8) & 0xff00);
@@ -165,7 +165,7 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 
_size_byte, u8  *pbuf)
}
}
/*  Read next PG header */
-   rtemp8 = *(phymap+eFuse_Addr);
+   rtemp8 = *(phymap + eFuse_Addr);
 
if (rtemp8 != 0xFF && (eFuse_Addr < 
EFUSE_REAL_CONTENT_LEN_88E)) {
efuse_utilized++;
@@ -178,8 +178,8 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 
_size_byte, u8  *pbuf)
/*  */
for (i = 0; i < EFUSE_MAX_SECTION_88E; i++) {
for (j = 0; j < EFUSE_MAX_WORD_UNIT; j++) {
-   efuseTbl[(i*8)+(j*2)] = (eFuseWord[i][j] & 0xff);
-   efuseTbl[(i*8)+((j*2)+1)] = ((eFuseWord[i][j] >> 8) & 
0xff);
+   efuseTbl[(i * 8) + (j * 2)] = (eFuseWord[i][j] & 0xff);
+   efuseTbl[(i * 8) + ((j * 2) + 1)] = ((eFuseWord[i][j] 
>> 8) & 0xff);
}
}
 
@@ -187,7 +187,7 @@ efuse_phymap_to_logical(u8 *phymap, u16 _offset, u16 
_size_byte, u8  *pbuf)
/*  4. Copy from Efuse map to output pointer memory!!! */
/*  */
for (i = 0; i < _size_byte; i++)
-   pbuf[i] = efuseTbl[_offset+i];
+   pbuf[i] = efuseTbl[_offset + i];
 
/*  */
/*  5. Calculate Efuse utilization. */
@@ -218,16 +218,16 @@ static void efuse_read_phymap_from_txpktbuf(
u8 *pos = content;
 
if (bcnhead < 0) /* if not valid */
-   bcnhead = usb_read8(adapter, REG_TDECTRL+1);
+   bcnhead = usb_read8(adapter, REG_TDECTRL + 1);
 
DBG_88E("%s bcnhead:%d\n", __func__, bcnhead);
 
usb_write8(adapter, REG_PKT_BUFF_ACCESS_CTRL, TXPKT_BUF_SELECT);
 
-   dbg_addr = bcnhead*128/8; /* 8-bytes addressing */
+   dbg_addr = bcnhead * 128 / 8; /* 8-bytes addressing */
 
while (1) {
-   usb_write16(adapter, REG_PKTBUF_DBG_ADDR, dbg_addr+i);
+   usb_wri

[PATCH] staging: rtl8723bs: core: correct spelling mistake in rtw_ap.c

[Saurav Girepunje]

Correct spelling mistake in rtw_ap.c

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_ap.c | 12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c 
b/drivers/staging/rtl8723bs/core/rtw_ap.c
index 7bd5c61b055c..78b5782fb675 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -94,7 +94,7 @@ static void update_BCNTIM(struct adapter *padapter)
} else {
tim_ielen = 0;
 
-   /* calucate head_len */
+   /* calculate head_len */
offset = _FIXED_IE_LENGTH_;
 
/* get ssid_ie len */
@@ -143,7 +143,7 @@ static void update_BCNTIM(struct adapter *padapter)
*dst_ie++ = tim_ielen;
 
*dst_ie++ = 0;/* DTIM count */
-   *dst_ie++ = 1;/* DTIM peroid */
+   *dst_ie++ = 1;/* DTIM period */
 
if (pstapriv->tim_bitmap&BIT(0))/* for bc/mc frames */
*dst_ie++ = BIT(0);/* bitmap ctrl */
@@ -292,7 +292,7 @@ void expire_timeout_chk(struct adapter *padapter)
 
if (psta->state & WIFI_SLEEP_STATE) {
if (!(psta->state & WIFI_STA_ALIVE_CHK_STATE)) {
-   /* to check if alive by another methods 
if staion is at ps mode. */
+   /* to check if alive by another methods 
if station is at ps mode. */
psta->expire_to = pstapriv->expire_to;
psta->state |= WIFI_STA_ALIVE_CHK_STATE;
 
@@ -916,7 +916,7 @@ void start_bss_network(struct adapter *padapter, u8 *pbuf)
UpdateBrateTbl(padapter, pnetwork->SupportedRates);
rtw_hal_set_hwreg(padapter, HW_VAR_BASIC_RATE, 
pnetwork->SupportedRates);
 
-   /* udpate capability after cur_wireless_mode updated */
+   /* update capability after cur_wireless_mode updated */
update_capinfo(
padapter,
rtw_get_capability((struct wlan_bssid_ex *)pnetwork)
@@ -1015,7 +1015,7 @@ int rtw_check_beacon_data(struct adapter *padapter, u8 
*pbuf,  int len)
pbss_network->Ssid.SsidLength = ie_len;
}
 
-   /* chnnel */
+   /* channel */
channel = 0;
pbss_network->Configuration.Length = 0;
p = rtw_get_ie(
@@ -1827,7 +1827,7 @@ void update_beacon(struct adapter *padapter, u8 ie_id, u8 
*oui, u8 tx)
 
 /*
 op_mode
-Set to 0 (HT pure) under the followign conditions
+Set to 0 (HT pure) under the following conditions
- all STAs in the BSS are 20/40 MHz HT in 20/40 MHz BSS or
- all STAs in the BSS are 20 MHz HT in 20 MHz BSS
 Set to 1 (HT non-member protection) if there may be non-HT STAs
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: core: Remove Unnecessary parentheses

[Saurav Girepunje]

Remove Unnecessary parentheses around padapter->mlmepriv,
pmlmeext->mlmext_info and pmlmeinfo->network in rtw_ap.c

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_ap.c | 12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c 
b/drivers/staging/rtl8723bs/core/rtw_ap.c
index 78b5782fb675..45a745696af6 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -17,7 +17,7 @@ extern unsigned char WFD_OUI[];
 
 void init_mlme_ap_info(struct adapter *padapter)
 {
-   struct mlme_priv *pmlmepriv = &(padapter->mlmepriv);
+   struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
struct sta_priv *pstapriv = &padapter->stapriv;
struct wlan_acl_pool *pacl_list = &pstapriv->acl_list;
 
@@ -34,9 +34,9 @@ void init_mlme_ap_info(struct adapter *padapter)
 void free_mlme_ap_info(struct adapter *padapter)
 {
struct sta_info *psta = NULL;
-   struct mlme_priv *pmlmepriv = &(padapter->mlmepriv);
+   struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
-   struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
+   struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
 
/* stop_ap_mode(padapter); */
 
@@ -58,9 +58,9 @@ void free_mlme_ap_info(struct adapter *padapter)
 static void update_BCNTIM(struct adapter *padapter)
 {
struct sta_priv *pstapriv = &padapter->stapriv;
-   struct mlme_ext_priv *pmlmeext = &(padapter->mlmeextpriv);
-   struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
-   struct wlan_bssid_ex *pnetwork_mlmeext = &(pmlmeinfo->network);
+   struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
+   struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
+   struct wlan_bssid_ex *pnetwork_mlmeext = &pmlmeinfo->network;
unsigned char *pie = pnetwork_mlmeext->IEs;
 
/* DBG_871X("%s\n", __func__); */
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: core: add spaces around '-', '+', '&','|' and '?'

[Saurav Girepunje]

Add spaces around '-', '+', '&','|' and '?' to improve readability
and follow kernel coding style in rtw_ap.c

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_ap.c | 88 -
 1 file changed, 44 insertions(+), 44 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c 
b/drivers/staging/rtl8723bs/core/rtw_ap.c
index 45a745696af6..02f5478845b4 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -83,7 +83,7 @@ static void update_BCNTIM(struct adapter *padapter)
if (p != NULL && tim_ielen > 0) {
tim_ielen += 2;
 
-   premainder_ie = p+tim_ielen;
+   premainder_ie = p + tim_ielen;
 
tim_ie_offset = (sint)(p - pie);
 
@@ -105,7 +105,7 @@ static void update_BCNTIM(struct adapter *padapter)
(pnetwork_mlmeext->IELength - 
_BEACON_IE_OFFSET_)
);
if (p != NULL)
-   offset += tmp_len+2;
+   offset += tmp_len + 2;
 
/*  get supported rates len */
p = rtw_get_ie(
@@ -114,7 +114,7 @@ static void update_BCNTIM(struct adapter *padapter)
(pnetwork_mlmeext->IELength - 
_BEACON_IE_OFFSET_)
);
if (p !=  NULL)
-   offset += tmp_len+2;
+   offset += tmp_len + 2;
 
/* DS Parameter Set IE, len =3 */
offset += 3;
@@ -135,7 +135,7 @@ static void update_BCNTIM(struct adapter *padapter)
 
*dst_ie++ = _TIM_IE_;
 
-   if ((pstapriv->tim_bitmap&0xff00) && 
(pstapriv->tim_bitmap&0x00fe))
+   if ((pstapriv->tim_bitmap & 0xff00) && (pstapriv->tim_bitmap & 
0x00fe))
tim_ielen = 5;
else
tim_ielen = 4;
@@ -145,7 +145,7 @@ static void update_BCNTIM(struct adapter *padapter)
*dst_ie++ = 0;/* DTIM count */
*dst_ie++ = 1;/* DTIM period */
 
-   if (pstapriv->tim_bitmap&BIT(0))/* for bc/mc frames */
+   if (pstapriv->tim_bitmap & BIT(0))/* for bc/mc frames */
*dst_ie++ = BIT(0);/* bitmap ctrl */
else
*dst_ie++ = 0;
@@ -153,7 +153,7 @@ static void update_BCNTIM(struct adapter *padapter)
if (tim_ielen == 4) {
__le16 pvb;
 
-   if (pstapriv->tim_bitmap&0xff00)
+   if (pstapriv->tim_bitmap & 0xff00)
pvb = cpu_to_le16(pstapriv->tim_bitmap >> 8);
else
pvb = tim_bitmap_le;
@@ -188,8 +188,8 @@ u8 chk_sta_is_alive(struct sta_info *psta)
/*  STA_RX_PKTS_ARG(psta) */
, STA_RX_PKTS_DIFF_ARG(psta)
, psta->expire_to
-   , psta->state&WIFI_SLEEP_STATE?"PS, ":""
-   , psta->state&WIFI_STA_ALIVE_CHK_STATE?"SAC, ":""
+   , psta->state & WIFI_SLEEP_STATE ? "PS, " : ""
+   , psta->state & WIFI_STA_ALIVE_CHK_STATE ? "SAC, " : ""
, psta->sleepq_len
);
#endif
@@ -325,10 +325,10 @@ void expire_timeout_chk(struct adapter *padapter)
updated = ap_free_sta(padapter, psta, false, 
WLAN_REASON_DEAUTH_LEAVING);
} else {
/* TODO: Aging mechanism to digest frames in sleep_q to 
avoid running out of xmitframe */
-   if (psta->sleepq_len > 
(NR_XMITFRAME/pstapriv->asoc_list_cnt)
+   if (psta->sleepq_len > (NR_XMITFRAME / 
pstapriv->asoc_list_cnt)
&& padapter->xmitpriv.free_xmitframe_cnt < ((
-   NR_XMITFRAME/pstapriv->asoc_list_cnt
-   )/2)
+   NR_XMITFRAME / pstapriv->asoc_list_cnt
+   ) / 2)
) {
DBG_871X(
"%s sta:"MAC_FMT", sleepq_len:%u, 
free_xmitframe_cnt:%u, asoc_list_cnt:%u, clear sleep_q\n",
@@ -586,8 +586,8 @@ void update_sta_info_apmode(struct adapter *padapter, 
struct sta_info *psta)
phtpriv_sta->ampdu_enable = phtpriv_ap->ampdu_enable;
 
phtpriv_sta->rx_ampdu_min_spacing = (

[PATCH] staging: rtl8723bs: hal: Drop condition with no effect

[Saurav Girepunje]

As the "else if" and "else" branch body are identical the condition
has no effect. So drop the else if condition.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/hal/hal_btcoex.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8723bs/hal/hal_btcoex.c 
b/drivers/staging/rtl8723bs/hal/hal_btcoex.c
index 19486f0e0ead..6e4a1fcb8790 100644
--- a/drivers/staging/rtl8723bs/hal/hal_btcoex.c
+++ b/drivers/staging/rtl8723bs/hal/hal_btcoex.c
@@ -482,10 +482,8 @@ static u8 halbtcoutsrc_Get(void *pBtcContext, u8 getType, 
void *pOutBuf)
*pU4Tmp = BTC_WIFI_BW_LEGACY;
else if (pHalData->CurrentChannelBW == CHANNEL_WIDTH_20)
*pU4Tmp = BTC_WIFI_BW_HT20;
-   else if (pHalData->CurrentChannelBW == CHANNEL_WIDTH_40)
-   *pU4Tmp = BTC_WIFI_BW_HT40;
else
-   *pU4Tmp = BTC_WIFI_BW_HT40; /* todo */
+   *pU4Tmp = BTC_WIFI_BW_HT40;
break;
 
case BTC_GET_U4_WIFI_TRAFFIC_DIRECTION:
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rt;8723bs: os_dep: Drop condition with no effect

[Saurav Girepunje]

As the "else if" and "else" branch body are identical the condition
has no effect. So drop the "else if" condition.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c 
b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
index 90c2997256b7..36ea7256329a 100644
--- a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
+++ b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
@@ -215,8 +215,6 @@ static char *translate_scan(struct adapter *padapter,
} else if (ht_cap) {
if (mcs_rate&0x8000) { /* MCS15 */
max_rate = (bw_40MHz) ? 
((short_GI)?300:270):((short_GI)?144:130);
-   } else if (mcs_rate&0x0080) { /* MCS7 */
-   max_rate = (bw_40MHz) ? 
((short_GI)?150:135):((short_GI)?72:65);
} else { /* default MCS7 */
/* DBG_871X("wx_get_scan, mcs_rate_bitmap = 0x%x\n", 
mcs_rate); */
max_rate = (bw_40MHz) ? 
((short_GI)?150:135):((short_GI)?72:65);
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: os_dep: Drop condition with no effect

[Saurav Girepunje]

As the "else if" and "else" branch body are identical the condition
has no effect. So drop the "else if" condition.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c 
b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
index 90c2997256b7..36ea7256329a 100644
--- a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
+++ b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
@@ -215,8 +215,6 @@ static char *translate_scan(struct adapter *padapter,
} else if (ht_cap) {
if (mcs_rate&0x8000) { /* MCS15 */
max_rate = (bw_40MHz) ? 
((short_GI)?300:270):((short_GI)?144:130);
-   } else if (mcs_rate&0x0080) { /* MCS7 */
-   max_rate = (bw_40MHz) ? 
((short_GI)?150:135):((short_GI)?72:65);
} else { /* default MCS7 */
/* DBG_871X("wx_get_scan, mcs_rate_bitmap = 0x%x\n", 
mcs_rate); */
max_rate = (bw_40MHz) ? 
((short_GI)?150:135):((short_GI)?72:65);
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: core: Drop condition with no effect

[Saurav Girepunje]

As the "else if" and "else" branch body are identical the condition
has no effect. So drop the "else if" condition

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_cmd.c | 10 --
 1 file changed, 10 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_cmd.c 
b/drivers/staging/rtl8723bs/core/rtw_cmd.c
index c6565b0e502c..3f2f9b12cb61 100644
--- a/drivers/staging/rtl8723bs/core/rtw_cmd.c
+++ b/drivers/staging/rtl8723bs/core/rtw_cmd.c
@@ -507,19 +507,9 @@ int rtw_cmd_thread(void *context)
 
cmd_process_time = jiffies_to_msecs(jiffies - cmd_start_time);
if (cmd_process_time > 1000) {
-   if (pcmd->cmdcode == GEN_CMD_CODE(_Set_Drv_Extra)) {
-   DBG_871X(ADPT_FMT" cmd =%d process_time =%lu > 
1 sec\n",
-   ADPT_ARG(pcmd->padapter), 
pcmd->cmdcode, cmd_process_time);
-   /* rtw_warn_on(1); */
-   } else if (pcmd->cmdcode == 
GEN_CMD_CODE(_Set_MLME_EVT)) {
DBG_871X(ADPT_FMT" cmd =%d, process_time =%lu > 
1 sec\n",
ADPT_ARG(pcmd->padapter), 
pcmd->cmdcode, cmd_process_time);
/* rtw_warn_on(1); */
-   } else {
-   DBG_871X(ADPT_FMT" cmd =%d, process_time =%lu > 
1 sec\n",
-   ADPT_ARG(pcmd->padapter), 
pcmd->cmdcode, cmd_process_time);
-   /* rtw_warn_on(1); */
-   }
}
 
/* call callback function for post-processed */
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: os_dep: Remove unused variable

[Saurav Girepunje]

Remove unused variable ret from functions rtw_mp_ioctl_hdl,
rtw_get_ap_info, rtw_mp_efuse_set, rtw_tdls, rtw_tdls_get .

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 15 +--
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c 
b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
index 90c2997256b7..a51009963295 100644
--- a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
+++ b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
@@ -2433,8 +2433,7 @@ static  int rtw_drvext_hdl(struct net_device *dev, struct 
iw_request_info *info,
 static int rtw_mp_ioctl_hdl(struct net_device *dev, struct iw_request_info 
*info,
union iwreq_data *wrqu, char 
*extra)
 {
-   int ret = 0;
-   return ret;
+   return 0;
 }
 
 static int rtw_get_ap_info(struct net_device *dev,
@@ -4467,24 +4466,21 @@ static int rtw_mp_efuse_get(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wdata, char *extra)
 {
-   int err = 0;
-   return err;
+   return 0;
 }
 
 static int rtw_mp_efuse_set(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wdata, char *extra)
 {
-   int err = 0;
-   return err;
+   return 0;
 }
 
 static int rtw_tdls(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wrqu, char *extra)
 {
-   int ret = 0;
-   return ret;
+   return 0;
 }
 
 
@@ -4492,8 +4488,7 @@ static int rtw_tdls_get(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wrqu, char *extra)
 {
-   int ret = 0;
-   return ret;
+   return 0;
 }
 
 
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: os_dep: Remove return variables

[Saurav Girepunje]

Remove return variables, return the values
directly, as the functions all return 0 in all cases.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/os_dep/ioctl_linux.c | 15 +--
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c 
b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
index 36ea7256329a..5ec48ec926eb 100644
--- a/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
+++ b/drivers/staging/rtl8723bs/os_dep/ioctl_linux.c
@@ -2431,8 +2431,7 @@ static  int rtw_drvext_hdl(struct net_device *dev, struct 
iw_request_info *info,
 static int rtw_mp_ioctl_hdl(struct net_device *dev, struct iw_request_info 
*info,
union iwreq_data *wrqu, char 
*extra)
 {
-   int ret = 0;
-   return ret;
+   return 0;
 }
 
 static int rtw_get_ap_info(struct net_device *dev,
@@ -4465,24 +4464,21 @@ static int rtw_mp_efuse_get(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wdata, char *extra)
 {
-   int err = 0;
-   return err;
+   return 0;
 }
 
 static int rtw_mp_efuse_set(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wdata, char *extra)
 {
-   int err = 0;
-   return err;
+   return 0;
 }
 
 static int rtw_tdls(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wrqu, char *extra)
 {
-   int ret = 0;
-   return ret;
+   return 0;
 }
 
 
@@ -4490,8 +4486,7 @@ static int rtw_tdls_get(struct net_device *dev,
struct iw_request_info *info,
union iwreq_data *wrqu, char *extra)
 {
-   int ret = 0;
-   return ret;
+   return 0;
 }
 
 
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] RTWAP:Fixed Coding function and style issues

[Saurav-Girepunje]

RTWAP:Fixed Coding function and style issues
  Removed Unnecessary parentheses

Signed-off-by: Saurav-Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_ap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c 
b/drivers/staging/rtl8723bs/core/rtw_ap.c
index c98b9c0d79fe..2c443d1b28b2 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -34,7 +34,7 @@ void init_mlme_ap_info(struct adapter *padapter)
 void free_mlme_ap_info(struct adapter *padapter)
 {
struct sta_info *psta = NULL;
-   struct mlme_priv *pmlmepriv = &(padapter->mlmepriv);
+   struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
 
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: Remove Unnecessary parentheses

[Saurav-Girepunje]

Removed Unnecessary parentheses around "padapter->mlmepriv",
"pmlmeext->mlmext_info" and "pmlmeinfo->network"

Signed-off-by: Saurav-Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_ap.c | 12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c 
b/drivers/staging/rtl8723bs/core/rtw_ap.c
index 7bd5c61b055c..ba30ba04a103 100644
--- a/drivers/staging/rtl8723bs/core/rtw_ap.c
+++ b/drivers/staging/rtl8723bs/core/rtw_ap.c
@@ -17,7 +17,7 @@ extern unsigned char WFD_OUI[];
 
 void init_mlme_ap_info(struct adapter *padapter)
 {
-   struct mlme_priv *pmlmepriv = &(padapter->mlmepriv);
+   struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
struct sta_priv *pstapriv = &padapter->stapriv;
struct wlan_acl_pool *pacl_list = &pstapriv->acl_list;
 
@@ -34,9 +34,9 @@ void init_mlme_ap_info(struct adapter *padapter)
 void free_mlme_ap_info(struct adapter *padapter)
 {
struct sta_info *psta = NULL;
-   struct mlme_priv *pmlmepriv = &(padapter->mlmepriv);
+   struct mlme_priv *pmlmepriv = &padapter->mlmepriv;
struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
-   struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
+   struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
 
/* stop_ap_mode(padapter); */
 
@@ -58,9 +58,9 @@ void free_mlme_ap_info(struct adapter *padapter)
 static void update_BCNTIM(struct adapter *padapter)
 {
struct sta_priv *pstapriv = &padapter->stapriv;
-   struct mlme_ext_priv *pmlmeext = &(padapter->mlmeextpriv);
-   struct mlme_ext_info *pmlmeinfo = &(pmlmeext->mlmext_info);
-   struct wlan_bssid_ex *pnetwork_mlmeext = &(pmlmeinfo->network);
+   struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
+   struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
+   struct wlan_bssid_ex *pnetwork_mlmeext = &pmlmeinfo->network;
unsigned char *pie = pnetwork_mlmeext->IEs;
 
/* DBG_871X("%s\n", __func__); */
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: greybus: bootrom: fix uninitialized variables

[Saurav Girepunje]

fix uninitialized variables issue found using static code analysis tool

(error) Uninitialized variable: offset
(error) Uninitialized variable: size

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/greybus/bootrom.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/greybus/bootrom.c 
b/drivers/staging/greybus/bootrom.c
index a8efb86..9eabeb3 100644
--- a/drivers/staging/greybus/bootrom.c
+++ b/drivers/staging/greybus/bootrom.c
@@ -245,7 +245,7 @@ static int gb_bootrom_get_firmware(struct gb_operation *op)
struct gb_bootrom_get_firmware_request *firmware_request;
struct gb_bootrom_get_firmware_response *firmware_response;
struct device *dev = &op->connection->bundle->dev;
-   unsigned int offset, size;
+   unsigned int offset = 0, size = 0;
enum next_request_type next_request;
int ret = 0;
 
--

1.9.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH] staging: greybus: bootrom: fix uninitialized variables

[SAURAV GIREPUNJE]

On 25/01/20 11:00 +0100, Johan Hovold wrote:

On Sat, Jan 25, 2020 at 02:14:03PM +0530, Saurav Girepunje wrote:

fix uninitialized variables issue found using static code analysis tool


Which tool is that?


(error) Uninitialized variable: offset
(error) Uninitialized variable: size

Signed-off-by: Saurav Girepunje 
---
  drivers/staging/greybus/bootrom.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/greybus/bootrom.c 
b/drivers/staging/greybus/bootrom.c
index a8efb86..9eabeb3 100644
--- a/drivers/staging/greybus/bootrom.c
+++ b/drivers/staging/greybus/bootrom.c
@@ -245,7 +245,7 @@ static int gb_bootrom_get_firmware(struct gb_operation *op)
struct gb_bootrom_get_firmware_request *firmware_request;
struct gb_bootrom_get_firmware_response *firmware_response;
struct device *dev = &op->connection->bundle->dev;
-   unsigned int offset, size;
+   unsigned int offset = 0, size = 0;
enum next_request_type next_request;
int ret = 0;


I think this has come up in the past, and while the code in question is
overly complicated and confuses static checkers as well as humans, it
looks correct to me.

Please make sure to verify the output of any tools before posting
patches based on them.

Johan

I used cppcheck tool .
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: comedi: drivers: fix condition with no effect

[Saurav Girepunje]

fix warning reorted by coccicheck
WARNING: possible condition with no effect (if == else)

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/comedi/drivers/das1800.c | 6 --
 1 file changed, 6 deletions(-)

diff --git a/drivers/staging/comedi/drivers/das1800.c 
b/drivers/staging/comedi/drivers/das1800.c
index f16aa7e9..7ab72e8 100644
--- a/drivers/staging/comedi/drivers/das1800.c
+++ b/drivers/staging/comedi/drivers/das1800.c
@@ -1299,12 +1299,6 @@ static int das1800_attach(struct comedi_device *dev,
outb(DAC(i), dev->iobase + DAS1800_SELECT);
outw(0, dev->iobase + DAS1800_DAC);
}
-   } else if (board->id == DAS1800_ID_AO) {
-   /*
-* 'ao' boards have waveform analog outputs that are not
-* currently supported.
-*/
-   s->type  = COMEDI_SUBD_UNUSED;
} else {
s->type  = COMEDI_SUBD_UNUSED;
}
--
1.9.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: core: fix condition with no effect

[Saurav Girepunje]

fix warning reorted by coccicheck
WARNING: possible condition with no effect (if == else)

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/core/rtw_mlme.c | 14 ++
 1 file changed, 2 insertions(+), 12 deletions(-)

diff --git a/drivers/staging/rtl8723bs/core/rtw_mlme.c 
b/drivers/staging/rtl8723bs/core/rtw_mlme.c
index 71fcb46..5f110c3 100644
--- a/drivers/staging/rtl8723bs/core/rtw_mlme.c
+++ b/drivers/staging/rtl8723bs/core/rtw_mlme.c
@@ -2771,18 +2771,8 @@ void rtw_update_ht_cap(struct adapter *padapter, u8 
*pie, uint ie_len, u8 channe
DBG_871X("+rtw_update_ht_cap()\n");
 
 	/* maybe needs check if ap supports rx ampdu. */

-   if (!(phtpriv->ampdu_enable) && pregistrypriv->ampdu_enable == 1) {
-   if (pregistrypriv->wifi_spec == 1) {
-   /* remove this part because testbed AP should disable 
RX AMPDU */
-   /* phtpriv->ampdu_enable = false; */
-   phtpriv->ampdu_enable = true;
-   } else {
-   phtpriv->ampdu_enable = true;
-   }
-   } else if (pregistrypriv->ampdu_enable == 2) {
-   /* remove this part because testbed AP should disable RX AMPDU 
*/
-   /* phtpriv->ampdu_enable = true; */
-   }
+   if (!(phtpriv->ampdu_enable) && pregistrypriv->ampdu_enable == 1)
+   phtpriv->ampdu_enable = true;
 
 	/* check Max Rx A-MPDU Size */

len = 0;
--
1.9.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: rtl8723bs: hal: fix condition with no effect

[Saurav Girepunje]

fix warning reorted by coccicheck
WARNING: possible condition with no effect (if == else)

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/rtl8723bs/hal/HalBtc8723b2Ant.c | 8 
 1 file changed, 8 deletions(-)

diff --git a/drivers/staging/rtl8723bs/hal/HalBtc8723b2Ant.c 
b/drivers/staging/rtl8723bs/hal/HalBtc8723b2Ant.c
index 02da0a8..8dfa9b9 100644
--- a/drivers/staging/rtl8723bs/hal/HalBtc8723b2Ant.c
+++ b/drivers/staging/rtl8723bs/hal/HalBtc8723b2Ant.c
@@ -1610,8 +1610,6 @@ static void halbtc8723b2ant_TdmaDurationAdjust(

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(13);
else if (maxInterval == 2)

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(14);
-   else if (maxInterval == 3)
-   
HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(15);
else

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(15);
} else {
@@ -1619,8 +1617,6 @@ static void halbtc8723b2ant_TdmaDurationAdjust(

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(9);
else if (maxInterval == 2)

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(10);
-   else if (maxInterval == 3)
-   
HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(11);
else

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(11);
}
@@ -1630,8 +1626,6 @@ static void halbtc8723b2ant_TdmaDurationAdjust(

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(5);
else if (maxInterval == 2)

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(6);
-   else if (maxInterval == 3)
-   
HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(7);
else

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(7);
} else {
@@ -1639,8 +1633,6 @@ static void halbtc8723b2ant_TdmaDurationAdjust(

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(1);
else if (maxInterval == 2)

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(2);
-   else if (maxInterval == 3)
-   
HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(3);
else

HAL_BTC8723B2ANT_DMA_DURATION_ADJUST(3);
}
--
1.9.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: vt6656: fix Unneeded variable: "ret"

[Saurav Girepunje]

Remove unneeded variable "ret". Issues reported by coccicheck.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/vt6656/card.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/vt6656/card.c b/drivers/staging/vt6656/card.c
index 7958fc1..654cebc 100644
--- a/drivers/staging/vt6656/card.c
+++ b/drivers/staging/vt6656/card.c
@@ -719,7 +719,6 @@ int vnt_radio_power_off(struct vnt_private *priv)
  */
 int vnt_radio_power_on(struct vnt_private *priv)
 {
-   int ret = 0;
 
 	vnt_exit_deep_sleep(priv);
 
@@ -739,7 +738,7 @@ int vnt_radio_power_on(struct vnt_private *priv)
 
 	vnt_mac_reg_bits_off(priv, MAC_REG_GPIOCTL1, GPIO3_INTMD);
 
-	return ret;

+   return 0;
 }
 
 void vnt_set_bss_mode(struct vnt_private *priv)

--
1.9.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: greybus: fix fw is NULL but dereferenced.

[Saurav Girepunje]

Fix the warning reported by cocci check.

Changes:

In queue_work fw dereference before it actually get assigned.
move queue_work before gb_bootrom_set_timeout.

As gb_bootrom_get_firmware () return NEXT_REQ_READY_TO_BOOT
only when there is no error and offset + size is actually equal
to fw->size. So initialized next_request to NEXT_REQ_GET_FIRMWARE
for return in other case.

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/greybus/bootrom.c | 6 ++
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/greybus/bootrom.c 
b/drivers/staging/greybus/bootrom.c
index a8efb86..f54514e 100644
--- a/drivers/staging/greybus/bootrom.c
+++ b/drivers/staging/greybus/bootrom.c
@@ -246,7 +246,7 @@ static int gb_bootrom_get_firmware(struct gb_operation *op)
struct gb_bootrom_get_firmware_response *firmware_response;
struct device *dev = &op->connection->bundle->dev;
unsigned int offset, size;
-   enum next_request_type next_request;
+   enum next_request_type next_request = NEXT_REQ_GET_FIRMWARE;
int ret = 0;
 
 	/* Disable timeouts */

@@ -296,13 +296,11 @@ static int gb_bootrom_get_firmware(struct gb_operation 
*op)
 unlock:
mutex_unlock(&bootrom->mutex);
 
-queue_work:

/* Refresh timeout */
if (!ret && (offset + size == fw->size))
next_request = NEXT_REQ_READY_TO_BOOT;
-   else
-   next_request = NEXT_REQ_GET_FIRMWARE;
 
+queue_work:

gb_bootrom_set_timeout(bootrom, next_request, NEXT_REQ_TIMEOUT_MS);
 
 	return ret;

--
1.9.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] Staging: comedi: driver: Remove condition with no effect

[Saurav Girepunje]

Remove below warning in das1800.c
WARNING: possible condition with no effect (if == else)

Signed-off-by: Saurav Girepunje 
---
 drivers/staging/comedi/drivers/das1800.c | 6 --
 1 file changed, 6 deletions(-)

diff --git a/drivers/staging/comedi/drivers/das1800.c 
b/drivers/staging/comedi/drivers/das1800.c
index f16aa7e9f4f3..7ab72e83d3d0 100644
--- a/drivers/staging/comedi/drivers/das1800.c
+++ b/drivers/staging/comedi/drivers/das1800.c
@@ -1299,12 +1299,6 @@ static int das1800_attach(struct comedi_device *dev,
outb(DAC(i), dev->iobase + DAS1800_SELECT);
outw(0, dev->iobase + DAS1800_DAC);
}
-   } else if (board->id == DAS1800_ID_AO) {
-   /*
-* 'ao' boards have waveform analog outputs that are not
-* currently supported.
-*/
-   s->type = COMEDI_SUBD_UNUSED;
} else {
s->type = COMEDI_SUBD_UNUSED;
}
-- 
2.20.1

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH] Staging: comedi: driver: Remove condition with no effect

[SAURAV GIREPUNJE]

On Mon, Jul 13, 2020 at 02:41:19PM +0100, Ian Abbott wrote:
> On 13/07/2020 14:34, Greg KH wrote:
> > On Sun, Jul 12, 2020 at 12:36:28PM +0530, Saurav Girepunje wrote:
> > > Remove below warning in das1800.c
> > > WARNING: possible condition with no effect (if == else)
> > > 
> > > Signed-off-by: Saurav Girepunje 
> > > ---
> > >   drivers/staging/comedi/drivers/das1800.c | 6 --
> > >   1 file changed, 6 deletions(-)
> > > 
> > > diff --git a/drivers/staging/comedi/drivers/das1800.c 
> > > b/drivers/staging/comedi/drivers/das1800.c
> > > index f16aa7e9f4f3..7ab72e83d3d0 100644
> > > --- a/drivers/staging/comedi/drivers/das1800.c
> > > +++ b/drivers/staging/comedi/drivers/das1800.c
> > > @@ -1299,12 +1299,6 @@ static int das1800_attach(struct comedi_device 
> > > *dev,
> > >   outb(DAC(i), dev->iobase + DAS1800_SELECT);
> > >   outw(0, dev->iobase + DAS1800_DAC);
> > >   }
> > > - } else if (board->id == DAS1800_ID_AO) {
> > > - /*
> > > -  * 'ao' boards have waveform analog outputs that are not
> > > -  * currently supported.
> > > -  */
> > > - s->type = COMEDI_SUBD_UNUSED;
> > 
> > What gave that warning?  The comment should show you why this is good to
> > keep as-is, right?
> 
> One option is to move the comment into the '} else {' part that follows this
> part.
> 
> -- 
> -=( Ian Abbott  || Web: www.mev.co.uk )=-
> -=( MEV Ltd. is a company registered in England & Wales. )=-
> -=( Registered number: 02862268.  Registered address:)=-
> -=( 15 West Park Road, Bramhall, STOCKPORT, SK7 3JZ, UK. )=-
@gregkh : Got the warning using coccinelle. 
@Ian: Yes, we can move comment to else part.
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel