Re: OOPS 2.6.0-test2, modprobe i810fb

From: Bill Davidsen (davidsen@tmr.com)
Date: Tue Jul 29 2003 - 21:56:00 EST


On Mon, 28 Jul 2003, Andrew Morton wrote:

> "S. Anderson" <sa@xmission.com> wrote:
> >
> > when that driver is
> > "i810fb, i810_audio or intel-agp" (and i810fb, i810_audio or intel-agp
> > is allready loaded) the id_table is at an address that cant be handled,
> > thus cauing the oops. I am having trouble figuring out why
> > pci_drv->id_table isnt valid in this case.
>
> Does this fix? I'm not sure whether that "{ }" in there will generate
> another table entry...
>
>
> diff -puN drivers/char/agp/intel-agp.c~intel-agp-oops-fix drivers/char/agp/intel-agp.c
> --- 25/drivers/char/agp/intel-agp.c~intel-agp-oops-fix 2003-07-28 22:30:30.000000000 -0700
> +++ 25-akpm/drivers/char/agp/intel-agp.c 2003-07-28 22:30:53.000000000 -0700
> @@ -1426,7 +1426,7 @@ static struct pci_device_id agp_intel_pc
> .subvendor = PCI_ANY_ID,
> .subdevice = PCI_ANY_ID,
> },
> - { }
> + { 0, },
> };
>
> MODULE_DEVICE_TABLE(pci, agp_intel_pci_table);
>
> _

Sure about that last comma? Any compiler version issues?

-- 
bill davidsen <davidsen@tmr.com>
  CTO, TMR Associates, Inc
Doing interesting things with little computers since 1979.

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



This archive was generated by hypermail 2b29 : Thu Jul 31 2003 - 22:00:43 EST