aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/media
diff options
context:
space:
mode:
authorIngo Molnar <mingo@elte.hu>2009-09-21 14:23:45 -0400
committerMauro Carvalho Chehab <mchehab@redhat.com>2009-09-22 17:58:17 -0400
commitb1912a85b54c27738afe1c4fa069df02d3316f0c (patch)
tree07ad08fa40efa233cd3cff5ce53c81711d16a402 /drivers/media
parent7fa07729e439a6184bd824746d06a49cca553f15 (diff)
media: video: Fix build in saa7164
-tip testing found that the x86 build (64-bit allyesconfig) fails due to: LD vmlinux.o drivers/built-in.o:(.bss+0x4b648): multiple definition of `debug' arch/x86/built-in.o:(.kprobes.text+0x88): first defined here ld: Warning: size of symbol `debug' changed from 90 in arch/x86/built-in.o to 4 in drivers/built-in.o make: *** [vmlinux.o] Error 1 This is because recent saa7164 changes introduced a global symbol named 'debug'. The x86 platform code already defines a 'debug' symbol. (which is named in a too generic way as well - but it can be used nicely to weed out too generic symbols in drivers ;-) Rename it to saa_debug. [mchehab@redhat.com: use module_param_named to preserve old name] Signed-off-by: Ingo Molnar <mingo@elte.hu> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media')
-rw-r--r--drivers/media/video/saa7164/saa7164-api.c8
-rw-r--r--drivers/media/video/saa7164/saa7164-cmd.c2
-rw-r--r--drivers/media/video/saa7164/saa7164-core.c6
-rw-r--r--drivers/media/video/saa7164/saa7164.h4
4 files changed, 10 insertions, 10 deletions
diff --git a/drivers/media/video/saa7164/saa7164-api.c b/drivers/media/video/saa7164/saa7164-api.c
index bb6df1b276b..6f094a96ac8 100644
--- a/drivers/media/video/saa7164/saa7164-api.c
+++ b/drivers/media/video/saa7164/saa7164-api.c
@@ -415,7 +415,7 @@ int saa7164_api_enum_subdevs(struct saa7164_dev *dev)
415 goto out; 415 goto out;
416 } 416 }
417 417
418 if (debug & DBGLVL_API) 418 if (saa_debug & DBGLVL_API)
419 saa7164_dumphex16(dev, buf, (buflen/16)*16); 419 saa7164_dumphex16(dev, buf, (buflen/16)*16);
420 420
421 saa7164_api_dump_subdevs(dev, buf, buflen); 421 saa7164_api_dump_subdevs(dev, buf, buflen);
@@ -480,7 +480,7 @@ int saa7164_api_i2c_read(struct saa7164_i2c *bus, u8 addr, u32 reglen, u8 *reg,
480 480
481 dprintk(DBGLVL_API, "%s() len = %d bytes\n", __func__, len); 481 dprintk(DBGLVL_API, "%s() len = %d bytes\n", __func__, len);
482 482
483 if (debug & DBGLVL_I2C) 483 if (saa_debug & DBGLVL_I2C)
484 saa7164_dumphex16(dev, buf, 2 * 16); 484 saa7164_dumphex16(dev, buf, 2 * 16);
485 485
486 ret = saa7164_cmd_send(bus->dev, unitid, GET_CUR, 486 ret = saa7164_cmd_send(bus->dev, unitid, GET_CUR,
@@ -488,7 +488,7 @@ int saa7164_api_i2c_read(struct saa7164_i2c *bus, u8 addr, u32 reglen, u8 *reg,
488 if (ret != SAA_OK) 488 if (ret != SAA_OK)
489 printk(KERN_ERR "%s() error, ret(2) = 0x%x\n", __func__, ret); 489 printk(KERN_ERR "%s() error, ret(2) = 0x%x\n", __func__, ret);
490 else { 490 else {
491 if (debug & DBGLVL_I2C) 491 if (saa_debug & DBGLVL_I2C)
492 saa7164_dumphex16(dev, buf, sizeof(buf)); 492 saa7164_dumphex16(dev, buf, sizeof(buf));
493 memcpy(data, (buf + 2 * sizeof(u32) + reglen), datalen); 493 memcpy(data, (buf + 2 * sizeof(u32) + reglen), datalen);
494 } 494 }
@@ -548,7 +548,7 @@ int saa7164_api_i2c_write(struct saa7164_i2c *bus, u8 addr, u32 datalen,
548 *((u32 *)(buf + 1 * sizeof(u32))) = datalen - reglen; 548 *((u32 *)(buf + 1 * sizeof(u32))) = datalen - reglen;
549 memcpy((buf + 2 * sizeof(u32)), data, datalen); 549 memcpy((buf + 2 * sizeof(u32)), data, datalen);
550 550
551 if (debug & DBGLVL_I2C) 551 if (saa_debug & DBGLVL_I2C)
552 saa7164_dumphex16(dev, buf, sizeof(buf)); 552 saa7164_dumphex16(dev, buf, sizeof(buf));
553 553
554 ret = saa7164_cmd_send(bus->dev, unitid, SET_CUR, 554 ret = saa7164_cmd_send(bus->dev, unitid, SET_CUR,
diff --git a/drivers/media/video/saa7164/saa7164-cmd.c b/drivers/media/video/saa7164/saa7164-cmd.c
index e097f1a0969..c45966edc0c 100644
--- a/drivers/media/video/saa7164/saa7164-cmd.c
+++ b/drivers/media/video/saa7164/saa7164-cmd.c
@@ -250,7 +250,7 @@ int saa7164_cmd_wait(struct saa7164_dev *dev, u8 seqno)
250 unsigned long stamp; 250 unsigned long stamp;
251 int r; 251 int r;
252 252
253 if (debug >= 4) 253 if (saa_debug >= 4)
254 saa7164_bus_dump(dev); 254 saa7164_bus_dump(dev);
255 255
256 dprintk(DBGLVL_CMD, "%s(seqno=%d)\n", __func__, seqno); 256 dprintk(DBGLVL_CMD, "%s(seqno=%d)\n", __func__, seqno);
diff --git a/drivers/media/video/saa7164/saa7164-core.c b/drivers/media/video/saa7164/saa7164-core.c
index f0dbead188c..709affc3104 100644
--- a/drivers/media/video/saa7164/saa7164-core.c
+++ b/drivers/media/video/saa7164/saa7164-core.c
@@ -45,8 +45,8 @@ MODULE_LICENSE("GPL");
45 32 bus 45 32 bus
46 */ 46 */
47 47
48unsigned int debug; 48unsigned int saa_debug;
49module_param(debug, int, 0644); 49module_param_named(debug, saa_debug, int, 0644);
50MODULE_PARM_DESC(debug, "enable debug messages"); 50MODULE_PARM_DESC(debug, "enable debug messages");
51 51
52unsigned int waitsecs = 10; 52unsigned int waitsecs = 10;
@@ -653,7 +653,7 @@ static int __devinit saa7164_initdev(struct pci_dev *pci_dev,
653 printk(KERN_ERR "%s() Unsupported board detected, " 653 printk(KERN_ERR "%s() Unsupported board detected, "
654 "registering without firmware\n", __func__); 654 "registering without firmware\n", __func__);
655 655
656 dprintk(1, "%s() parameter debug = %d\n", __func__, debug); 656 dprintk(1, "%s() parameter debug = %d\n", __func__, saa_debug);
657 dprintk(1, "%s() parameter waitsecs = %d\n", __func__, waitsecs); 657 dprintk(1, "%s() parameter waitsecs = %d\n", __func__, waitsecs);
658 658
659fail_fw: 659fail_fw:
diff --git a/drivers/media/video/saa7164/saa7164.h b/drivers/media/video/saa7164/saa7164.h
index 6753008a9c9..42660b546f0 100644
--- a/drivers/media/video/saa7164/saa7164.h
+++ b/drivers/media/video/saa7164/saa7164.h
@@ -375,9 +375,9 @@ extern int saa7164_buffer_dealloc(struct saa7164_tsport *port,
375 375
376/* ----------------------------------------------------------- */ 376/* ----------------------------------------------------------- */
377 377
378extern unsigned int debug; 378extern unsigned int saa_debug;
379#define dprintk(level, fmt, arg...)\ 379#define dprintk(level, fmt, arg...)\
380 do { if (debug & level)\ 380 do { if (saa_debug & level)\
381 printk(KERN_DEBUG "%s: " fmt, dev->name, ## arg);\ 381 printk(KERN_DEBUG "%s: " fmt, dev->name, ## arg);\
382 } while (0) 382 } while (0)
383 383