Message ID | 1454636118-9943-5-git-send-email-chaehyun.lim@gmail.com (mailing list archive) |
---|---|
State | Not Applicable |
Delegated to: | Kalle Valo |
Headers | show |
On Fri, Feb 05, 2016 at 10:35:15AM +0900, Chaehyun Lim wrote: > wilc_send_config_pkt is returned 0 or -ETIMEDOUT according to return > value of wilc_wlan_cfg_get or wilc_wlan_cfg_set. > It is better to use int type to represent linux standard error code. > > Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> > --- > drivers/staging/wilc1000/coreconfigurator.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) This, and the remaining patches in the series, don't apply anymore due to patches I took right before your series. Can you refresh these and resend? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/wilc1000/coreconfigurator.c index 49ae9b1..323a0cd 100644 --- a/drivers/staging/wilc1000/coreconfigurator.c +++ b/drivers/staging/wilc1000/coreconfigurator.c @@ -449,10 +449,11 @@ s32 wilc_dealloc_assoc_resp_info(tstrConnectRespInfo *pstrConnectRespInfo) * @date 1 Mar 2012 * @version 1.0 */ -s32 wilc_send_config_pkt(struct wilc *wilc, u8 mode, struct wid *wids, +int wilc_send_config_pkt(struct wilc *wilc, u8 mode, struct wid *wids, u32 count, u32 drv) { - s32 counter = 0, ret = 0; + s32 counter = 0; + int ret = 0; if (mode == GET_CFG) { for (counter = 0; counter < count; counter++) {
wilc_send_config_pkt is returned 0 or -ETIMEDOUT according to return value of wilc_wlan_cfg_get or wilc_wlan_cfg_set. It is better to use int type to represent linux standard error code. Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com> --- drivers/staging/wilc1000/coreconfigurator.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)