2.6.15-mm3, current -git: drivers/media/video/ compile errors

From: Adrian Bunk
Date: Wed Jan 11 2006 - 17:20:43 EST


On Wed, Jan 11, 2006 at 05:21:23PM +0100, Dominik Karall wrote:
>
> hi!
> it doesn't compile here.
>
> CC drivers/media/video/tveeprom.o
> LD drivers/media/video/built-in.o
> drivers/media/video/tuner.o:(.bss+0x0): multiple definition of `debug'
> drivers/media/video/msp3400.o:(.bss+0xc): first defined here
> make[3]: *** [drivers/media/video/built-in.o] Fehler 1
>...

I'm getting even one more error:

<-- snip -->

...
drivers/media/video/tuner.o:(.bss+0x0): multiple definition of `debug'
drivers/media/video/msp3400.o:(.bss+0xc): first defined here
drivers/media/video/cx25840/built-in.o:(.bss+0x0): multiple definition of `debug'
drivers/media/video/msp3400.o:(.bss+0xc): first defined here
make[3]: *** [drivers/media/video/built-in.o] Error 1

<-- snip -->

There's sometime a need for variables being global being visible in
all objects of a module.

That's OK.

But they should never have generic names like "debug" or "once" (the
latter and some similar ones don't seem to cause compile errors since
they are currently used only once, but they are equally wrong.

> greets,
> dominik

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/