[PATCH:xf86-input-joystick] Adapt to USB HID header changes on NetBSD-8.99.9.

From: Manuel Bouyer Signed-off-by: Thomas Klausner --- configure.ac | 1 + src/backend_bsd.c | 3 +++ 2 files changed, 4 insertions(+) diff --git a/configure.ac b/configure.ac index 40475dc..4877f57 100644 --- a/configure.ac +++ b/configure.ac @@ -100,6 +100,7 @@ AC_CHECK_HEADERS

Re: [PATCH xserver 2/3] modesetting: Remove #ifdefs XF86_PDEV_SERVER_FD

I see a build failure in xorg-server-1.19.99.901 on NetBSD. GEN modesetting.4 driver.c: In function 'probe_hw': driver.c:247:38: error: dereferencing pointer to incomplete type 'struct xf86_platform_device' if (platform_dev && (platform_dev->flags & XF86_PDEV_SERVER_FD)) {

Re: [PATCH xserver 2/3] modesetting: Remove #ifdefs XF86_PDEV_SERVER_FD

On Wed, Mar 14, 2018 at 01:33:28PM -0700, Alan Coopersmith wrote: > On 03/14/18 01:01 PM, Thomas Klausner wrote: > > I see a build failure in xorg-server-1.19.99.901 on NetBSD. > > Looks like the same failure I saw on Solaris and sent in a patch > for to revert the remo

Re: [ANNOUNCE] xorg-server 1.19.99.902

I see a new compilation error compared to the previous beta release (on NetBSD): gmake[4]: Entering directory '/scratch/wip/modular-xorg-server-devel/work/xorg-server-1.19.99.902/hw/xfree86' CC sdksyms.o sdksyms.c:1773:15: error: expected expression before ',' token (void *) &,

Re: [ANNOUNCE] xorg-server 1.19.99.904

I still see this build failure on NetBSD: sdksyms.c:1773:15: error: expected expression before ',' token (void *) &, /* ../../dri3/dri3.h:110 */ ^ Thomas On Tue, Apr 10, 2018 at 03:57:25PM -0400, Adam Jackson wrote: > Another

Re: [ANNOUNCE] xorg-server 1.19.99.904

On Wed, Apr 11, 2018 at 04:24:02PM -0500, Jeff Smith wrote: > On Wed, Apr 11, 2018 at 2:44 AM, Thomas Klausner wrote: > > I still see this build failure on NetBSD: > > > > sdksyms.c:1773:15: error: expected expression before &#x

Re: [PATCH xserver] sdksyms: Skip empty symbols

/dri3.h:110 */ > > I've been unable to reproduce that locally (even in a NetBSD vm), but > an obvious workaround might be to just notice empty symbol names and > ignore them rather than emit invalid C code. > > Cc: Thomas Klausner > Signed-off-by: Adam Jackson >

Re: [ANNOUNCE] xorg-server 1.19.99.904

On Wed, Apr 18, 2018 at 10:13:36AM -0500, Jeff Smith wrote: > Here is something that may help in tracking down the cause of the > disappearing symbol. > 1: apply the following changes to sdksyms.sh > 2: re-run the build > (this will output what is being fed into awk from the pre-processor > into

Re: [ANNOUNCE] xorg-server 1.19.99.904

On Fri, Apr 20, 2018 at 11:07:04AM -0400, Adam Jackson wrote: > On Fri, 2018-04-20 at 08:18 +0200, Thomas Klausner wrote: > > > extern __attribute__((visibility("default"))) > > # 112 "../../dri3/dri3.h" 3 4 > >

Re: [ANNOUNCE] xorg-server 1.20.0

On Thu, May 10, 2018 at 12:42:24PM -0400, Adam Jackson wrote: > Lots of Present, DRI3 modifier, and buildsystem fixes. Thanks to > everyone who contributed to this release! I've updated the pkgsrc package, but now lots of drivers don't build. xf86-video-tseng-1.2.5: tseng_driver.c: In function '

Re: [ANNOUNCE] xorg-server 1.20.0

On Fri, May 11, 2018 at 03:40:18PM -0700, Matt Turner wrote: > On Fri, May 11, 2018 at 2:28 PM, Thomas Klausner wrote: > > xf86-video-intel-2.99.917: > > > > checking whether to include DRI1 support... no > > configure: error: DRI1 requested but prerequisites not

Re: [ANNOUNCE] xorg-server 1.20.0

On Mon, May 14, 2018 at 02:23:48PM -0400, Adam Jackson wrote: > On Sat, 2018-05-12 at 07:48 +0200, Thomas Klausner wrote: > > On Fri, May 11, 2018 at 03:40:18PM -0700, Matt Turner wrote: > > > On Fri, May 11, 2018 at 2:28 PM, Thomas Klausner wrote: > > >

maxHValue/maxVValue replacement?

Some old drivers use maxHValue/maxVValue, which xorg-server-1.20.0 does not provide any longer. What is the proper replacement? Thanks, Thomas ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://l

Re: maxHValue/maxVValue replacement?

On Fri, May 18, 2018 at 12:48:57PM -0400, Adam Jackson wrote: > On Fri, 2018-05-18 at 08:34 +0200, Thomas Klausner wrote: > > Some old drivers use maxHValue/maxVValue, which xorg-server-1.20.0 > > does not provide any longer. What is the proper replacement? > > The only t

Re: [ANNOUNCE] xorg-server 1.20.0

On Fri, May 11, 2018 at 11:28:23PM +0200, Thomas Klausner wrote: > I've updated the pkgsrc package, but now lots of drivers don't build. > > xf86-video-tseng-1.2.5: > xf86-video-sis-0.10.9: > xf86-video-s3virge-1.10.7: > xf86-video-r128-6.10.2: > xf86-video-ati-18.

[PATCH:xf86-input-vmmouse 1/2] Replace unportable test(1) operator.

Signed-off-by: Thomas Klausner --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index 70ed3a6..55b2345 100644 --- a/configure.ac +++ b/configure.ac @@ -135,7 +135,7 @@ case $host_os in AC_DEFINE(VMMOUSE_OS_BSD, 1

[PATCH:xf86-input-vmmouse 2/2] Add NetBSD/amd64 support for iopl calls.

While here, correct AC_DEFINE usage. Signed-off-by: Thomas Klausner --- configure.ac | 13 + tools/vmmouse_iopl.c | 50 +++--- 2 files changed, 20 insertions(+), 43 deletions(-) diff --git a/configure.ac b/configure.ac index

[PATCH:xf86-input-vmmouse 2/2 v2] Add NetBSD/amd64 support for iopl calls.

Updated patch: move header include lower and be consistent in ifdef vs. if defined Thomas >From 6cbbb834ee10b4b840f893d29659e53d3719504c Mon Sep 17 00:00:00 2001 From: Thomas Klausner Date: Fri, 26 Jun 2015 10:31:53 +0200 Subject: [PATCH:xf86-input-vmmouse 2/2] Add NetBSD/amd64 support

Re: [PATCH:xf86-input-vmmouse 2/2 v2] Add NetBSD/amd64 support for iopl calls.

On Fri, Jun 26, 2015 at 02:16:25PM +0200, Thomas Hellstrom wrote: > Thanks for the patches! I've reviewed them and pushed them to git > master. Unfortunately it will take a couple of weeks until the next > release. I hope that's not too much inconvenience. No, that's no problem. I'll just apply th

xproto changes from NetBSD xsrc and pkgsrc

The pkgsrc package has attracted quite a number of patches. This is a try getting them included. Additionally, I wonder if this _POSIX_SOURCE dance for limits.h is still useful (in Xos_r.h): # ifndef X_NOT_POSIX # ifdef _POSIX_SOURCE # include # else # define _POSIX_SOURCE # include #

[PATCH:xproto 1/3] Fix FD_SETSIZE definition.

From: Matthew Green Move the define of FD_SETSIZE before the include of select.h, so that is actually does something useful. Signed-off-by: Thomas Klausner --- Xpoll.h.in | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Xpoll.h.in b/Xpoll.h.in index 01be164

[PATCH:xproto 2/3] Check for multi-threaded library calls separately.

From: Joerg Sonnenberger Make Xos_r.h a config file. Signed-off-by: Thomas Klausner --- .gitignore |1 + Xos_r.h | 1095 -- Xos_r.h.in | 1095 ++ configure.ac |9

[PATCH:xproto 3/3] Fix getpwuid_r and getpwnam_r use for NetBSD and DragonFly.

From: Joerg Sonnenberger Signed-off-by: Thomas Klausner --- Xos_r.h.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Xos_r.h.in b/Xos_r.h.in index 2e24fcc..bc45755 100644 --- a/Xos_r.h.in +++ b/Xos_r.h.in @@ -325,7 +325,7 @@ static __inline__ void _Xpw_copyPasswd

[PATCH:xfs 1/2] FD_SETSIZE fixes.

From: Matthew Green Move X11/Xpoll.h include earlier than sys/select.h to avoid missing out on setting FD_SETSIZE early enough. Signed-off-by: Thomas Klausner --- os/connection.c | 2 +- os/io.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/os/connection.c b

[PATCH:xfs 2/2] Ignore .dirstamp files.

Signed-off-by: Thomas Klausner --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index e4efc3e..8b27e7f 100644 --- a/.gitignore +++ b/.gitignore @@ -43,6 +43,7 @@ stamp-h? symlink-tree texinfo.tex ylwrap +.dirstamp # Do not edit the following

[PATCH:libX11 2/2] Ignore test-driver (used by newer autoconf).

Signed-off-by: Thomas Klausner --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 9e3e3d4..554c7d2 100644 --- a/.gitignore +++ b/.gitignore @@ -41,6 +41,7 @@ mkinstalldirs py-compile stamp-h? symlink-tree +test-driver texinfo.tex ylwrap

[PATCH:libX11 1/2] Do not return() after exit().

Signed-off-by: Thomas Klausner --- src/XlibInt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/XlibInt.c b/src/XlibInt.c index 80c1298..bbc5c82 100644 --- a/src/XlibInt.c +++ b/src/XlibInt.c @@ -1256,7 +1256,7 @@ int _XDefaultIOError( } exit(1

[PATCH:x11perf] Make NameFromFormat() const.

Fixes a const-discard warning. Signed-off-by: Thomas Klausner --- x11perf.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/x11perf.c b/x11perf.c index 3872bcb..e98cca5 100644 --- a/x11perf.c +++ b/x11perf.c @@ -167,7 +167,7 @@ static int GetRops(int argi, int argc

[PATCH:twm] Do not try to extend the font-pattern with a wild-card.

From: Michael van Elst This will almost always select the wrong fontset and use wrong metrics. Signed-off-by: Thomas Klausner --- src/util.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/src/util.c b/src/util.c index 9e5675a..4b94051 100644 --- a/src/util.c +++ b

various patches pushed

Thank you, all pushed! Now only the recent xproto patches are not reviewed. Thomas ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PATCH:xmodmap] Properly hide config.h behind HAVE_CONFIG_H.

From: Matthew Green Signed-off-by: Thomas Klausner --- handle.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/handle.c b/handle.c index 00c95a6..02a5f41 100644 --- a/handle.c +++ b/handle.c @@ -26,7 +26,10 @@ from The Open Group. */ +#ifdef HAVE_CONFIG_H #include "con

[PATCH:xf86-video-geode] Properly hide config.h inclusion inside HAVE_CONFIG_H.

From: Matthew Green Signed-off-by: Thomas Klausner --- src/geode.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/geode.h b/src/geode.h index ac69368..244cadf 100644 --- a/src/geode.h +++ b/src/geode.h @@ -28,7 +28,9 @@ #define _GEODE_H_ #include "geode_pcirename.h&quo

[PATCH:xf86-input-keyboard] Add defines for Japanese 106-key USB keyboards.

From: Yasushi Oshima Via http://gnats.netbsd.org/48347 Signed-off-by: Thomas Klausner --- src/bsd_KbdMap.c | 30 +++--- 1 file changed, 23 insertions(+), 7 deletions(-) diff --git a/src/bsd_KbdMap.c b/src/bsd_KbdMap.c index 123ab96..21f337d 100644 --- a/src

[PATCH:xf86-video-vesa] When debugging, print the VGAbase as well.

From: Matthew Green Just in case it ends up zero and causes a segmentation fault. Signed-off-by: Thomas Klausner --- src/vesa.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/vesa.c b/src/vesa.c index 564bb06..5b9ccb0 100644 --- a/src/vesa.c +++ b/src/vesa.c

[PATCH:xmag] Protect config.h with HAVE_CONFIG_H.

Signed-off-by: Thomas Klausner --- Scale.c | 2 ++ xmag.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/Scale.c b/Scale.c index f32aeec..ce2064a 100644 --- a/Scale.c +++ b/Scale.c @@ -30,7 +30,9 @@ from The Open Group. * Author: Davor Matic, MIT X Consortium */ +#ifdef

Re: [PATCH:xf86-video-geode] Properly hide config.h inclusion inside HAVE_CONFIG_H.

On Mon, Jul 20, 2015 at 05:03:13PM +0100, Emil Velikov wrote: > Hi Thomas, > > On 20 July 2015 at 09:37, Thomas Klausner wrote: > > From: Matthew Green > > > > Signed-off-by: Thomas Klausner > > --- > > src/geode.h | 2 ++ > > 1 file changed, 2 ins

Re: [PATCH:xfs 1/2] FD_SETSIZE fixes.

Hi Emil! On Mon, Jul 20, 2015 at 05:00:43PM +0100, Emil Velikov wrote: > On 19 July 2015 at 08:56, Thomas Klausner wrote: > > From: Matthew Green > > > > Move X11/Xpoll.h include earlier than sys/select.h to avoid missing > > out on setting FD_SETSIZE early enough.

Re: [PATCH:xf86-video-geode] Properly hide config.h inclusion inside HAVE_CONFIG_H.

On Wed, Jul 22, 2015 at 03:05:28PM +0100, Emil Velikov wrote: > On 21 July 2015 at 22:12, Thomas Klausner wrote: > > On Mon, Jul 20, 2015 at 05:03:13PM +0100, Emil Velikov wrote: > >> Hi Thomas, > >> > >> On 20 July 2015 at 09:37, Thomas Klausne

Re: [PATCH:xfs 1/2] FD_SETSIZE fixes.

On Wed, Jul 22, 2015 at 03:08:33PM +0100, Emil Velikov wrote: > On 21 July 2015 at 22:18, Thomas Klausner wrote: > > Hi Emil! > > > > On Mon, Jul 20, 2015 at 05:00:43PM +0100, Emil Velikov wrote: > >> On 19 July 2015 at 08:56, Thomas Klausner wrote: > >> &g

Re: [PATCH:xfs 1/2] FD_SETSIZE fixes.

On Fri, Jul 24, 2015 at 04:04:00AM -0400, Olivier Fourdan wrote: > I posted a comment to your Xpoll patch already: > > http://lists.x.org/archives/xorg-devel/2015-July/046951.html Yes, thanks for your comment. > You may disagree with my comment, just wanted to make sure we considered that. No,

[PATCH:luit] Protect config.h inclusion by HAVE_CONFIG_H.

Signed-off-by: Thomas Klausner --- other.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/other.h b/other.h index d18b586..1b156a7 100644 --- a/other.h +++ b/other.h @@ -23,7 +23,9 @@ THE SOFTWARE. #ifndef LUIT_OTHER_H #define LUIT_OTHER_H 1 +#ifdef HAVE_CONFIG_H #include "con

[PATCH:xf86-input-mouse] Add some standard defines to pkg-config file.

Not strictly needed in the xorg build, but helps the NetBSD reachover build, and follows implicit conventions about pkg-config files more closely. See also http://gnats.netbsd.org/48991 Signed-off-by: Thomas Klausner --- xorg-mouse.pc.in | 4 1 file changed, 4 insertions(+) diff --git a

FreeBSD/DragonFly patches for xf86-video-ati, xf86-input-mouse, and xf86-video-intel

Hi! David Shao reported some issues against pkgsrc, providing build/run fixes for xorg software on DragonFly BSD and FreeBSD. However, I'd like to avoid having pkgsrc-only patches. So I'm wondering what your opinions are on getting (versions of) these patches included. He says that the patches a

Re: [PATCH:x11proto] Incorrect guard block in HPkeysym.h

On Sun, Jul 26, 2015 at 08:40:10AM -0700, Alan Coopersmith wrote: > https://bugs.freedesktop.org/show_bug.cgi?id=91469 > > Signed-off-by: Alan Coopersmith Reviewed-by: Thomas Klausner > --- > HPkeysym.h |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > d

[PATCH:mkfontscale] Add new ConstList functions and use them.

Gets rid of lots of const warnings. Signed-off-by: Thomas Klausner --- Makefile.am | 2 ++ constlist.c | 93 +++ constlist.h | 35 ++ mkfontscale.c | 24 --- 4 files changed, 143 insertions(+), 11

mkfontscale const poisoning

Second try at making mkfontscale const-safe. Please review. ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PATCH:xkbcomp] Adjust a NUL-string assignment to avoid const issues.

From: Matthew Green Signed-off-by: Thomas Klausner --- expr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/expr.c b/expr.c index e0f957c..841ab07 100644 --- a/expr.c +++ b/expr.c @@ -729,8 +729,8 @@ ExprResolveString(ExprDef * expr, val_rtrn->

Re: [PATCH:xf86-video-mga] Fix pointer cast warning.

So what can we do to make this progress? Thomas On Thu, Feb 26, 2015 at 02:59:49AM -0500, Connor Behan wrote: > On 25/02/15 06:41 PM, Tormod Volden wrote: > >> And then disable the DownloadFromScreen() implementation, or perhaps > >> even disable EXA altogether. > > If nobody can find the real pr

Re: [PATCH:xf86-video-mga 2/2] Use correct type in function call.

On Tue, Feb 24, 2015 at 07:47:09AM +0100, Mark Kettenis wrote: > > On Mon, Feb 23, 2015 at 4:28 AM, Thomas Klausner wrote: > > > Fixes > > > passing argument 2 of 'pci_device_cfg_read_u32' from incompatible > > > pointer type > > > pciaccess.h:

[PATCH:xf86-video-tdfx] Don't mess with BARs if only one card is present.

From: Michael Lorenz Fixes this driver on NetBSD/macppc. Signed-off-by: Thomas Klausner --- src/tdfx_driver.c | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/src/tdfx_driver.c b/src/tdfx_driver.c index 03fa165..b15ff9c 100644 --- a/src/tdfx_driver.c

[PATCH:xserver] Move X11/Xpoll.h include earlier than sys/select.h

From: Matthew Green Assures setting FD_SETSIZE early enough. Signed-off-by: Thomas Klausner --- os/WaitFor.c| 3 ++- os/connection.c | 3 ++- os/io.c | 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/os/WaitFor.c b/os/WaitFor.c index 431f1a6..ad9fd3d 100644

Re: [PATCH:xf86-video-mga] Fix pointer cast warning.

On Sat, Aug 01, 2015 at 12:02:27AM -0400, Connor Behan wrote: > I just looked through the drivers on the main site and found 7 > problematic ones. > > * Geode is not careful enough with pSrc and its drawables > * Mga is not careful enough either, plus it has a useless DFS hook > * Radeonhd is not

[PATCH:xf86-video-xgi 11/19] Do not build in DEBUG mode by default.

Signed-off-by: Thomas Klausner --- src/xgi.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/src/xgi.h b/src/xgi.h index dbae51f..4a0a431 100755 --- a/src/xgi.h +++ b/src/xgi.h @@ -32,13 +32,6 @@ #ifndef _XGI_H_ #define _XGI_H_ -#define DEBUG -#define DEBUG1 -#define DEBUG2

[PATCH:xf86-video-xgi 06/19] Add prototype for bGetEDID.

Fixes warning: implicit declaration of function 'bGetEDID' Signed-off-by: Thomas Klausner --- src/vb_i2c.h | 1 + src/xgi_driver.c | 1 + 2 files changed, 2 insertions(+) diff --git a/src/vb_i2c.h b/src/vb_i2c.h index 69f02cd..1284e9d 100755 --- a/src/vb_i2c.h +++ b/src/vb_i2c.h

[PATCH:xf86-video-xgi 02/19] Stop using obsolete header file.

Signed-off-by: Thomas Klausner --- src/xgi_driver.c | 2 +- src/xgi_setup.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/xgi_driver.c b/src/xgi_driver.c index 8551625..7e5ede1 100755 --- a/src/xgi_driver.c +++ b/src/xgi_driver.c @@ -80,7 +80,7 @@ #include

[PATCH:xf86-video-xgi 03/19] Use correct type for variable.

Fixes many warnings of this type: xgi.h:1016:14: note: expected 'XGIIOADDRESS' but argument is of type 'PUCHAR' Signed-off-by: Thomas Klausner --- src/vb_i2c.c | 38 +++--- src/vgatypes.h | 2 +- 2 files changed, 20 insertions(+), 20 deleti

xf86-driver-xgi patches from NetBSD xsrc

This patch series contains all local changes in NetBSD xsrc for the xf86-video-xgi driver. ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

[PATCH:xf86-video-xgi 10/19] Improve format strings in messages and errors.

Signed-off-by: Thomas Klausner --- src/vb_ext.c | 2 +- src/xgi_accel.c | 7 --- src/xgi_driver.c | 18 +- 3 files changed, 14 insertions(+), 13 deletions(-) diff --git a/src/vb_ext.c b/src/vb_ext.c index 2f8c689..662cb7b 100755 --- a/src/vb_ext.c +++ b/src/vb_ext.c

[PATCH:xf86-video-xgi 05/19] Use correct variable type.

Fixes warning: passing argument 2 of 'pci_device_cfg_read_u32' from incompatible pointer type Signed-off-by: Thomas Klausner --- src/xgi_setup.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/xgi_setup.c b/src/xgi_setup.c index 2817966..310ea83 100755

[PATCH:xf86-video-xgi 09/19] Use better variable types.

Fixes a couple more warnings. Signed-off-by: Thomas Klausner --- src/vb_setmode.c | 16 src/vgatypes.h | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/vb_setmode.c b/src/vb_setmode.c index 98cd1c5..b0f64b1 100755 --- a/src/vb_setmode.c +++ b/src

[PATCH:xf86-video-xgi 04/19] Remove const marker for some variables.

They are used in non-const context. Fixes a couple of warnings: warning: assignment discards 'const' qualifier from pointer target type Signed-off-by: Thomas Klausner --- src/vb_table.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/vb_tab

[PATCH:xf86-video-xgi 08/19] Uncomment header includes.

Needed for open() and ioctl(). Signed-off-by: Thomas Klausner --- src/xgi_driver.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/xgi_driver.c b/src/xgi_driver.c index fec7b78..a8a04db 100755 --- a/src/xgi_driver.c +++ b/src/xgi_driver.c @@ -110,11 +110,8 @@ #define DEFAULT_DPI 96

[PATCH:xf86-video-xgi 12/19] Make PutImage implementation match prototype.

Fixes warning: assignment from incompatible pointer type Signed-off-by: Thomas Klausner --- src/xgi_video.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/xgi_video.c b/src/xgi_video.c index 716d6e1..420ba07 100755 --- a/src/xgi_video.c +++ b/src/xgi_video.c

[PATCH:xf86-video-xgi 07/19] Consistently use Bool instead of BOOL.

Signed-off-by: Thomas Klausner --- src/xgi.h| 8 src/xgi_driver.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/xgi.h b/src/xgi.h index a5702c4..dbae51f 100755 --- a/src/xgi.h +++ b/src/xgi.h @@ -51,7 +51,7 @@ #ifndef XGI_VIDEO_HW /* avoid compile

[PATCH:xf86-video-xgi 01/19] Avoid redefinitions of u8, uint8_t, and GETBITS.

Signed-off-by: Thomas Klausner --- src/vb_i2c.h | 6 +- src/xgi.h| 2 ++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/src/vb_i2c.h b/src/vb_i2c.h index 1e9de25..69f02cd 100755 --- a/src/vb_i2c.h +++ b/src/vb_i2c.h @@ -7,7 +7,9 @@ #endif #ifndef u32 #define u32

[PATCH:xf86-video-xgi 17/19] Make two unused Scaling variables static.

Avoids xgi_drv_pic.a(xgi_setup.pico):(.rodata+0x30): multiple definition of `XGIScalingP4Regs' Signed-off-by: Thomas Klausner --- src/xgi_driver.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/xgi_driver.h b/src/xgi_driver.h index 0754f16..fe8a8c1 100755 ---

[PATCH:xf86-video-xgi 19/19] Remove const from variable.

Fixes. assignment discards ‘const’ qualifier from pointer target type Signed-off-by: Thomas Klausner --- src/xgi_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/xgi_driver.c b/src/xgi_driver.c index 4806476..abaf1e1 100755 --- a/src/xgi_driver.c +++ b/src

[PATCH:xf86-video-xgi 15/19] Fix return value.

From: Alan Barrett In XGI_GetTVPtrIndex2(), return the correct value through *tempch by using (*tempch)++ instead of *tempch++. Found using clang -Wunused-value. Signed-off-by: Thomas Klausner --- src/vb_setmode.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src

[PATCH:xf86-video-xgi 14/19] Remove PACKAGE_* definitions from xgi_dri.c.

They are provided by configure (via config.h or command line). Signed-off-by: Thomas Klausner --- src/xgi_dri.c| 5 - src/xgi_driver.c | 6 +++--- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/src/xgi_dri.c b/src/xgi_dri.c index c4cbfed..fcfab89 100755 --- a/src

[PATCH:xf86-video-xgi 18/19] Add unistd.h for usleep().

Signed-off-by: Thomas Klausner --- src/vb_i2c.c| 1 + src/xgi_accel.c | 1 + 2 files changed, 2 insertions(+) diff --git a/src/vb_i2c.c b/src/vb_i2c.c index 90e4630..69cdb8a 100755 --- a/src/vb_i2c.c +++ b/src/vb_i2c.c @@ -2,6 +2,7 @@ #ifdef HAVE_CONFIG_H #include "config.h&quo

[PATCH:xf86-video-xgi 16/19] Compare integer return values to integers.

Fixes: warning: comparison between pointer and integer Signed-off-by: Thomas Klausner --- src/xgi_opt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/xgi_opt.c b/src/xgi_opt.c index c0608d5..54153fc 100755 --- a/src/xgi_opt.c +++ b/src/xgi_opt.c @@ -315,13 +315,13

[PATCH:xf86-video-xgi 13/19] Make cast match target variable type.

Signed-off-by: Thomas Klausner --- src/vb_ext.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/vb_ext.c b/src/vb_ext.c index 662cb7b..f4b5d68 100755 --- a/src/vb_ext.c +++ b/src/vb_ext.c @@ -139,7 +139,7 @@ void XGI_GetSenseStatus( PXGI_HW_DEVICE_INFO HwDeviceExtension

Re: [PATCH:xf86-video-xgi 14/19] Remove PACKAGE_* definitions from xgi_dri.c.

Here's an improved patch, really removing both of them. Thomas >From 86e939d6d521847ca22c43640fa50aa0f220f5c2 Mon Sep 17 00:00:00 2001 From: Thomas Klausner Date: Thu, 6 Aug 2015 14:33:02 +0200 Subject: [PATCH:xf86-video-xgi 14/19] Remove PACKAGE_* definitions from xgi_dri.c.

Re: xf86-driver-xgi patches from NetBSD xsrc

On Mon, Aug 10, 2015 at 03:13:32PM -0400, Connor Behan wrote: > On 06/08/15 08:56 AM, Thomas Klausner wrote: > > This patch series contains all local changes in NetBSD xsrc for the > > xf86-video-xgi driver. > Patch 16 appears to introduce a whitespace error. Otherwise, ser

[PATCH:xf86-video-mga] Use correct type in function call.

Fixes passing argument 2 of 'pci_device_cfg_read_u32' from incompatible pointer type pciaccess.h:153:5: note: expected '__uint32_t *' but argument is of type 'CARD32 *' Signed-off-by: Thomas Klausner --- src/mga_dac3026.c | 6 +- 1 file changed, 5 insertions(+

[PATCH:xf86-video-sis] Fix arguments for miSetPointerPosition.

For the call for the API between versions 5 and 13. --- src/sis_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/sis_driver.c b/src/sis_driver.c index 854cb54..c1d26f3 100644 --- a/src/sis_driver.c +++ b/src/sis_driver.c @@ -9397,7 +9397,7 @@ SISMergedPointerMoved(S

[PATCH:xf86-video-siliconmotion 1/2] Fix endian detection on NetBSD.

NetBSD provides BYTE_ENDIAN, but this driver looks for __BYTE_ENDIAN. --- src/smi.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/smi.h b/src/smi.h index 5e44892..f3227e0 100644 --- a/src/smi.h +++ b/src/smi.h @@ -67,6 +67,12 @@ authorization from the XFree86 Project and Silicon Mo

[PATCH:xserver] Move X11/Xpoll.h include earlier than sys/select.h

From: Matthew Green Assures setting FD_SETSIZE early enough. Signed-off-by: Thomas Klausner --- os/WaitFor.c| 3 ++- os/connection.c | 3 ++- os/io.c | 3 ++- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/os/WaitFor.c b/os/WaitFor.c index 431f1a6..ad9fd3d 100644

[PATCH:xf86-video-siliconmotion 2/2] Support Gdium.

From: Michael Lorenz Comment out Int10 support since it's useless on Gdium/SM50x. --- src/smi_driver.c | 7 +-- src/smilynx_crtc.c | 8 src/smilynx_hw.c | 5 - src/smilynx_output.c | 3 ++- 4 files changed, 15 insertions(+), 8 deletions(-) diff --git a/src/smi_driver.

Re: [PATCH:xproto 1/3] Fix FD_SETSIZE definition.

Can someone please review this set? Thanks, Thomas On Fri, Jul 17, 2015 at 10:24:17AM +0200, Thomas Klausner wrote: > From: Matthew Green > > Move the define of FD_SETSIZE before the include of select.h, so that > is actually does something useful. > > Signed-off-b

xf86-video-siliconmotion patches from NetBSD xsrc

This is not the final version, but posted as a base for discussion. Please suggest how this should be done properly. Thanks! ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/l

Re: [PATCH:xf86-video-siliconmotion 1/2] Fix endian detection on NetBSD.

On Mon, Sep 14, 2015 at 10:22:06AM -0700, Alan Coopersmith wrote: > On 09/14/15 09:39 AM, Thomas Klausner wrote: > >NetBSD provides BYTE_ENDIAN, but this driver looks for __BYTE_ENDIAN. > >--- > > src/smi.h | 6 ++ > > 1 file changed, 6 insertions(+) > > >

[PATCH:twm] Add some const.

From: Christos Zoulas Signed-off-by: Thomas Klausner --- src/util.c | 4 ++-- src/util.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/util.c b/src/util.c index 4b94051..5e8f0db 100644 --- a/src/util.c +++ b/src/util.c @@ -256,11 +256,11 @@ Zoom(Window wf, Window

Re: [PATCH:twm] Add some const.

ree. Would be nice to double-check > callers. > > On Wed, Sep 23, 2015 at 1:58 AM, Thomas Klausner wrote: > > From: Christos Zoulas > > > > Signed-off-by: Thomas Klausner > > --- > > src/util.c | 4 ++-- > > src/util.h | 2 +- > > 2 files

Re: [PATCH:twm] Add some const.

); > > ... which would still work, but the != is now dead code. If callers > were unconditionally freeing before, I heavily suspect something more > serious here. > > On Wed, Sep 23, 2015 at 9:30 AM, Thomas Klausner wrote: > > Well, yes, but for context, here is t

[PATCH:xf86-video-s3virge 1/2] Increase BIOS_BSIZE.

From: Nick Hudson libpciaccess expects a much bigger buffer when reading ROM. Bump it to the (normal) IO space size. Signed-off-by: Thomas Klausner --- src/s3v_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/s3v_driver.c b/src/s3v_driver.c index d519dc8

[PATCH:xf86-video-s3virge 2/2] Add cast to remove a warning on 32-bit machines.

PCI_REGION_BASE returns a 64-bit value, which needs to be converted down to 32-bit before it can be put into a 32-bit pointer. --- src/s3v_dga.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/s3v_dga.c b/src/s3v_dga.c index 0e603cf..0ab374d 100644 --- a/src/s3v_dga.c +++ b

Re: [PATCH:libX11] Get rid of some extraneous ; at the end of C source lines

Reviewed-by: Thomas Klausner On Sun, Sep 27, 2015 at 06:38:32PM -0700, Alan Coopersmith wrote: > Signed-off-by: Alan Coopersmith > --- > modules/im/ximcp/imExten.c |2 +- > src/xkb/XKBNames.c |2 +- > src/xlibi18n/lcPrTxt.c |2 +- > 3 files changed,

Re: [PATCH:xf86-video-sis] Fix arguments for miSetPointerPosition.

hw/xfree86/common/xf86Module.h says ABI_XINPUT_VERSION is 14. But > mi/mipointer.c still has miPointerSetPosition() taking pointers to integers. Good catch, new patch attached. Thomas >From c11f3e7bf57df27b3bcd0ab1a7b176525c248002 Mon Sep 17 00:00:00 2001 From: Thomas Klausner Date: Su

Re: [PATCH:xf86-video-siliconmotion 1/2] Fix endian detection on NetBSD.

On Wed, Sep 30, 2015 at 04:21:37PM -0400, Connor Behan wrote: > Reviewed-by: Connor Behan Thanks, pushed; and the second patch from this series is retracted. Thomas ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xor

Re: [PATCH] Stop accessing the attribute array using the attribute name enum.

The whole SMI_VideoAttributes array looks quite strange to me. static XF86AttributeRec SMI_VideoAttributes[2] = { {XvSettable | XvGettable,0, 255, XV_BRIGHTNESS_NAME}, {XvSettable | XvGettable, 0x00, 0xFF, XV_COLORKEY_NAME}, }; but it is accessed using these

[PATCH:xf86-video-modesetting] Fix build when XSERVER_PLATFORM_BUS is not defined.

From: Jared McNeill --- src/driver.c | 4 1 file changed, 4 insertions(+) diff --git a/src/driver.c b/src/driver.c index d99b027..21326d9 100644 --- a/src/driver.c +++ b/src/driver.c @@ -93,6 +93,10 @@ static const struct pci_id_match ms_device_match[] = { }; #endif +#ifndef XSERVER_PL

Re: [PATCH:xf86-video-modesetting] Fix build when XSERVER_PLATFORM_BUS is not defined.

On Mon, Nov 09, 2015 at 03:43:44PM -0500, Adam Jackson wrote: > On Fri, 2015-11-06 at 10:30 +0100, Thomas Klausner wrote: > > From: Jared McNeill > > It's not wrong, but this driver lives in-server now. Thanks for the information. I have attached anew patch against the xs

[PATCH:xf86-input-keyboard 2/2] Scancode translation table for wskbd on Amiga keyboards.

From: Frank Wille Signed-off-by: Thomas Klausner --- src/bsd_KbdMap.c | 121 +++ src/bsd_kbd.c| 5 +++ 2 files changed, 126 insertions(+) diff --git a/src/bsd_KbdMap.c b/src/bsd_KbdMap.c index 95c6bd8..4b785af 100644 --- a/src

[PATCH:xf86-input-keyboard 1/2] Whitespace cleanup.

Signed-off-by: Thomas Klausner --- src/bsd_KbdMap.c | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/src/bsd_KbdMap.c b/src/bsd_KbdMap.c index 21f337d..95c6bd8 100644 --- a/src/bsd_KbdMap.c +++ b/src/bsd_KbdMap.c @@ -20,7 +20,7

[PATCH:xf86-video-ati] Fix uninitialized variable warnings reported by clang.

Signed-off-by: Thomas Klausner --- src/radeon_drm_queue.c | 8 src/radeon_kms.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/radeon_drm_queue.c b/src/radeon_drm_queue.c index 5e54ef8..84d0509 100644 --- a/src/radeon_drm_queue.c +++ b/src

[PATCH:xserver] Fix uninitialized variable warnings reported by clang. (was Re: [PATCH:xf86-video-ati] Fix uninitialized variable warnings reported by clang.)

Like this? Thomas On Tue, Dec 15, 2015 at 05:25:49PM +0900, Michel Dänzer wrote: > On 12.12.2015 00:46, Thomas Klausner wrote: > > Signed-off-by: Thomas Klausner > > --- > > src/radeon_drm_queue.c | 8 > > src/radeon_kms.c | 2 +- > > 2

Re: [PATCH:xserver] Fix uninitialized variable warnings reported by clang. (was Re: [PATCH:xf86-video-ati] Fix uninitialized variable warnings reported by clang.)

00:00 2001 From: Thomas Klausner Date: Tue, 15 Dec 2015 11:03:39 +0100 Subject: [PATCH:xserver 3/3] Fix uninitialized variable warnings reported by clang. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Thomas Klausner Acked-by: Michel Dänze

Re: [PATCH xserver v2] Fix uninitialized variable warnings reported by clang

M -0500, Adam Jackson wrote: > From: Thomas Klausner > > v2: Move initializing pos into the first clause of the for statement. We > have to keep this macro equivalent to a plain for statement from the > user's perspective, otherwise callers need to {} things to keep control

Re: [ANNOUNCE] xorg-server 1.19.0

The release breaks most currently existing graphics driver releases and two input driver releases (details below). What is the proper fix? When are new releases planned? Thanks, Thomas chips: ct_driver.c: In function 'chipsBlockHandler': ct_driver.c:7477:5: error: too many arguments to functio

  1   2   3   4   5   >