Bug 104069 - Radeon MetaModes driver option interferes with SaX2 option handling
Summary: Radeon MetaModes driver option interferes with SaX2 option handling
Status: RESOLVED FIXED
Alias: None
Product: SUSE LINUX 10.0
Classification: openSUSE
Component: X.Org (show other bugs)
Version: Beta 1
Hardware: All All
: P5 - None : Major
Target Milestone: ---
Assignee: Marcus Schaefer
QA Contact: Stefan Dirsch
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-11 10:01 UTC by Matthias Hopf
Modified: 2005-08-12 09:48 UTC (History)
2 users (show)

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Patch for adding ',' and ';' as additional delimiter characters (10.39 KB, patch)
2005-08-11 10:02 UTC, Matthias Hopf 		Details | Diff
New patch, for SL 10.0 (8.27 KB, patch)
2005-08-11 16:52 UTC, Matthias Hopf 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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