Re: 2.6.11-mm3: saa7134-core.c compile error

From: Adrian Bunk
Date: Tue Mar 15 2005 - 13:00:29 EST


On Mon, Mar 14, 2005 at 08:38:22PM -0800, Andrew Morton wrote:
> Adrian Bunk <bunk@xxxxxxxxx> wrote:
> >
> > This doesn't compile with CONFIG_MODULES=n:
> >
> > <-- snip -->
> >
> > ...
> > CC drivers/media/video/saa7134/saa7134-core.o
> > drivers/media/video/saa7134/saa7134-core.c: In function `saa7134_fini':
> > drivers/media/video/saa7134/saa7134-core.c:1215: error: `pending_registered' undeclared (first use in this function)
>
> Like this, I guess:

This fixes the compilation for me.

> --- 25/drivers/media/video/saa7134/saa7134-core.c~saa7134-build-fix 2005-03-14 20:37:16.000000000 -0800
> +++ 25-akpm/drivers/media/video/saa7134/saa7134-core.c 2005-03-14 20:37:27.000000000 -0800
> @@ -1212,8 +1212,10 @@ static int saa7134_init(void)
>
> static void saa7134_fini(void)
> {
> +#ifdef CONFIG_MODULES
> if (pending_registered)
> unregister_module_notifier(&pending_notifier);
> +#endif
> pci_unregister_driver(&saa7134_pci_driver);
> }

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/