On Thu, Apr 30, 2015 at 02:09:13PM -0500, Larry Finger wrote: > On 04/30/2015 12:50 PM, Gujulan Elango, Hari Prasath (H.) wrote: > > > >From: Larry Finger <larry.fin...@gmail.com> on behalf of Larry Finger > ><larry.fin...@lwfinger.net> > >Sent: Thursday, April 30, 2015 11:03 PM > >To: Gujulan Elango, Hari Prasath (H.); Dan Carpenter > >Cc: de...@driverdev.osuosl.org; julia.law...@lip6.fr; > >gre...@linuxfoundation.org; wlan...@realtek.com; > >mateusz.kulikow...@gmail.com; Babu, Viswanathan (V.) > >Subject: Re: [PATCH] staging: rtl8192e: fix wrong assignment > > > >On 04/30/2015 11:59 AM, Gujulan Elango, Hari Prasath (H.) wrote: > >>From: Dan Carpenter <dan.carpen...@oracle.com> > >>Sent: Thursday, April 30, 2015 6:33 PM > >>To: Gujulan Elango, Hari Prasath (H.) > >>Cc: de...@driverdev.osuosl.org; julia.law...@lip6.fr; > >>gre...@linuxfoundation.org; wlan...@realtek.com; > >>mateusz.kulikow...@gmail.com; Babu, Viswanathan (V.); > >>larry.fin...@lwfinger.net > >>Subject: Re: [PATCH] staging: rtl8192e: fix wrong assignment > >> > >>On Thu, Apr 30, 2015 at 12:06:28PM +0000, Gujulan Elango, Hari Prasath (H.) > >>wrote: > >>>This patch addresses a spatch warning on assigning a negative > >>>value to a unsigned integer.Similar patch has been submitted by > >>>Larry Finger earlier to silence the same spatch warning in another > >>>file. > >> > >>What's the git hash and title for Larry's patch? > >> > >>>The logic is the negative number to an unsigned quantity is > >>>fixed by adding 256 to -98 to get the equivalent negative number as > >>>per Larry Finger. > >> > >>>>It was a spatch warning.I am not sure if that change by Larry went in,but > >>>>here's the link where I found a submission by Larry. > >>http://permalink.gmane.org/gmane.linux.kernel.wireless.general/113125 > > > >If you actually read that patch, you will see that the "fix" was to remove > >the > >noise member initialization. The git hash, title, and commit message for the > >previous change was as follows: > > > >commit 354d0f3c40fb40193213e40f3177ff528798ca8d > >Author: Larry Finger <larry.fin...@lwfinger.net> > >Date: Wed Sep 25 12:57:47 2013 -0500 > > > > rtlwifi: Fix smatch warnings in usb.c > > > > Smatch displays the following: > > CHECK drivers/net/wireless/rtlwifi/usb.c > > drivers/net/wireless/rtlwifi/usb.c:458 _rtl_usb_rx_process_agg() warn: > >assigning (-98) to unsigned variable 'stats.noise' > > drivers/net/wireless/rtlwifi/usb.c:503 _rtl_usb_rx_process_noagg() > > warn: > >assigning (-98) to unsigned variable 'stats.noise' > > drivers/net/wireless/rtlwifi/usb.c:596 _rtl_rx_get_padding() info: > > ignoring > >unreachable code. > > drivers/net/wireless/rtlwifi/usb.c:596 _rtl_rx_get_padding() info: > > ignoring > >unreachable code. > > > > The variable 'stats.noise' is not used, thus the initializers are > > removed. > > The unreachable code info is fixed by including the appropriate > > section inside > > #ifdef .. #endif constructions. > > > > Signed-off-by: Larry Finger <larry.fin...@lwfinger.net> > > Signed-off-by: John W. Linville <linvi...@tuxdriver.com> > > > >If you are going to be submitting kernel patches, then I suggest a tutorial > >on > >the usage of git so that you can find previous commits. > > > >Unlike the rtlwifi drivers, this one appears to set the noise value, thus a > >simple removal of the initialization is not appropriate. Your fix of setting > >the > >value to 256-98 seems to be correct. > > > >>>>I did see the git log related to this file. I was not sure if I could > >>>>remove the variable right away assuming the variable is used somewhere. > >>>>Hence I sticked with just silencing the smatch warning. > > > > > > > > Obviously, you cannot remove the variable now. > > Please fix your mailer. The most recent material to which you were > replying was not indented any levels, but the "new" material was 3 > levels deep. It is extremely difficult to follow the thread. > > Larry >
Please exuse me for the replies being screwed up. I just figured out how to reply to e-mails from Mutt.Hope you would be getting cleaner replies from now on. _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel