Bug#508108: [PATCH] Additional PCI id for sunxvr500 driver

2010-02-26 Thread David Miller
From: Ben Hutchings 
Date: Fri, 19 Feb 2010 02:56:21 +

> Intergraph bought 3D Labs and some XVR-500 chips have Intergraph's
> vendor id.
> 
> Reported-by: Jurij Smakov 
> Signed-off-by: Ben Hutchings 
> Cc: sta...@kernel.org

Applied, thanks Ben.



-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20100226.043741.201314057.da...@davemloft.net



Bug#508108: [PATCH] Additional PCI id for sunxvr500 driver

2010-02-18 Thread Ben Hutchings
Intergraph bought 3D Labs and some XVR-500 chips have Intergraph's
vendor id.

Reported-by: Jurij Smakov 
Signed-off-by: Ben Hutchings 
Cc: sta...@kernel.org
---
Jurij Smakov wrote on 2008-12-01:
> Hi,
> 
> My SunBlade 1000 has the following graphics chip:
> 
> :00:01.0 VGA compatible controller: Intergraph Corporation Sun 
> Expert3D-Lite Graphics Accelerator
> 
> with PCI ID 1091:07a0 (vendor ID is Intergraph Corporation). I've 
> noticed that XVR-500 framebuffer driver (drivers/video/sunxvr500.c) 
> supports the framebuffer device 3d3d:07a0, which only has the same 
> device ID but a different vendor (3D Labs). Given that 3D Labs bought 
> up Intergraph Corporation at some point, it's probably the same chip, 
> so I tried adding 1091:07a0 to the list of supported PCI IDs for 
> sunxvr500.c and it worked without problems, the console now is much 
> nicer than the default PROM monochrome one. Please consider applying 
> the included patch, which adds this card to the list of supported 
> devices for XVR-500 driver.

Jurij originally posted a patch for sunxvr500 which also touched
pci_ids.h.  This is the same thing without involving pci_ids.h.

Ben.

---
--- a/drivers/video/sunxvr500.c
+++ b/drivers/video/sunxvr500.c
@@ -400,6 +400,7 @@
 
 static struct pci_device_id e3d_pci_table[] = {
{   PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x7a0),},
+   {   PCI_DEVICE(0x1091, 0x7a0),  },
{   PCI_DEVICE(PCI_VENDOR_ID_3DLABS, 0x7a2),},
{   .vendor = PCI_VENDOR_ID_3DLABS,
.device = PCI_ANY_ID,

-- 
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.



--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/1266548181.10567.727.ca...@localhost