diff options
author | Ingo Molnar <mingo@elte.hu> | 2009-09-21 14:23:45 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-09-22 17:58:17 -0400 |
commit | b1912a85b54c27738afe1c4fa069df02d3316f0c (patch) | |
tree | 07ad08fa40efa233cd3cff5ce53c81711d16a402 | |
parent | 7fa07729e439a6184bd824746d06a49cca553f15 (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>
-rw-r--r-- | drivers/media/video/saa7164/saa7164-api.c | 8 | ||||
-rw-r--r-- | drivers/media/video/saa7164/saa7164-cmd.c | 2 | ||||
-rw-r--r-- | drivers/media/video/saa7164/saa7164-core.c | 6 | ||||
-rw-r--r-- | drivers/media/video/saa7164/saa7164.h | 4 |
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 | ||
48 | unsigned int debug; | 48 | unsigned int saa_debug; |
49 | module_param(debug, int, 0644); | 49 | module_param_named(debug, saa_debug, int, 0644); |
50 | MODULE_PARM_DESC(debug, "enable debug messages"); | 50 | MODULE_PARM_DESC(debug, "enable debug messages"); |
51 | 51 | ||
52 | unsigned int waitsecs = 10; | 52 | unsigned 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 | ||
659 | fail_fw: | 659 | fail_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 | ||
378 | extern unsigned int debug; | 378 | extern 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 | ||