Bug 104069

Summary:	Radeon MetaModes driver option interferes with SaX2 option handling
Product:	[openSUSE] SUSE LINUX 10.0	Reporter:	Matthias Hopf <mhopf>
Component:	X.Org	Assignee:	Marcus Schaefer <ms>
Status:	RESOLVED FIXED	QA Contact:	Stefan Dirsch <sndirsch>
Severity:	Major
Priority:	P5 - None	CC:	ms, sndirsch
Version:	Beta 1
Target Milestone:	---
Hardware:	All
OS:	All
Whiteboard:
Found By:	Other	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	Patch for adding ',' and ';' as additional delimiter characters New patch, for SL 10.0

Description Matthias Hopf 2005-08-11 10:01:03 UTC

The radeon driver's MetaModes driver option needs Spaces for seperating
different modes, which does not work with SaX2 secondary monitor configuration.

Instead of '-' and ' ' as delimiting characters the driver should additionally
respect ',' and ';', which would make it compatible to the NVidia driver as well.

Comment 1 Matthias Hopf 2005-08-11 10:02:25 UTC

Created attachment 45687 [details]
Patch for adding ',' and ';' as additional delimiter characters

This patch updates the radeon, sis, and mga modules, which share about the same
code.

Comment 2 Matthias Hopf 2005-08-11 10:03:20 UTC

Stefan, please assign to Marcus after fixing, to track the SaX2 changes.

Comment 3 Stefan Dirsch 2005-08-11 14:08:24 UTC

Patch #291 (p_radeon_mga_sis_metamodes.diff):
+ patch -p0 -s
7 out of 7 hunks FAILED -- saving rejects to file
programs/Xserver/hw/xfree86/drivers/ati/radeon_mergedfb.c.rej

Could you recreate it for our SuSE 10.0 X.Org version, please? :-)

Comment 4 Matthias Hopf 2005-08-11 16:52:56 UTC

Created attachment 45809 [details]
New patch, for SL 10.0

This should be it.

Comment 5 Stefan Dirsch 2005-08-11 21:33:38 UTC

Thanks. Patch applied and xorg-x11 package submitted for STABLE now.

Comment 6 Marcus Schaefer 2005-08-12 09:48:43 UTC

Thanks, enabled full featured MetaModes handling in SaX now