Age | Commit message (Collapse) | Author | Files | Lines |
|
Direct calls to ScreenRec->DestroyPixmap() blocks cleaning up the wrapping
jungle, so use the proper dix function instead.
See: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1754
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/15>
|
|
Clears -Wcalloc-transposed-args warnings from gcc 14.1, such as:
r128_dri.c: In function ‘R128DRIScreenInit’:
r128_dri.c:822:48: warning: ‘calloc’ sizes specified with ‘sizeof’ in the
earlier argument and not in the later argument [-Wcalloc-transposed-args]
822 | if (!(pR128DRI = (R128DRIPtr)calloc(sizeof(R128DRIRec),1))) {
| ^~~~~~~~~~
r128_dri.c:822:48: note: earlier argument should specify number of elements,
later size of each element
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/13>
|
|
We're now relying on at least 1.18
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/11>
|
|
Compilers on recent Debian (gcc as well as clang) break compilation if
there's no statement after a label:
> ../../src/r128_driver.c:571:1: error: expected statement
> }
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/12>
|
|
Already effectively required by use of XNFcallocarray() introduced in
xorg/xserver@b96dc999 - xserver-1.18.0, released in Nov. 2015.
Allows dropping remnants of code for XAA and pre-pciaccess X servers
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/10>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/10>
|
|
xnfcalloc is just an alias for XNFcallocarray() that doesn't seem to serve
any practical purpose, so it can go away once all drivers stopped using it.
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/8>
|
|
based on a patch by Michael Lorenz.
Signed-off-by: Nia Alarie <nia@NetBSD.org>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/7>
|
|
Patch adapted from work by Michael Lorenz.
Signed-off-by: Nia Alarie <nia@NetBSD.org>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/7>
|
|
With this we no longer need to specify PanelWidth and PanelHeight in
xorg.conf on non-x86 platforms.
Patch adapted from work by Michael Lorenz.
Signed-off-by: Nia Alarie <nia@NetBSD.org>
Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-r128/-/merge_requests/7>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Fixes build failure when compiling with Solaris Studio compiler
using -xldscope=hidden:
"r128_probe.c", line 359: redeclaration must have the same or more
restrictive linker scoping: R128
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Clears 4 gcc warnings:
r128_driver.c:178:36: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
{ 4, 4, 3, 3, 1, 3, 1, 16, 12, "128-bit SDR SGRAM 1:1" },
^~~~~~~~~~~~~~~~~~~~~~~
r128_driver.c:179:36: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
{ 4, 8, 3, 3, 1, 3, 1, 17, 13, "64-bit SDR SGRAM 1:1" },
^~~~~~~~~~~~~~~~~~~~~~
r128_driver.c:180:36: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
{ 4, 4, 1, 2, 1, 2, 1, 16, 12, "64-bit SDR SGRAM 2:1" },
^~~~~~~~~~~~~~~~~~~~~~
r128_driver.c:181:36: warning: initialization discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
{ 4, 4, 3, 3, 2, 3, 1, 16, 12, "64-bit DDR SGRAM" },
^~~~~~~~~~~~~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Found by using:
codespell --builtin clear,rare,usage,informal,code,names
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Move acquiring of the option settings into r128AcquireOption and
r128UMSOption functions.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
The turn on timing is premature. Wait until completion of the
initialization process.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
The previous implementation had duplicated code depending on which
standard is being used (i.e., DRI1 and / or EXA).
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Moving them away from R128ScreenInit callback function. This is how
other DDXs do it.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
It is no longer used.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|
|
Starting the process of separating legacy XAA and newer EXA
initialization code.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Stop the use of #include to include r128_exa_render.c inside
r128_exa.c.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Dylan Aïssi <bob.dybian@gmail.com>
|
|
Due to faulty assumptions made by commit 2072d9a, many RAGE 128 Pro
VGA display will be detected as DVI, and causes out of range display
error with the attached monitor. While this commit is less than
perfect and only works on x86 platform, it avoids 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 <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
The variable sizes were not matching the intended print format.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Deprecating R128TRACE macro in favor of DEBUG macro used inside
OpenChrome DDX.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Merge it into R128LegacyMS function.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
Apparently this is needed in src/atipcirename.h since xorg-server
1.20 in order to still build this driver.
Signed-off-by: Stefan Dirsch <sndirsch@suse.de>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
|
|
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 <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|
|
r128_output.c is the more logical place this function should be located.
Signed-off-by: Kevin Brace <kevinbrace@gmx.com>
|
|
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 <kevinbrace@gmx.com>
|