Margit Schubert-While wrote:
>  From drivers/char/agp/agpgart_be.c
> 4554,4559
>     { PCI_DEVICE_ID_INTEL_845_G_0,
>                  PCI_VENDOR_ID_INTEL,
>                  INTEL_I845_G,
>                  "Intel",
>                  "i845G",
>                  intel_830mp_setup },
> 
> Surely this is wrong or ?
> Should be "intel_845_setup", I think.
> 

IIRC, the 845G is a "new" version of the 830MP chipset (it had been
added by Abraham vd Merwe & Graeme Fisher some months ago), but acts
basically just as the 830MP. Therefore the entry is correct.... Or maybe
if it gets confusing adding a comment would not hurt...

> 
> Also in drivers/char/drm/drm_agpsupport.h, the switch statement at 262 
> is missing the
> cases for INTEL_I830_M, INTEL_I845_G.


That's true. It is also missing in 2.5.51.
I attach two patches, one for 2.4.21-pre1 and one for 2.5.51 that should 
fix this.


Regards

Nicolas.