[PATCH] xextproto 7.1 support update to the source code

While the compilation script was updated, for some reason, the source code did not get the update. Signed-off-by: Kevin Brace <kevinbr...@gmx.com> --- src/r128_crtc.c |6 ++ src/r128_output.c |6 ++ 2 files changed, 12 insertions(+) diff --git a/src/r128_crtc.c

[PATCH] Merging Render code into r128_exa.c

Signed-off-by: Kevin Brace <kevinbr...@gmx.com> --- src/Makefile.am | 1 - src/r128_exa.c| 675 ++- src/r128_exa_render.c | 705 -- 3 files changed, 674 insertions(+), 707 del

[PATCH] Merging Render code into r128_exa.c

, but not with OpenChrome for now due to its KMS bug in the drm-openchrome). Of course, I still see the rendering bugs, and I will like to fix it for people who still care about it. Regards, Kevin Brace ___ xorg-driver-ati mailing list xorg-driver-ati

[PATCH xf86-video-r128 2/2] Move R128InitCrtc2Registers to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 82 +++ src/r128_driver.c | 82

[PATCH xf86-video-r128 1/2] Move R128InitCrtcRegisters to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 111 ++ src/r128_driver.c | 109

[PATCH xf86-video-r128 3/5] Convert R128Div to an inline function

Signed-off-by: Kevin Brace --- src/r128.h| 6 ++ src/r128_driver.c | 6 -- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/r128.h b/src/r128.h index aa98dd0..6142ff8 100644 --- a/src/r128.h +++ b/src/r128.h @@ -498,6 +498,12 @@ do

[PATCH xf86-video-r128 5/5] Move R128InitPLL2Registers to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 55 +++ src/r128_driver.c | 55

[PATCH xf86-video-r128 2/5] Move R128InitCrtc2Base to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 28 src/r128_driver.c | 28 3

[PATCH xf86-video-r128 4/5] Move R128InitPLLRegisters to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 55 +++ src/r128_driver.c | 55

[PATCH xf86-video-r128 1/5] Move R128InitCrtcBase to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 28 src/r128_driver.c | 28 3

[PATCH xf86-video-r128 3/4] Move R128InitDACRegisters to r128_output.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_output.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_driver.c | 13 - src/r128_output.c | 13 + 3 files changed, 13 insertions

[PATCH xf86-video-r128 1/4] Move R128RestoreCrtcRegisters to r128_crtc.c

r128_crtc.c is the more logical place this function should be located. Signed-off-by: Kevin Brace --- src/r128_crtc.c | 20 src/r128_driver.c | 20 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/src/r128_crtc.c b/src/r128_crtc.c

[PATCH xf86-video-r128 4/4] Move R128RestoreDACRegisters to r128_output.c

r128_output.c is the more logical place this function should be located. Signed-off-by: Kevin Brace --- src/r128_driver.c | 10 -- src/r128_output.c | 10 ++ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 3a20bbd

[PATCH xf86-video-r128 2/4] Move R128RestoreCrtc2Registers to r128_crtc.c

r128_crtc.c is the more logical place this function should be located. Signed-off-by: Kevin Brace --- src/r128_crtc.c | 18 ++ src/r128_driver.c | 18 -- 2 files changed, 18 insertions(+), 18 deletions(-) diff --git a/src/r128_crtc.c b/src/r128_crtc.c index

[PATCH xf86-video-r128 3/4] Move R128InitDDARegisters to r128_crtc.c

It does not make sense for this function to be inside r128_driver.c since it is only called from a function inside r128_crtc.c. Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_crtc.c | 78 +++ src/r128_driver.c | 73

[PATCH xf86-video-r128 1/4] Move R128RestorePLLRegisters to r128_crtc.c

r128_crtc.c is the more logical place this function should be located. R128PLLWaitForReadUpdateComplete and R128PLLWriteUpdate functions were moved as well. Signed-off-by: Kevin Brace --- src/r128_crtc.c | 89 +++ src/r128_driver.c | 89

[PATCH xf86-video-r128 2/4] Move R128RestorePLL2Registers to r128_crtc.c

r128_crtc.c is the more logical place this function should be located. R128PLL2WaitForReadUpdateComplete and R128PLL2WriteUpdate functions were moved as well. Signed-off-by: Kevin Brace --- src/r128_crtc.c | 85 ++ src/r128_driver.c | 86

[PATCH xf86-video-r128 1/4] Move the location where R128PreInitGamma is called

This is similar to how the sample code for PreInit callback function shown in XFree86 DDX Design (ddxDesign.pdf) design guide does it. Signed-off-by: Kevin Brace --- src/r128_driver.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c

