# Re: \[PATCH\] video: add MODULE\_DEVICE\_TABLE

- From: Alan Cox \<hidden\>
- Date: 2011-03-07 10:31:58 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2011/03/3686667
- Thread: https://ratatoskr.run/linux-fbdev/2011/03/3686596/t.md

```text
On Mon, 07 Mar 2011 16:26:13 +0800
Axel Lin [ref] wrote:

> The device table is required to load modules based on modaliases.

Doing this then triggers autoloading which for some of these drivers is
the wrong thing.

> diff --git a/drivers/video/sstfb.c b/drivers/video/sstfb.c
> index 2ab7041..597e80b 100644
> --- a/drivers/video/sstfb.c
> +++ b/drivers/video/sstfb.c
> @@ -1485,6 +1485,7 @@ static const struct pci_device_id sstfb_id_tbl[] = {
>  		.driver_data = ID_VOODOO2, },
>  	{ 0 },
>  };
> +MODULE_DEVICE_TABLE(pci, sstfb_id_tbl);

NAK - as things stand in the frame buffer world you don't want sstfb
autoloading and stealing your main framebuffer away.

```