Re: [PATCH] staging: wlan-ng: remove unused debug macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 16, 2015 at 05:09:35AM +0000, Gujulan Elango, Hari Prasath (H.) wrote:
> On Mon, Jun 15, 2015 at 09:55:15PM -0700, gregkh@xxxxxxxxxxxxxxxxxxx wrote:
> > On Mon, Jun 15, 2015 at 11:44:24AM +0000, Gujulan Elango, Hari Prasath (H.) wrote:
> > > From: Hari Prasath Gujulan Elango <hgujulan@xxxxxxxxxxx>
> > > 
> > > Thi patch removes ununsed debug macro and its references in this
> > > file.The macro is used around pr_debug which itself can be dynamically
> > > enabled/disabled.
> > > 
> > > Signed-off-by: Hari Prasath Gujulan Elango <hgujulan@xxxxxxxxxxx>
> > > ---
> > >  drivers/staging/wlan-ng/p80211wep.c | 7 -------
> > >  1 file changed, 7 deletions(-)
> > > 
> > > diff --git a/drivers/staging/wlan-ng/p80211wep.c b/drivers/staging/wlan-ng/p80211wep.c
> > > index c4fabad..f1afd0e 100644
> > > --- a/drivers/staging/wlan-ng/p80211wep.c
> > > +++ b/drivers/staging/wlan-ng/p80211wep.c
> > > @@ -53,7 +53,6 @@
> > >  #include <linux/random.h>
> > >  #include <linux/kernel.h>
> > >  
> > > -/* #define WEP_DEBUG	*/
> > >  
> > >  #include "p80211hdr.h"
> > >  #include "p80211types.h"
> > > @@ -133,10 +132,8 @@ int wep_change_key(wlandevice_t *wlandev, int keynum, u8 *key, int keylen)
> > >  	if (keynum >= NUM_WEPKEYS)
> > >  		return -1;
> > >  
> > > -#ifdef WEP_DEBUG
> > >  	pr_debug("WEP key %d len %d = %*phC\n", keynum, keylen,
> > >  			  8, key);
> > > -#endif
> > 
> > You just added this to the driver now, not what you were saying you were
> > doing :(
> 
> Greg,if my understanding is correct,do you mean to say that by 
> removing the MACRO WEP_DEBUG which is not defined,this patch adds the
> dynamic debug statements in the driver?

Yes.

> As we know that pr_debug is
> something that can be dynamically enable/disable during runtime,then
> why to have a MACRO which can enable/disable statically ?Please 
> correct me I am wrong.

Why not just delete these as obviously they are not being used?

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux