Re: linux-next: Tree for Aug 14 (media/dvb/dvb-usb-v2/anysee)

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

 



On 08/14/2012 08:54 PM, Randy Dunlap wrote:
On 08/13/2012 08:55 PM, Stephen Rothwell wrote:

Hi all,

Changes since 20120813:




on x86_64:

In file included from drivers/media/dvb/dvb-usb-v2/anysee.c:34:0:
drivers/media/dvb/dvb-usb-v2/anysee.h:51:0: warning: "debug_dump" redefined
drivers/media/dvb/dvb-usb-v2/anysee.h:47:0: note: this is the location of the previous definition

Thank you Randy for the report. Attached patch fix it.

Unfortunately it will not apply for current next as Mauro reorganized drivers/media/ today and this one is top of that. I will make another patch to change whole driver to use Kernel dev_* debugging and sent those linus-media. I hope those are in next quite soon.

regards
Antti

--
http://palosaari.fi/
>From cbd18c187db27686f2fd14348255713defe1d90a Mon Sep 17 00:00:00 2001
From: Antti Palosaari <crope@xxxxxx>
Date: Tue, 14 Aug 2012 21:23:01 +0300
Subject: [PATCH] anysee: fix compiler warning

debug_dump macro was defined twice when CONFIG_DVB_USB_DEBUG was
not set. Move debug_dump macro to correct place.

Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
Signed-off-by: Antti Palosaari <crope@xxxxxx>
---
 drivers/media/usb/dvb-usb-v2/anysee.h | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/media/usb/dvb-usb-v2/anysee.h b/drivers/media/usb/dvb-usb-v2/anysee.h
index dc40dcf..834dc12 100644
--- a/drivers/media/usb/dvb-usb-v2/anysee.h
+++ b/drivers/media/usb/dvb-usb-v2/anysee.h
@@ -41,19 +41,18 @@
 #ifdef CONFIG_DVB_USB_DEBUG
 #define dprintk(var, level, args...) \
 	do { if ((var & level)) printk(args); } while (0)
-#define DVB_USB_DEBUG_STATUS
-#else
-#define dprintk(args...)
-#define debug_dump(b, l, func)
-#define DVB_USB_DEBUG_STATUS " (debugging is not enabled)"
-#endif
-
 #define debug_dump(b, l, func) {\
 	int loop_; \
 	for (loop_ = 0; loop_ < l; loop_++) \
 		func("%02x ", b[loop_]); \
 	func("\n");\
 }
+#define DVB_USB_DEBUG_STATUS
+#else
+#define dprintk(args...)
+#define debug_dump(b, l, func)
+#define DVB_USB_DEBUG_STATUS " (debugging is not enabled)"
+#endif
 
 #define deb_info(args...) dprintk(dvb_usb_anysee_debug, 0x01, args)
 #define deb_xfer(args...) dprintk(dvb_usb_anysee_debug, 0x02, args)
-- 
1.7.11.2


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux