lkml.org 
[lkml]   [2017]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] staging: rtl8192e: fix coding style issue, improve error handling
Hi Suniel,

[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v4.11-rc1 next-20170309]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Suniel-Mahesh/staging-rtl8192e-fix-coding-style-issue-improve-error-handling/20170310-015706
config: i386-randconfig-x076-201710 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All error/warnings (new ones prefixed by >>):

drivers/staging/rtl8192e/rtl8192e/rtl_core.c: In function 'rtl92e_set_rf_state':
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:220:13: error: 'WLAN_REASON_DISASSOC_STA_' undeclared (first use in this function)
WLAN_REASON_DISASSOC_STA_
^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:220:13: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:221:9: error: expected ')' before 'HAS_LEFT'
HAS_LEFT);
^~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c: In function '_rtl92e_init_priv_variable':
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:970:10: warning: 'return' with a value, in function returning void
return -ENOMEM;
^
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:888:13: note: declared here
static void _rtl92e_init_priv_variable(struct net_device *dev)
^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c: In function '_rtl92e_if_silent_reset':
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1247:24: warning: missing terminating " character
RT_TRACE(COMP_RESET, "%s():======>start to down the
^
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1248:13: warning: missing terminating " character
driver\n", __func__);
^
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1251:5: warning: missing terminating " character
"%s():111111111111111111111111======>start to down the
^
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1252:13: warning: missing terminating " character
driver\n", __func__);
^
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:2751:0: error: unterminated argument list invoking macro "RT_TRACE"
MODULE_PARM_DESC(channels, " Channel bitmask for specific locales. NYI");

>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1247:3: error: 'RT_TRACE' undeclared (first use in this function)
RT_TRACE(COMP_RESET, "%s():======>start to down the
^~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1247:3: error: expected ';' at end of input
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1247:3: error: expected declaration or statement at end of input
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1247:3: error: expected declaration or statement at end of input
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1232:1: warning: label 'RESET_START' defined but not used [-Wunused-label]
RESET_START:
^~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1226:4: error: label 'END' used but not defined
goto END;
^~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1217:24: warning: unused variable 'ieee' [-Wunused-variable]
struct rtllib_device *ieee = priv->rtllib;
^~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1216:6: warning: unused variable 'reset_status' [-Wunused-variable]
int reset_status = 0;
^~~~~~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:1215:5: warning: unused variable 'reset_times' [-Wunused-variable]
u8 reset_times = 0;
^~~~~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c: At top level:
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:71:12: warning: '_rtl92e_pci_probe' used but never defined
static int _rtl92e_pci_probe(struct pci_dev *pdev,
^~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:73:13: warning: '_rtl92e_pci_disconnect' used but never defined
static void _rtl92e_pci_disconnect(struct pci_dev *pdev);
^~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:74:20: warning: '_rtl92e_irq' used but never defined
static irqreturn_t _rtl92e_irq(int irq, void *netdev);
^~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:86:13: warning: '_rtl92e_watchdog_wq_cb' used but never defined
static void _rtl92e_watchdog_wq_cb(void *data);
^~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:87:13: warning: '_rtl92e_watchdog_timer_cb' used but never defined
static void _rtl92e_watchdog_timer_cb(unsigned long data);
^~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:88:13: warning: '_rtl92e_hard_data_xmit' used but never defined
static void _rtl92e_hard_data_xmit(struct sk_buff *skb, struct net_device *dev,
^~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:90:12: warning: '_rtl92e_hard_start_xmit' used but never defined
static int _rtl92e_hard_start_xmit(struct sk_buff *skb, struct net_device *dev);
^~~~~~~~~~~~~~~~~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:91:13: warning: '_rtl92e_tx_cmd' declared 'static' but never defined [-Wunused-function]
static void _rtl92e_tx_cmd(struct net_device *dev, struct sk_buff *skb);
^~~~~~~~~~~~~~
drivers/staging/rtl8192e/rtl8192e/rtl_core.c:92:14: warning: '_rtl92e_tx' declared 'static' but never defined [-Wunused-function]
static short _rtl92e_tx(struct net_device *dev, struct sk_buff *skb);
^~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:93:14: warning: '_rtl92e_pci_initdescring' used but never defined
static short _rtl92e_pci_initdescring(struct net_device *dev);
^~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:94:13: warning: '_rtl92e_irq_tx_tasklet' used but never defined
static void _rtl92e_irq_tx_tasklet(struct r8192_priv *priv);
^~~~~~~~~~~~~~~~~~~~~~
>> drivers/staging/rtl8192e/rtl8192e/rtl_core.c:95:13: warning: '_rtl92e_irq_rx_tasklet' used but never defined
static void _rtl92e_irq_rx_tasklet(struct r8192_priv *priv);
^~~~~~~~~~~~~~~~~~~~~~

vim +/WLAN_REASON_DISASSOC_STA_ +220 drivers/staging/rtl8192e/rtl8192e/rtl_core.c

65 {RTL_PCI_DEVICE(0x07aa, 0x0047, rtl819xp_ops)},
66 {}
67 };
68
69 MODULE_DEVICE_TABLE(pci, rtl8192_pci_id_tbl);
70
> 71 static int _rtl92e_pci_probe(struct pci_dev *pdev,
72 const struct pci_device_id *id);
> 73 static void _rtl92e_pci_disconnect(struct pci_dev *pdev);
> 74 static irqreturn_t _rtl92e_irq(int irq, void *netdev);
75
76 static struct pci_driver rtl8192_pci_driver = {
77 .name = DRV_NAME, /* Driver name */
78 .id_table = rtl8192_pci_id_tbl, /* PCI_ID table */
79 .probe = _rtl92e_pci_probe, /* probe fn */
80 .remove = _rtl92e_pci_disconnect, /* remove fn */
81 .suspend = rtl92e_suspend, /* PM suspend fn */
82 .resume = rtl92e_resume, /* PM resume fn */
83 };
84
85 static short _rtl92e_is_tx_queue_empty(struct net_device *dev);
> 86 static void _rtl92e_watchdog_wq_cb(void *data);
> 87 static void _rtl92e_watchdog_timer_cb(unsigned long data);
> 88 static void _rtl92e_hard_data_xmit(struct sk_buff *skb, struct net_device *dev,
89 int rate);
> 90 static int _rtl92e_hard_start_xmit(struct sk_buff *skb, struct net_device *dev);
> 91 static void _rtl92e_tx_cmd(struct net_device *dev, struct sk_buff *skb);
> 92 static short _rtl92e_tx(struct net_device *dev, struct sk_buff *skb);
> 93 static short _rtl92e_pci_initdescring(struct net_device *dev);
> 94 static void _rtl92e_irq_tx_tasklet(struct r8192_priv *priv);
> 95 static void _rtl92e_irq_rx_tasklet(struct r8192_priv *priv);
> 96 static void _rtl92e_cancel_deferred_work(struct r8192_priv *priv);
> 97 static int _rtl92e_up(struct net_device *dev, bool is_silent_reset);
98 static int _rtl92e_try_up(struct net_device *dev);
99 static int _rtl92e_down(struct net_device *dev, bool shutdownrf);
> 100 static void _rtl92e_restart(void *data);
101
102 /* IO STUFF */
103
104 u8 rtl92e_readb(struct net_device *dev, int x)
105 {
106 return 0xff & readb((u8 __iomem *)dev->mem_start + x);
107 }
108
109 u32 rtl92e_readl(struct net_device *dev, int x)
110 {
111 return readl((u8 __iomem *)dev->mem_start + x);
112 }
113
114 u16 rtl92e_readw(struct net_device *dev, int x)
115 {
116 return readw((u8 __iomem *)dev->mem_start + x);
117 }
118
119 void rtl92e_writeb(struct net_device *dev, int x, u8 y)
120 {
121 writeb(y, (u8 __iomem *)dev->mem_start + x);
122
123 udelay(20);
124 }
125
126 void rtl92e_writel(struct net_device *dev, int x, u32 y)
127 {
128 writel(y, (u8 __iomem *)dev->mem_start + x);
129
130 udelay(20);
131 }
132
133 void rtl92e_writew(struct net_device *dev, int x, u16 y)
134 {
135 writew(y, (u8 __iomem *)dev->mem_start + x);
136
137 udelay(20);
138 }
139
140 /* GENERAL FUNCTION */
141 bool rtl92e_set_rf_state(struct net_device *dev,
142 enum rt_rf_power_state StateToSet,
143 RT_RF_CHANGE_SOURCE ChangeSource)
144 {
145 struct r8192_priv *priv = rtllib_priv(dev);
146 struct rtllib_device *ieee = priv->rtllib;
147 bool bActionAllowed = false;
148 bool bConnectBySSID = false;
149 enum rt_rf_power_state rtState;
150 u16 RFWaitCounter = 0;
151 unsigned long flag;
152
153 RT_TRACE((COMP_PS | COMP_RF),
154 "===>rtl92e_set_rf_state(): StateToSet(%d)\n", StateToSet);
155
156 while (true) {
157 spin_lock_irqsave(&priv->rf_ps_lock, flag);
158 if (priv->RFChangeInProgress) {
159 spin_unlock_irqrestore(&priv->rf_ps_lock, flag);
160 RT_TRACE((COMP_PS | COMP_RF),
161 "rtl92e_set_rf_state(): RF Change in progress! Wait to set..StateToSet(%d).\n",
162 StateToSet);
163
164 while (priv->RFChangeInProgress) {
165 RFWaitCounter++;
166 RT_TRACE((COMP_PS | COMP_RF),
167 "rtl92e_set_rf_state(): Wait 1 ms (%d times)...\n",
168 RFWaitCounter);
169 mdelay(1);
170
171 if (RFWaitCounter > 100) {
172 netdev_warn(dev,
173 "%s(): Timeout waiting for RF change.\n",
174 __func__);
175 return false;
176 }
177 }
178 } else {
179 priv->RFChangeInProgress = true;
180 spin_unlock_irqrestore(&priv->rf_ps_lock, flag);
181 break;
182 }
183 }
184
185 rtState = priv->rtllib->eRFPowerState;
186
187 switch (StateToSet) {
188 case eRfOn:
189 priv->rtllib->RfOffReason &= (~ChangeSource);
190
191 if ((ChangeSource == RF_CHANGE_BY_HW) && priv->bHwRadioOff)
192 priv->bHwRadioOff = false;
193
194 if (!priv->rtllib->RfOffReason) {
195 priv->rtllib->RfOffReason = 0;
196 bActionAllowed = true;
197
198 if (rtState == eRfOff &&
199 ChangeSource >= RF_CHANGE_BY_HW)
200 bConnectBySSID = true;
201 } else {
202 RT_TRACE((COMP_PS | COMP_RF),
203 "rtl92e_set_rf_state - eRfon reject pMgntInfo->RfOffReason= 0x%x, ChangeSource=0x%X\n",
204 priv->rtllib->RfOffReason, ChangeSource);
205 }
206
207 break;
208
209 case eRfOff:
210
211 if ((priv->rtllib->iw_mode == IW_MODE_INFRA) ||
212 (priv->rtllib->iw_mode == IW_MODE_ADHOC)) {
213 if ((priv->rtllib->RfOffReason > RF_CHANGE_BY_IPS) ||
214 (ChangeSource > RF_CHANGE_BY_IPS)) {
215 if (ieee->state == RTLLIB_LINKED)
216 priv->blinked_ingpio = true;
217 else
218 priv->blinked_ingpio = false;
219 rtllib_MgntDisconnect(priv->rtllib,
> 220 WLAN_REASON_DISASSOC_STA_
> 221 HAS_LEFT);
222 }
223 }
224 if ((ChangeSource == RF_CHANGE_BY_HW) && !priv->bHwRadioOff)

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[unhandled content-type:application/gzip]
\
 
 \ /
  Last update: 2017-03-09 19:53    [W:0.348 / U:0.200 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site