* [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used @ 2015-01-31 15:12 Rickard Strandqvist 2015-02-01 23:01 ` Jes Sorensen 0 siblings, 1 reply; 5+ messages in thread From: Rickard Strandqvist @ 2015-01-31 15:12 UTC (permalink / raw) To: Larry Finger, Jes Sorensen Cc: Rickard Strandqvist, Greg Kroah-Hartman, Joe Perches, linux-wireless, devel, linux-kernel Variable was assigned a value that was never used. I have also removed all the code that thereby serves no purpose. This was found using a static code analysis program called cppcheck Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> --- drivers/staging/rtl8723au/hal/usb_ops_linux.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8723au/hal/usb_ops_linux.c b/drivers/staging/rtl8723au/hal/usb_ops_linux.c index a6d16ad..4ae0a8a 100644 --- a/drivers/staging/rtl8723au/hal/usb_ops_linux.c +++ b/drivers/staging/rtl8723au/hal/usb_ops_linux.c @@ -26,11 +26,10 @@ u8 rtl8723au_read8(struct rtw_adapter *padapter, u16 addr) { struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); struct usb_device *udev = pdvobjpriv->pusbdev; - int len; u8 data; mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, addr, 0, &pdvobjpriv->usb_buf.val8, sizeof(data), RTW_USB_CONTROL_MSG_TIMEOUT); @@ -45,11 +44,10 @@ u16 rtl8723au_read16(struct rtw_adapter *padapter, u16 addr) { struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); struct usb_device *udev = pdvobjpriv->pusbdev; - int len; u16 data; mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, addr, 0, &pdvobjpriv->usb_buf.val16, sizeof(data), RTW_USB_CONTROL_MSG_TIMEOUT); @@ -64,11 +62,10 @@ u32 rtl8723au_read32(struct rtw_adapter *padapter, u16 addr) { struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); struct usb_device *udev = pdvobjpriv->pusbdev; - int len; u32 data; mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, addr, 0, &pdvobjpriv->usb_buf.val32, sizeof(data), RTW_USB_CONTROL_MSG_TIMEOUT); -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used 2015-01-31 15:12 [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used Rickard Strandqvist @ 2015-02-01 23:01 ` Jes Sorensen 2015-02-01 23:17 ` Rickard Strandqvist 0 siblings, 1 reply; 5+ messages in thread From: Jes Sorensen @ 2015-02-01 23:01 UTC (permalink / raw) To: Rickard Strandqvist Cc: Larry Finger, Greg Kroah-Hartman, Joe Perches, linux-wireless, devel, linux-kernel Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> writes: > Variable was assigned a value that was never used. > I have also removed all the code that thereby serves no purpose. > > This was found using a static code analysis program called cppcheck > > Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> > --- > drivers/staging/rtl8723au/hal/usb_ops_linux.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/staging/rtl8723au/hal/usb_ops_linux.c b/drivers/staging/rtl8723au/hal/usb_ops_linux.c > index a6d16ad..4ae0a8a 100644 > --- a/drivers/staging/rtl8723au/hal/usb_ops_linux.c > +++ b/drivers/staging/rtl8723au/hal/usb_ops_linux.c > @@ -26,11 +26,10 @@ u8 rtl8723au_read8(struct rtw_adapter *padapter, u16 addr) > { > struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); > struct usb_device *udev = pdvobjpriv->pusbdev; > - int len; > u8 data; > > mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); > - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), > + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), > REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, > addr, 0, &pdvobjpriv->usb_buf.val8, sizeof(data), > RTW_USB_CONTROL_MSG_TIMEOUT); > @@ -45,11 +44,10 @@ u16 rtl8723au_read16(struct rtw_adapter *padapter, u16 addr) > { > struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); > struct usb_device *udev = pdvobjpriv->pusbdev; > - int len; > u16 data; > > mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); > - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), > + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), > REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, > addr, 0, &pdvobjpriv->usb_buf.val16, sizeof(data), > RTW_USB_CONTROL_MSG_TIMEOUT); > @@ -64,11 +62,10 @@ u32 rtl8723au_read32(struct rtw_adapter *padapter, u16 addr) > { > struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); > struct usb_device *udev = pdvobjpriv->pusbdev; > - int len; > u32 data; > > mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); > - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), > + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), > REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, > addr, 0, &pdvobjpriv->usb_buf.val32, sizeof(data), > RTW_USB_CONTROL_MSG_TIMEOUT); Again, fix the formatting if you want to make this change. NACK Jes ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used 2015-02-01 23:01 ` Jes Sorensen @ 2015-02-01 23:17 ` Rickard Strandqvist 2015-02-01 23:28 ` Julian Calaby 0 siblings, 1 reply; 5+ messages in thread From: Rickard Strandqvist @ 2015-02-01 23:17 UTC (permalink / raw) To: Jes Sorensen Cc: Larry Finger, Greg Kroah-Hartman, Joe Perches, linux-wireless@vger.kernel.org, devel@driverdev.osuosl.org, Linux Kernel Mailing List 2015-02-02 0:01 GMT+01:00 Jes Sorensen <Jes.Sorensen@redhat.com>: > Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> writes: >> Variable was assigned a value that was never used. >> I have also removed all the code that thereby serves no purpose. >> >> This was found using a static code analysis program called cppcheck >> >> Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> >> --- >> drivers/staging/rtl8723au/hal/usb_ops_linux.c | 9 +++------ >> 1 file changed, 3 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/staging/rtl8723au/hal/usb_ops_linux.c b/drivers/staging/rtl8723au/hal/usb_ops_linux.c >> index a6d16ad..4ae0a8a 100644 >> --- a/drivers/staging/rtl8723au/hal/usb_ops_linux.c >> +++ b/drivers/staging/rtl8723au/hal/usb_ops_linux.c >> @@ -26,11 +26,10 @@ u8 rtl8723au_read8(struct rtw_adapter *padapter, u16 addr) >> { >> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >> struct usb_device *udev = pdvobjpriv->pusbdev; >> - int len; >> u8 data; >> >> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >> addr, 0, &pdvobjpriv->usb_buf.val8, sizeof(data), >> RTW_USB_CONTROL_MSG_TIMEOUT); >> @@ -45,11 +44,10 @@ u16 rtl8723au_read16(struct rtw_adapter *padapter, u16 addr) >> { >> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >> struct usb_device *udev = pdvobjpriv->pusbdev; >> - int len; >> u16 data; >> >> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >> addr, 0, &pdvobjpriv->usb_buf.val16, sizeof(data), >> RTW_USB_CONTROL_MSG_TIMEOUT); >> @@ -64,11 +62,10 @@ u32 rtl8723au_read32(struct rtw_adapter *padapter, u16 addr) >> { >> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >> struct usb_device *udev = pdvobjpriv->pusbdev; >> - int len; >> u32 data; >> >> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >> addr, 0, &pdvobjpriv->usb_buf.val32, sizeof(data), >> RTW_USB_CONTROL_MSG_TIMEOUT); > > Again, fix the formatting if you want to make this change. > > NACK > > Jes Hi I am sorry, please ignore this. I will not send any more patches. Kind regards Rickard Strandqvist ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used 2015-02-01 23:17 ` Rickard Strandqvist @ 2015-02-01 23:28 ` Julian Calaby 2015-02-01 23:36 ` Jes Sorensen 0 siblings, 1 reply; 5+ messages in thread From: Julian Calaby @ 2015-02-01 23:28 UTC (permalink / raw) To: Rickard Strandqvist Cc: Jes Sorensen, Larry Finger, Greg Kroah-Hartman, Joe Perches, linux-wireless@vger.kernel.org, devel@driverdev.osuosl.org, Linux Kernel Mailing List Hi Rickard, On Mon, Feb 2, 2015 at 10:17 AM, Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> wrote: > 2015-02-02 0:01 GMT+01:00 Jes Sorensen <Jes.Sorensen@redhat.com>: >> Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> writes: >>> Variable was assigned a value that was never used. >>> I have also removed all the code that thereby serves no purpose. >>> >>> This was found using a static code analysis program called cppcheck >>> >>> Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> >>> --- >>> drivers/staging/rtl8723au/hal/usb_ops_linux.c | 9 +++------ >>> 1 file changed, 3 insertions(+), 6 deletions(-) >>> >>> diff --git a/drivers/staging/rtl8723au/hal/usb_ops_linux.c b/drivers/staging/rtl8723au/hal/usb_ops_linux.c >>> index a6d16ad..4ae0a8a 100644 >>> --- a/drivers/staging/rtl8723au/hal/usb_ops_linux.c >>> +++ b/drivers/staging/rtl8723au/hal/usb_ops_linux.c >>> @@ -26,11 +26,10 @@ u8 rtl8723au_read8(struct rtw_adapter *padapter, u16 addr) >>> { >>> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >>> struct usb_device *udev = pdvobjpriv->pusbdev; >>> - int len; >>> u8 data; >>> >>> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >>> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >>> addr, 0, &pdvobjpriv->usb_buf.val8, sizeof(data), >>> RTW_USB_CONTROL_MSG_TIMEOUT); >>> @@ -45,11 +44,10 @@ u16 rtl8723au_read16(struct rtw_adapter *padapter, u16 addr) >>> { >>> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >>> struct usb_device *udev = pdvobjpriv->pusbdev; >>> - int len; >>> u16 data; >>> >>> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >>> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >>> addr, 0, &pdvobjpriv->usb_buf.val16, sizeof(data), >>> RTW_USB_CONTROL_MSG_TIMEOUT); >>> @@ -64,11 +62,10 @@ u32 rtl8723au_read32(struct rtw_adapter *padapter, u16 addr) >>> { >>> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >>> struct usb_device *udev = pdvobjpriv->pusbdev; >>> - int len; >>> u32 data; >>> >>> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >>> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >>> addr, 0, &pdvobjpriv->usb_buf.val32, sizeof(data), >>> RTW_USB_CONTROL_MSG_TIMEOUT); >> >> Again, fix the formatting if you want to make this change. >> >> NACK >> >> Jes > > > Hi > > I am sorry, please ignore this. > I will not send any more patches. The issue isn't you sending patches, the issue is that you don't spend enough time on them. As I understand it, the formatting for function calls is that if the arguments flow onto a new line, they start in the column after the opening bracket of the function call. I.e. do_something(arg, arg, arg, /* 13 cols */arg, arg, arg); (Using tabs and spaces instead of the comment of course) In this case, you're dropping 6 characters from the line with the function call. You then need to re-flow the arguments to suit. Thanks, -- Julian Calaby Email: julian.calaby@gmail.com Profile: http://www.google.com/profiles/julian.calaby/ ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used 2015-02-01 23:28 ` Julian Calaby @ 2015-02-01 23:36 ` Jes Sorensen 0 siblings, 0 replies; 5+ messages in thread From: Jes Sorensen @ 2015-02-01 23:36 UTC (permalink / raw) To: Julian Calaby Cc: Rickard Strandqvist, Larry Finger, Greg Kroah-Hartman, Joe Perches, linux-wireless@vger.kernel.org, devel@driverdev.osuosl.org, Linux Kernel Mailing List Julian Calaby <julian.calaby@gmail.com> writes: > Hi Rickard, > > On Mon, Feb 2, 2015 at 10:17 AM, Rickard Strandqvist > <rickard_strandqvist@spectrumdigital.se> wrote: >> 2015-02-02 0:01 GMT+01:00 Jes Sorensen <Jes.Sorensen@redhat.com>: >>> Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> writes: >>>> Variable was assigned a value that was never used. >>>> I have also removed all the code that thereby serves no purpose. >>>> >>>> This was found using a static code analysis program called cppcheck >>>> >>>> Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> >>>> --- >>>> drivers/staging/rtl8723au/hal/usb_ops_linux.c | 9 +++------ >>>> 1 file changed, 3 insertions(+), 6 deletions(-) >>>> >>>> diff --git a/drivers/staging/rtl8723au/hal/usb_ops_linux.c b/drivers/staging/rtl8723au/hal/usb_ops_linux.c >>>> index a6d16ad..4ae0a8a 100644 >>>> --- a/drivers/staging/rtl8723au/hal/usb_ops_linux.c >>>> +++ b/drivers/staging/rtl8723au/hal/usb_ops_linux.c >>>> @@ -26,11 +26,10 @@ u8 rtl8723au_read8(struct rtw_adapter *padapter, u16 addr) >>>> { >>>> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >>>> struct usb_device *udev = pdvobjpriv->pusbdev; >>>> - int len; >>>> u8 data; >>>> >>>> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >>>> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>>> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>>> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >>>> addr, 0, &pdvobjpriv->usb_buf.val8, sizeof(data), >>>> RTW_USB_CONTROL_MSG_TIMEOUT); >>>> @@ -45,11 +44,10 @@ u16 rtl8723au_read16(struct rtw_adapter *padapter, u16 addr) >>>> { >>>> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >>>> struct usb_device *udev = pdvobjpriv->pusbdev; >>>> - int len; >>>> u16 data; >>>> >>>> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >>>> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>>> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>>> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >>>> addr, 0, &pdvobjpriv->usb_buf.val16, sizeof(data), >>>> RTW_USB_CONTROL_MSG_TIMEOUT); >>>> @@ -64,11 +62,10 @@ u32 rtl8723au_read32(struct rtw_adapter *padapter, u16 addr) >>>> { >>>> struct dvobj_priv *pdvobjpriv = adapter_to_dvobj(padapter); >>>> struct usb_device *udev = pdvobjpriv->pusbdev; >>>> - int len; >>>> u32 data; >>>> >>>> mutex_lock(&pdvobjpriv->usb_vendor_req_mutex); >>>> - len = usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>>> + usb_control_msg(udev, usb_rcvctrlpipe(udev, 0), >>>> REALTEK_USB_VENQT_CMD_REQ, REALTEK_USB_VENQT_READ, >>>> addr, 0, &pdvobjpriv->usb_buf.val32, sizeof(data), >>>> RTW_USB_CONTROL_MSG_TIMEOUT); >>> >>> Again, fix the formatting if you want to make this change. >>> >>> NACK >>> >>> Jes >> >> >> Hi >> >> I am sorry, please ignore this. >> I will not send any more patches. > > The issue isn't you sending patches, the issue is that you don't spend > enough time on them. > > As I understand it, the formatting for function calls is that if the > arguments flow onto a new line, they start in the column after the > opening bracket of the function call. > > I.e. > > do_something(arg, arg, arg, > /* 13 cols */arg, arg, arg); > > (Using tabs and spaces instead of the comment of course) > > In this case, you're dropping 6 characters from the line with the > function call. You then need to re-flow the arguments to suit. This is correct, of all the automated patches that flow in the staging tree, I'd see these are some of the more useful ones, because they get rid of unused clutter, making it easier to follow the actual code. However, breaking coding style using automated tools is just going to result in someone else having to go back and fix that up in a second patch. As Julian points out, please spend the little extra effort so we get it right in the first patch. Cheers, Jes ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-02-01 23:36 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-01-31 15:12 [PATCH] staging: rtl8723au: hal: usb_ops_linux: Removed variables that is never used Rickard Strandqvist 2015-02-01 23:01 ` Jes Sorensen 2015-02-01 23:17 ` Rickard Strandqvist 2015-02-01 23:28 ` Julian Calaby 2015-02-01 23:36 ` Jes Sorensen
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).