Commit db613710 authored Oct 27, 2012 by Mauro Carvalho Chehab

[media] cx88: get rid of a warning at dprintk() macro

drivers/media/pci/cx88/cx88-mpeg.c: In function 'cx8802_mpeg_irq':
drivers/media/pci/cx88/cx88-mpeg.c:419:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-mpeg.c: In function 'cx8802_irq':
drivers/media/pci/cx88/cx88-mpeg.c:453:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-alsa.c: In function 'snd_cx88_create':
drivers/media/pci/cx88/cx88-alsa.c:818:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-alsa.c:837:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-alsa.c: In function 'cx88_audio_initdev':
drivers/media/pci/cx88/cx88-alsa.c:912:2: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c: In function 'blackbird_mbox_func':
drivers/media/pci/cx88/cx88-blackbird.c:327:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:333:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:360:4: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c: In function 'blackbird_find_mailbox':
drivers/media/pci/cx88/cx88-blackbird.c:421:2: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c: In function 'blackbird_load_firmware':
drivers/media/pci/cx88/cx88-blackbird.c:444:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:451:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:453:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:459:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:466:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:487:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:492:2: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:503:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c: In function 'blackbird_initialize_codec':
drivers/media/pci/cx88/cx88-blackbird.c:560:4: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:566:4: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]
drivers/media/pci/cx88/cx88-blackbird.c:569:3: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

parent 34206439

drivers/media/pci/cx88/cx88-alsa.c

+9 −5

Original line number	Original line	Diff line number	Diff line
	@@ -45,11 +45,15 @@
	#include "cx88.h"		#include "cx88.h"
	#include "cx88-reg.h"		#include "cx88-reg.h"

	#define dprintk(level,fmt, arg...) if (debug >= level) \		#define dprintk(level, fmt, arg...) do { \
	printk(KERN_INFO "%s/1: " fmt, chip->core->name , ## arg)		if (debug + 1 > level) \
			printk(KERN_INFO "%s/1: " fmt, chip->core->name , ## arg);\
	#define dprintk_core(level,fmt, arg...) if (debug >= level) \		} while(0)
	printk(KERN_DEBUG "%s/1: " fmt, chip->core->name , ## arg)
			#define dprintk_core(level, fmt, arg...) do { \
			if (debug + 1 > level) \
			printk(KERN_DEBUG "%s/1: " fmt, chip->core->name , ## arg);\
			} while(0)

	/****************************************************************************		/****************************************************************************
	Data type declarations - Can be moded to a header file later		Data type declarations - Can be moded to a header file later

drivers/media/pci/cx88/cx88-blackbird.c

+4 −3

Original line number	Original line	Diff line number	Diff line
	@@ -53,9 +53,10 @@ static unsigned int debug;
	module_param(debug,int,0644);		module_param(debug,int,0644);
	MODULE_PARM_DESC(debug,"enable debug messages [blackbird]");		MODULE_PARM_DESC(debug,"enable debug messages [blackbird]");

	#define dprintk(level,fmt, arg...) if (debug >= level) \		#define dprintk(level, fmt, arg...) do { \
	printk(KERN_DEBUG "%s/2-bb: " fmt, dev->core->name , ## arg)		if (debug + 1 > level) \
			printk(KERN_DEBUG "%s/2-bb: " fmt, dev->core->name , ## arg); \
			} while(0)

	/* ------------------------------------------------------------------ */		/* ------------------------------------------------------------------ */

drivers/media/pci/cx88/cx88-mpeg.c

+9 −5

Original line number	Original line	Diff line number	Diff line
	@@ -45,11 +45,15 @@ static unsigned int debug;
	module_param(debug,int,0644);		module_param(debug,int,0644);
	MODULE_PARM_DESC(debug,"enable debug messages [mpeg]");		MODULE_PARM_DESC(debug,"enable debug messages [mpeg]");

	#define dprintk(level,fmt, arg...) if (debug >= level) \		#define dprintk(level, fmt, arg...) do { \
	printk(KERN_DEBUG "%s/2-mpeg: " fmt, dev->core->name, ## arg)		if (debug + 1 > level) \
			printk(KERN_DEBUG "%s/2-mpeg: " fmt, dev->core->name, ## arg); \
	#define mpeg_dbg(level,fmt, arg...) if (debug >= level) \		} while(0)
	printk(KERN_DEBUG "%s/2-mpeg: " fmt, core->name, ## arg)
			#define mpeg_dbg(level, fmt, arg...) do { \
			if (debug + 1 > level) \
			printk(KERN_DEBUG "%s/2-mpeg: " fmt, core->name, ## arg); \
			} while(0)

	#if defined(CONFIG_MODULES) && defined(MODULE)		#if defined(CONFIG_MODULES) && defined(MODULE)
	static void request_module_async(struct work_struct *work)		static void request_module_async(struct work_struct *work)