Re: [PATCH] suspend/resume for ATI Mobility Radeon RV350

Previous thread: [ksoftirqd/1] CPU reporting. by Jon Smirl on Tuesday, September 30, 2008 - 7:41 am. (3 messages)

Next thread: [PATCH] properly reserve in bootmem the lmb reserved regions that cross numa nodes by Jon Tollefson on Tuesday, September 30, 2008 - 7:53 am. (7 messages)
From: Wolfgang Kroener
Date: Tuesday, September 30, 2008 - 7:06 am

Hi,
I have a ATI Mobility Radeon RV350 and similar problems with radeonfb
described in http://kernel.org/doc/Documentation/power/s2ram.txt
Adding the device to radeon_workaround_list solves the Problem.
Please accept the following patch:

--8<--
--- drivers/video/aty/radeon_pm.c.orig	2008-09-30 15:00:40.988111330 +0200
+++ drivers/video/aty/radeon_pm.c	2008-09-30 15:14:10.941524474 +0200
@@ -89,6 +89,9 @@
 	BUGFIX("Acer Aspire 2010",
 	       PCI_VENDOR_ID_AI, 0x0061,
 	       radeon_pm_off, radeon_reinitialize_M10),
+	BUGFIX("Acer Travelmate 290D/292LMi",
+	       PCI_VENDOR_ID_AI, 0x005a,
+	       radeon_pm_off, radeon_reinitialize_M10),
 	{ .ident = NULL }
 };

-->8--
Kind regards,
Wolfgang

P. S. please CC me, because I'm not subscribed to the lkml
--

From: Benjamin Herrenschmidt
Date: Tuesday, September 30, 2008 - 3:10 pm

Can you resend it with a proper Signed-off-by: line ? Thanks !

Cheers,

--

From: Wolfgang Kroener
Date: Wednesday, October 1, 2008 - 3:22 am

Hi,


I hope, that's right.
Kind regards,
Wolfgang
-- 

radeonfb suspend/resume for Acer Travelmate 29X

This patch adds suspend/resume for the Acer Travelmate 290D/292LMi with
the following graphic-chip:
01:00.0 VGA compatible controller [0300]: ATI Technologies Inc RV350
[Mobility Radeon 9600 M10] [1002:4e50] (prog-if 00 [VGA controller])
	Subsystem: Acer Incorporated [ALI] TravelMate 290 [1025:005a]
	Flags: bus master, 66MHz, medium devsel, latency 128, IRQ 10
	Memory at a8000000 (32-bit, prefetchable) [size=128M]
	I/O ports at c100 [size=256]
	Memory at e0010000 (32-bit, non-prefetchable) [size=64K]
	[virtual] Expansion ROM at a0000000 [disabled] [size=128K]
	Capabilities: [58] AGP version 2.0
	Capabilities: [50] Power Management version 2
	Kernel driver in use: radeonfb
	Kernel modules: radeonfb

Signed-off-by: Wolfgang Kroener <lkml@azog.de>
---
 drivers/video/aty/radeon_pm.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/drivers/video/aty/radeon_pm.c b/drivers/video/aty/radeon_pm.c
index 675abda..5e4b2da 100644
--- a/drivers/video/aty/radeon_pm.c
+++ b/drivers/video/aty/radeon_pm.c
@@ -89,6 +89,9 @@ static struct radeon_device_id radeon_workaround_list[] = {
 	BUGFIX("Acer Aspire 2010",
 	       PCI_VENDOR_ID_AI, 0x0061,
 	       radeon_pm_off, radeon_reinitialize_M10),
+	BUGFIX("Acer Travelmate 290D/292LMi",
+	       PCI_VENDOR_ID_AI, 0x005a,
+	       radeon_pm_off, radeon_reinitialize_M10),
 	{ .ident = NULL }
 };
 
--

From: Wolfgang Kroener
Date: Thursday, January 8, 2009 - 1:57 pm

Hi,

I think the mail (http://lkml.org/lkml/2008/10/1/134) got lost
somewhere, so I try again:


radeonfb suspend/resume for Acer Travelmate 29X

This patch adds suspend/resume for the Acer Travelmate 290D/292LMi with
the following graphic-chip:
01:00.0 VGA compatible controller [0300]: ATI Technologies Inc RV350
[Mobility Radeon 9600 M10] [1002:4e50] (prog-if 00 [VGA controller])
	Subsystem: Acer Incorporated [ALI] TravelMate 290 [1025:005a]
	Flags: bus master, 66MHz, medium devsel, latency 128, IRQ 10
	Memory at a8000000 (32-bit, prefetchable) [size=128M]
	I/O ports at c100 [size=256]
	Memory at e0010000 (32-bit, non-prefetchable) [size=64K]
	[virtual] Expansion ROM at a0000000 [disabled] [size=128K]
	Capabilities: [58] AGP version 2.0
	Capabilities: [50] Power Management version 2
	Kernel driver in use: radeonfb
	Kernel modules: radeonfb

Signed-off-by: Wolfgang Kroener <lkml@azog.de>
---
 drivers/video/aty/radeon_pm.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/drivers/video/aty/radeon_pm.c b/drivers/video/aty/radeon_pm.c
index 675abda..5e4b2da 100644
--- a/drivers/video/aty/radeon_pm.c
+++ b/drivers/video/aty/radeon_pm.c
@@ -89,6 +89,9 @@ static struct radeon_device_id radeon_workaround_list[] = {
 	BUGFIX("Acer Aspire 2010",
 	       PCI_VENDOR_ID_AI, 0x0061,
 	       radeon_pm_off, radeon_reinitialize_M10),
+	BUGFIX("Acer Travelmate 290D/292LMi",
+	       PCI_VENDOR_ID_AI, 0x005a,
+	       radeon_pm_off, radeon_reinitialize_M10),
 	{ .ident = NULL }
 };
 
-- 
1.5.6.5
--

From: Benjamin Herrenschmidt
Date: Friday, January 9, 2009 - 12:34 am

Fell through the cracks along with some of my own radeonfb patches :-)

Thanks for reminding.

Cheers,

--

Previous thread: [ksoftirqd/1] CPU reporting. by Jon Smirl on Tuesday, September 30, 2008 - 7:41 am. (3 messages)

Next thread: [PATCH] properly reserve in bootmem the lmb reserved regions that cross numa nodes by Jon Tollefson on Tuesday, September 30, 2008 - 7:53 am. (7 messages)