Re: [PATCH 4/4] staging: brcm80211: remove nested #ifdef BCMDBG

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

 



Greg,

Please discard this patch.

Thanks,
AvS

On Thu, 10 Feb 2011 15:15:32 +0100, Arend van Spriel <arend@xxxxxxxxxxxx> wrote:

The function wlc_print_txstatus has conditional implementation
controlled by BCMDBG so the call to wlc_print_txs_status does
not need to be surrounded by #ifdef BCMDBG as well.

Reviewed-by: Brett Rudley <brudley@xxxxxxxxxxxx>
Reviewed-by: Roland Vossen <rvossen@xxxxxxxxxxxx>
Signed-off-by: Arend van Spriel <arend@xxxxxxxxxxxx>
---
 drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c b/drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c
index 701750f..c68288e 100644
--- a/drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c
+++ b/drivers/staging/brcm80211/brcmsmac/wlc_mac80211.c
@@ -4736,9 +4736,7 @@ void wlc_print_txstatus(tx_status_t *txs)
 	printf("FrameID: %04x   ", txs->frameid);
 	printf("TxStatus: %04x", s);
 	printf("\n");
-#ifdef BCMDBG
 	wlc_print_txs_status(s);
-#endif
 	printf("LastTxTime: %04x ", txs->lasttxtime);
 	printf("Seq: %04x ", txs->sequence);
 	printf("PHYTxStatus: %04x ", txs->phyerr);


--
"The most merciful thing in the world, I think, is the inability of the human
mind to correlate all its contents." - "The Call of Cthulhu"

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/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