[PATCH 1/4] radeonfb: add PCI Id for RS482/Radeon Xpress 200M

Previous thread: [PATCH 0/4] radeonfb: minor fixes by aherrman on Tuesday, September 4, 2007 - 6:57 am. (1 message)

Next thread: [PATCH 2/4] radeonfb: add new module parameter to force PLL calculation by aherrman on Tuesday, September 4, 2007 - 6:59 am. (3 messages)
To: <benh@...>, <linux-fbdev-devel@...>, <linux-kernel@...>
Date: Tuesday, September 4, 2007 - 6:58 am

.. which can be found in Acer Aspire 5100.

Signed-off-by: Andreas Herrmann <aherrman@arcor.de>
---
drivers/video/aty/ati_ids.h | 1 +
drivers/video/aty/radeon_base.c | 1 +
2 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/video/aty/ati_ids.h b/drivers/video/aty/ati_ids.h
index 685a754..648da1e 100644
--- a/drivers/video/aty/ati_ids.h
+++ b/drivers/video/aty/ati_ids.h
@@ -211,3 +211,4 @@
#define PCI_CHIP_RS350_7834 0x7834
#define PCI_CHIP_RS350_7835 0x7835
#define PCI_CHIP_RS480_5955 0x5955
+#define PCI_CHIP_RS482_5975 0x5975
diff --git a/drivers/video/aty/radeon_base.c b/drivers/video/aty/radeon_base.c
index 47ca62f..90a3957 100644
--- a/drivers/video/aty/radeon_base.c
+++ b/drivers/video/aty/radeon_base.c
@@ -102,6 +102,7 @@
static struct pci_device_id radeonfb_pci_table[] = {
/* Radeon Xpress 200m */
CHIP_DEF(PCI_CHIP_RS480_5955, RS480, CHIP_HAS_CRTC2 | CHIP_IS_IGP | CHIP_IS_MOBILITY),
+ CHIP_DEF(PCI_CHIP_RS482_5975, RS480, CHIP_HAS_CRTC2 | CHIP_IS_IGP | CHIP_IS_MOBILITY),
/* Mobility M6 */
CHIP_DEF(PCI_CHIP_RADEON_LY, RV100, CHIP_HAS_CRTC2 | CHIP_IS_MOBILITY),
CHIP_DEF(PCI_CHIP_RADEON_LZ, RV100, CHIP_HAS_CRTC2 | CHIP_IS_MOBILITY),
--
1.5.3

-

To: <linux-fbdev-devel@...>
Cc: <linux-kernel@...>
Date: Tuesday, September 4, 2007 - 9:45 am

-

To: Benjamin Herrenschmidt <benh@...>
Cc: <linux-kernel@...>, <linux-fbdev-devel@...>
Date: Monday, September 10, 2007 - 6:38 pm

Sorry, I have missed that 0x5975 was already defined for radeonfb.
So this patch shouldn't be applied.

We have already:

/*Mobility Xpress 200 */
CHIP_DEF(PCI_CHIP_RS485_5975, R300, CHIP_HAS_CRTC2 | CHIP_IS_IGP | CHIP_IS_MOBILITY),

This was introduced with commit b5f2f4d1a6d7efde39cfb5e1d034981c69f2214c

But obviously this is not correct as the 0x5975 is an RS48x not R300.
With the current git (w/o my patch) my display goes blank (or better it looks striped)
and the Laptop hangs.

I'll do some further tests on my Laptop and probably come up with a new patch.

Regards,

Andreas

-

Previous thread: [PATCH 0/4] radeonfb: minor fixes by aherrman on Tuesday, September 4, 2007 - 6:57 am. (1 message)

Next thread: [PATCH 2/4] radeonfb: add new module parameter to force PLL calculation by aherrman on Tuesday, September 4, 2007 - 6:59 am. (3 messages)