Bug 1401

Summary: Please update pcitable to recognize and configure new Intel GPU (Sandy Bridge)
Product: Mageia Reporter: Damien Lallement <mageia>
Component: RPM PackagesAssignee: Thierry Vignaud <thierry.vignaud>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal CC: lmenut, mageia
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: ldetect-lst-0.1.291-7.mga1.src.rpm CVE:
Status comment:

Description Damien Lallement 2011-05-23 13:42:51 CEST
Description of problem: please add the following line to pcitable: 0x8086  0x0126  "Card:Intel 810 and later"

It's working without any issue on my laptop.
[dams@maximus SOURCES]$ lspcidrake -v | grep DISP
i915            : Intel Corporation|Sandy Bridge Integrated Graphics Controller [DISPLAY_VGA] (vendor:8086 device:0126 subv:17aa subd:21da) (rev: 09)
[dams@maximus SOURCES]$ cat /etc/X11/xorg.conf | grep Driver
    Driver "intel"
[dams@maximus SOURCES]$
Comment 1 Luc Menut 2011-05-24 00:47:00 CEST
0x0126 isn't the only id for Intel Sandy Bridge GPU.
I had a look at intel_driver.h from our x11-driver-video-intel-2.15.0; for Intel Sandy Bridge, there are:

#ifndef PCI_CHIP_SANDYBRIDGE_BRIDGE
#define PCI_CHIP_SANDYBRIDGE_BRIDGE	0x0100	/* Desktop */
#define PCI_CHIP_SANDYBRIDGE_GT1	0x0102
#define PCI_CHIP_SANDYBRIDGE_GT2	0x0112
#define PCI_CHIP_SANDYBRIDGE_GT2_PLUS	0x0122
#define PCI_CHIP_SANDYBRIDGE_BRIDGE_M	0x0104	/* Mobile */
#define PCI_CHIP_SANDYBRIDGE_M_GT1	0x0106
#define PCI_CHIP_SANDYBRIDGE_M_GT2	0x0116
#define PCI_CHIP_SANDYBRIDGE_M_GT2_PLUS	0x0126
#define PCI_CHIP_SANDYBRIDGE_BRIDGE_S	0x0108	/* Server */
#define PCI_CHIP_SANDYBRIDGE_S_GT	0x010A
#endif

shouldn't we add all the chipsets id? I'm not sure for  PCI_CHIP_SANDYBRIDGE_BRIDGE* (0x0100, 0x0104, 0x0108), but It seems the others are for various Sandy Bridge GPU.

CC: (none) => lmenut, mageia

Comment 2 Thierry Vignaud 2011-05-24 22:51:50 CEST
I've added all IDS.
Please commit in SVN rather than patching btw...

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 3 Ahmad Samir 2011-05-24 23:08:27 CEST
It didn't exist in svn/soft a week ago (you yourself used a patch for the qxl chipsets).