[PATCH xf86-video-r128 3/4] Move the assignment to monitor field of ScrnInfoRec record

This is similar to how the sample code for PreInit callback function shown in XFree86 DDX Design (ddxDesign.pdf) design guide does it. Signed-off-by: Kevin Brace --- src/r128_driver.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/r128_driver.c b/src/r128_driver.c

[PATCH xf86-video-r128 4/4] Move R128PreInitVisual and R128PreInitGamma calls

This is similar to how the sample code for PreInit callback function shown in XFree86 DDX Design (ddxDesign.pdf) design guide does it. Signed-off-by: Kevin Brace --- src/r128_driver.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/r128_driver.c b/src

[PATCH xf86-video-r128 2/4] Move assignment of color depth variables away from R128PreInitVisual

Signed-off-by: Kevin Brace --- src/r128_driver.c | 35 --- 1 file changed, 16 insertions(+), 19 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 347c804..8bfa784 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -643,8 +643,6

[PATCH xf86-video-r128 6/8] Refactor R128PreInitWeight

Signed-off-by: Kevin Brace --- src/r128_driver.c | 36 +++- 1 file changed, 19 insertions(+), 17 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index b29b191..dd3521c 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -680,29 +680,31

[PATCH xf86-video-r128 7/8] Move the location xf86CrtcConfigInit is called inside R128PreInit

