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? 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.
Thanks & Regards,
Hari Prasath
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