Signed-off-by: Kevin Brace --- src/r128_driver.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index dd3521c..0b30b83 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1278,9 +1278,6 @@ Bool R128PreInit(ScrnInfoPtr pScrn

[PATCH xf86-video-r128 5/8] Remove R128ProbeDDC

This removes a VBE (VESA BIOS Extension) related call when probing. Signed-off-by: Kevin Brace --- src/r128_driver.c | 23 --- 1 file changed, 4 insertions(+), 19 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 5c526f7..b29b191 100644 --- a/src

[PATCH xf86-video-r128 8/8] Move existing mode setting initialization code into R128LegacyMS

In preparation of adding a new mode setting code path. Signed-off-by: Kevin Brace --- src/r128_driver.c | 67 +++ 1 file changed, 38 insertions(+), 29 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 0b30b83..aaa4245

[PATCH xf86-video-r128] Using __func__ string for debug message function identification

Signed-off-by: Kevin Brace --- src/r128_driver.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 4918a0e..4cc95e6 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1264,7 +1264,7 @@ Bool R128PreInit

[PATCH xf86-video-r128] Remove compilation warnings related to debug messages

The variable sizes were not matching the intended print format. Signed-off-by: Kevin Brace --- src/r128_accel.c | 8 src/r128_driver.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/r128_accel.c b/src/r128_accel.c index ab04b25..0ad4edc 100644 --- a/src

[PATCH xf86-video-r128 1/2] Move the location R128PreInitDRI is called

Merge it into R128LegacyMS function. Signed-off-by: Kevin Brace --- src/r128_driver.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 87395a7..c272a30 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1242,6

[PATCH xf86-video-r128 2/2] Deprecate R128TRACE macro when generating debug messages

Deprecating R128TRACE macro in favor of DEBUG macro used inside OpenChrome DDX. Signed-off-by: Kevin Brace --- src/r128.h| 13 +++--- src/r128_accel.c | 57 src/r128_crtc.c | 117 +- src

[PATCH xf86-video-r128 v2] Consolidate frame buffer initialization code

The previous implementation had duplicated code depending on which standard is being used (i.e., DRI1 and / or EXA). Signed-off-by: Kevin Brace --- src/r128_driver.c | 110 +++--- 1 file changed, 30 insertions(+), 80 deletions(-) diff --git a/src

[PATCH xf86-video-r128 1/2] Prevent disabling of EXA inside R128ScreenInit

Signed-off-by: Kevin Brace --- src/r128_driver.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index afa41ca..32beddd 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1748,8 +1748,6 @@ Bool R128ScreenInit(SCREEN_INIT_ARGS_DECL

[PATCH xf86-video-r128] Deprecate R128AccelInit

Signed-off-by: Kevin Brace --- src/r128_driver.c | 39 --- 1 file changed, 12 insertions(+), 27 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index e2de57d..1642ea9 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1667,30 +1667,6

[PATCH xf86-video-r128] Deprecate R128VerboseInitEXA

Signed-off-by: Kevin Brace --- src/r128_driver.c | 39 ++- 1 file changed, 14 insertions(+), 25 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 1642ea9..ed560b7 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1643,30 +1643,6

[PATCH xf86-video-r128] Deprecate R128EXAAccelInit

Signed-off-by: Kevin Brace --- src/r128.h| 1 - src/r128_driver.c | 42 +- src/r128_exa.c| 19 --- 3 files changed, 21 insertions(+), 41 deletions(-) diff --git a/src/r128.h b/src/r128.h index 032c40b..2df674f 100644 --- a/src

[PATCH xf86-video-r128 v2] Evaluate software cursor option inside R128PreInit

Since SWCursor option is referenced only once, it should be handled inside R128PreInit callback function rather than R128ScreenInit callback function. Signed-off-by: Kevin Brace --- src/r128.h| 1 + src/r128_driver.c | 9 - 2 files changed, 9 insertions(+), 1 deletion(-) diff

[PATCH xf86-video-r128 2/4] Move many DDX UMS options into R128UMSOption

Signed-off-by: Kevin Brace --- src/r128_driver.c | 170 ++ 1 file changed, 94 insertions(+), 76 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index d21117a..c4a1019 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c

[PATCH xf86-video-r128 3/4] Delete USE_CRT_ONLY conditional compilation directive

Signed-off-by: Kevin Brace --- src/r128_driver.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index c4a1019..e34b1cd 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -109,9 +109,6 @@ #endif -#define USE_CRT_ONLY 0

[PATCH xf86-video-r128 1/4] Remove OPTION_DAC_8BIT option

It is no longer used. Signed-off-by: Kevin Brace --- src/r128_driver.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 57f4d05..d21117a 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -121,7 +121,6 @@ typedef enum { OPTION_NOACCEL

[PATCH xf86-video-r128] Reorganize r128_exa.c

Stop the use of #include to include r128_exa_render.c inside r128_exa.c. Signed-off-by: Kevin Brace --- src/Makefile.am | 23 -- src/r128.h| 24 +++ src/r128_accel.c | 22 +- src/r128_exa.c| 31

[PATCH xf86-video-r128 4/4] Delete OPTION_DISPLAY option enumeration

Signed-off-by: Kevin Brace --- src/r128_driver.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index e34b1cd..6c1a9e9 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -137,7 +137,6 @@ typedef enum { OPTION_BUFFER_SIZE, OPTION_PAGE_FLIP

[PATCH xf86-video-r128] Move RAMDAC module initialization into R128LegacyMS

Signed-off-by: Kevin Brace --- src/r128_driver.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 6c1a9e9..773038c 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1031,7 +1031,7 @@ static Bool R128PreInitCursor

[PATCH xf86-video-r128] Evaluate software cursor option inside R128PreInit

Since SWCursor option is referenced only once, it should be handled inside R128PreInit callback function rather than R128ScreenInit callback function. Signed-off-by: Kevin Brace --- src/r128.h| 1 + src/r128_driver.c | 9 - 2 files changed, 9 insertions(+), 1 deletion(-) diff

[PATCH xf86-video-r128] Acquire NoAccel option information inside R128PreInit

Since NoAccel option is referenced only once, it should be handled inside R128PreInit callback function rather than R128ScreenInit callback function. Signed-off-by: Kevin Brace --- src/r128.h| 2 ++ src/r128_driver.c | 20 2 files changed, 14 insertions(+), 8

[PATCH xf86-video-r128] Evaluate EXA related options inside R128PreInit

Since AccelMethod and RenderAccel options are referenced only once, it should be handled inside R128PreInit callback function rather than R128ScreenInit callback function. Signed-off-by: Kevin Brace --- src/r128_driver.c | 62 ++- 1 file

[PATCH xf86-video-r128] Consolidate EXA initialization code into R128EXAInit

Signed-off-by: Kevin Brace --- src/r128.h| 2 +- src/r128_driver.c | 42 +- src/r128_exa.c| 13 - 3 files changed, 26 insertions(+), 31 deletions(-) diff --git a/src/r128.h b/src/r128.h index 2df674f..5e6502b 100644 --- a/src/r128

[PATCH xf86-video-r128] Hold off enabling acceleration until EXA / XAA are fully initialized

The turn on timing is premature. Wait until completion of the initialization process. Signed-off-by: Kevin Brace --- src/r128_driver.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index a3b7ac4..04295a5 100644 --- a/src

[PATCH xf86-video-r128 2/2] Consolidate acquisition of many options into two functions

Move acquiring of the option settings into r128AcquireOption and r128UMSOption functions. Signed-off-by: Kevin Brace --- src/r128_driver.c | 170 +++--- 1 file changed, 97 insertions(+), 73 deletions(-) diff --git a/src/r128_driver.c b/src

[PATCH xf86-video-r128 1/2] Rename R128UMSOption to r128UMSOption

Signed-off-by: Kevin Brace --- src/r128_driver.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/r128_driver.c b/src/r128_driver.c index 04295a5..30a7a21 100644 --- a/src/r128_driver.c +++ b/src/r128_driver.c @@ -1196,7 +1196,8 @@ static Bool R128PreInitControllers

[PATCH xf86-video-r128] Move loading of EXA and XAA modules into R128PreInitAccel

Moving them away from R128ScreenInit callback function. This is how other DDXs do it. Signed-off-by: Kevin Brace --- src/r128_accel.c | 1 - src/r128_driver.c | 48 +++- src/r128_exa.c| 12 3 files changed, 43 insertions(+), 18

[ANNOUNCE] xf86-video-r128 6.11.0

Fix for cross build failure Kevin Brace (31): Move the location where R128PreInitGamma is called Move assignment of color depth variables away from R128PreInitVisual Move the assignment to monitor field of ScrnInfoRec record Move R128PreInitVisual and R128PreInitGamma ca

[PATCH xf86-video-r128 1/2] Fix for cross build failure

From: Helmut Grohne configure.ac fails to cross build from source, because it abuses AC_CHECK_FILE for finding headers. AC_CHECK_FILE is supposed to find files on the host system. You cannot usually expect the host system to have headers, so this use is incorrect. It happens to also break cross

[PATCH xf86-video-r128 2/2] Fix typo: Initalizing --> Initializing

From: Dylan Aïssi Signed-off-by: Dylan Aïssi --- src/r128_exa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/r128_exa.c b/src/r128_exa.c index 1bbf470..5099915 100644 --- a/src/r128_exa.c +++ b/src/r128_exa.c @@ -484,7 +484,7 @@ R128EXAInit(ScreenPtr pScreen) }

Can someone else with a RAGE 128 based laptop test a patch named, "Fix for some RAGE 128 Pro causing out of range display error?"

Location where the backup of r128_drv.so was stored")/r128_drv.so /usr/lib/xorg/modules/drivers/ Let me know how the patch works. If the patch works, I will like to release a new version of xf86-video-r128 soon. Kevin Brace Brace Computer Laboratory blog https://bracecomputerlab.com ___

[PATCH xf86-video-r128] Separate EXA and XAA initialization

Starting the process of separating legacy XAA and newer EXA initialization code. Signed-off-by: Kevin Brace --- src/r128.h| 3 ++- src/r128_accel.c | 44 +++- src/r128_driver.c | 51 ++- src

[PATCH xf86-video-r128] Reorganize r128_exa.c

Stop the use of #include to include r128_exa_render.c inside r128_exa.c. Signed-off-by: Kevin Brace --- src/Makefile.am | 23 -- src/r128.h| 24 +++ src/r128_accel.c | 22 +- src/r128_exa.c| 31

Changing xf86-video-r128 DDX version number assignment convention

; "major version"."minor version"."patch level") more frequently between minor versions? If I were to adopt this approach, I think the next development version after Version 6.11.0 is, 6.11.99. Is this correct? Kevin Brace Brace Computer Laboratory blo

[PATCH xf86-video-r128] Convert tabs to white spaces for r128_output.c

Signed-off-by: Kevin Brace --- src/r128_output.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/src/r128_output.c b/src/r128_output.c index 41a2533..f11acde 100644 --- a/src/r128_output.c +++ b/src/r128_output.c @@ -280,10 +280,10 @@ static

[PATCH xf86-video-r128] Fix for some RAGE 128 Pro causing out of range display error

. With this fix, there is no longer the need to specify the supported frequency range of the monitor inside xorg.conf. The code was tested on RAGE 128 Pro Ultra 16 MB AGP. Signed-off-by: Kevin Brace --- src/r128_output.c | 60 ++- 1 file changed, 33

[ANNOUNCE] xf86-video-r128 6.12.0

hard to maintain and very awkwardly written XAA / EXA initialization code. It is always possible that I can mess up the code, but it appears that the code is working correctly. --- Helmut Grohne (1): Another fix for cross build failure Kevin Brace (26): Version bumped to 6.11.99

Re: xf86-video-r128 fails to build with --disable-dri

should fix the reported bug for good. You should be able to close the bug with this update. Regards, Kevin Brace Brace Computer Laboratory https://bracecomputerlab.com > Sent: Tuesday, October 16, 2018 at 12:43 PM > From: "Matt Turner" > To: "Kevin Brace" >