Re: [PATCH xf86-video-dummy 5/6] Get rid of dPtr->FBBase

2016-12-09 Thread Bob Terek
On 01/24/2015 03:08 PM, Aaron Plattner wrote: The pointer to the screen's pixels is stored in the screen pixmap's devPrivate.ptr field. Just use that instead of storing it redundantly. In addition, don't leak the screen if the server was VT switched. Signed-off-by: Aaron Plattner I tried t

Re: [PATCH xf86-video-dummy 4/6] Remove DGA support

2016-12-09 Thread Bob Terek
On 01/24/2015 03:08 PM, Aaron Plattner wrote: DGA was so 2004. Signed-off-by: Aaron Plattner Applied patch to current dummy driver, built, started server and a few simple clients, used x11vnc to interact with it, so: Reviewed-by: Bob Terek Tested-by: Bob Terek Cheers, Bob Terek ---

Re: [PATCH xf86-video-dummy 3/6] Delete dPtr->screenSaver

2016-12-09 Thread Bob Terek
On 01/24/2015 03:08 PM, Aaron Plattner wrote: This is assigned, but never used. Signed-off-by: Aaron Plattner Applied patch to current dummy driver, built, started server and a few simple clients, used x11vnc to interact with it, so: Reviewed-by: Bob Terek Tested-by: Bob Terek Cheers, B

Re: [PATCH xf86-video-dummy 2/6] Delete XV stuff

2016-12-09 Thread Bob Terek
On 01/24/2015 03:08 PM, Aaron Plattner wrote: As far as I can tell, this was never implemented or used. Signed-off-by: Aaron Plattner Applied patch to current dummy driver, built, started server and a few simple clients, used x11vnc to interact with it, so: Reviewed-by: Bob Terek Tested-by

Re: [PATCH xserver 04/11] dix: Don't limit GCperDepth to MAXFORMATS

2016-12-09 Thread Keith Packard
Adam Jackson writes: > GCperDepth is the scratch GC pool. When we look up a scratch GC we match > against depth and pay no attention to MAXFORMATS, so we can size the > array dynamically based on how many depths are supported. You could go nuts and just have a static array indexed by depth. --

Re: [PATCH xserver 02/11] dix: Clear graphicsExposures for scratch GCs in one place

2016-12-09 Thread Keith Packard
Adam Jackson writes: > ... instead of in all the CreateScratchGC callers. Reviewed-by: Keith Packard -- -keith signature.asc Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel

Re: [PATCH xserver 01/11] dix: Rename (and retype) PixmapPerDepth[1] to defaultStipple

2016-12-09 Thread Keith Packard
Adam Jackson writes: > Signed-off-by: Adam Jackson Wow. This has been like this since X11R1. Reviewed-by: Keith Packard -- -keith signature.asc Description: PGP signature ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org

Re: [PATCH xf86-video-dummy 0/4] Cleanups Redux

2016-12-09 Thread Bob Terek
On 12/09/2016 03:13 AM, Emil Velikov wrote: On 6 December 2016 at 22:41, Bob Terek wrote: Resubmitting some of Aaron Plattner's cleanup patches to xf86-video-dummy: https://lists.x.org/archives/xorg-devel/2015-January/045395.html . . . . . . Afaict the patches are literally unchanged si

[PATCH xserver 02/11] dix: Clear graphicsExposures for scratch GCs in one place

2016-12-09 Thread Adam Jackson
... instead of in all the CreateScratchGC callers. Signed-off-by: Adam Jackson --- dix/gc.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/dix/gc.c b/dix/gc.c index 960e868..cb8db85 100644 --- a/dix/gc.c +++ b/dix/gc.c @@ -811,6 +811,7 @@ CreateScratchGC(ScreenPtr pS

[PATCH xserver 04/11] dix: Don't limit GCperDepth to MAXFORMATS

2016-12-09 Thread Adam Jackson
GCperDepth is the scratch GC pool. When we look up a scratch GC we match against depth and pay no attention to MAXFORMATS, so we can size the array dynamically based on how many depths are supported. Signed-off-by: Adam Jackson --- dix/gc.c | 15 +++ include/scrnintstr.h

[PATCH xserver 03/11] dix: Don't special-case a 1bpp scratch GC

2016-12-09 Thread Adam Jackson
The comment in CreateGCperDepth here is wrong, there is already a 1bpp entry in the pixmap format list. Signed-off-by: Adam Jackson --- dix/gc.c | 10 -- hw/dmx/dmxextension.c | 4 ++-- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/dix/gc.c b/dix/gc.c index

[PATCH xserver 06/11] dmx: Remove scrnDefDrawables to remove a use of MAXFORMATS

2016-12-09 Thread Adam Jackson
We only kinda needed this for the CreateGC screen hook, because that gets called only with the GC itself so we don't have a drawable already on the backend to point to. But since we know the GC depth, we can just create a 1x1 pixmap at that depth on the fly instead. The other place this was used w

[PATCH xserver 07/11] shadowfb: Rename this particular shadow.c to shadowfb.c

2016-12-09 Thread Adam Jackson
So as not to conflict with the one in miext/shadow. Signed-off-by: Adam Jackson --- hw/xfree86/shadowfb/Makefile.am | 2 +- hw/xfree86/shadowfb/{shadow.c => shadowfb.c} | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename hw/xfree86/shadowfb/{shadow.c => shadowfb.c} (100%) d

[PATCH xserver 09/11] shadow: Remove indirection stubs

2016-12-09 Thread Adam Jackson
These are no longer used in the drivers. Signed-off-by: Adam Jackson --- miext/shadow/shadow.h | 5 - miext/shadow/shpacked.c | 6 -- miext/shadow/shplanar.c | 12 miext/shadow/shrotate.c | 6 -- 4 files changed, 29 deletions(-) diff --git a/miext/shadow/shadow.h b

[PATCH xserver 08/11] shadow: Lift 32->24 conversion from modesetting to dix

2016-12-09 Thread Adam Jackson
Signed-off-by: Adam Jackson --- hw/xfree86/drivers/modesetting/Makefile.am| 2 -- hw/xfree86/drivers/modesetting/driver.c | 7 +++ hw/xfree86/drivers/modesetting/sh3224.h | 7 --- miext/shadow/Makefile.am

[PATCH xserver 05/11] vfb: Remove -pixdepths option

2016-12-09 Thread Adam Jackson
This really isn't especially useful, and we'd rather not be in the business of supporting arbitrary pixel formats. Signed-off-by: Adam Jackson --- hw/vfb/InitOutput.c | 18 -- hw/vfb/man/Xvfb.man | 13 - 2 files changed, 31 deletions(-) diff --git a/hw/vfb/InitOutput

[PATCH xserver 11/11] shadow: Remove unused shadowAlloc

2016-12-09 Thread Adam Jackson
Signed-off-by: Adam Jackson --- miext/shadow/Makefile.am | 1 - miext/shadow/shadow.h| 2 -- miext/shadow/shalloc.c | 49 3 files changed, 52 deletions(-) delete mode 100644 miext/shadow/shalloc.c diff --git a/miext/shadow/Makefile.am b/m

[PATCH xserver 10/11] shadow: Macro cleanup

2016-12-09 Thread Adam Jackson
shadowDamage is just obfuscation. The other two macros won't work outside shadow.c since the private key is in fact static there (meaning the extern decl is a lie). Signed-off-by: Adam Jackson --- hw/xwin/winshadddnl.c | 2 +- hw/xwin/winshadgdi.c | 2 +- miext/shadow/sh3224.c |

[PATCH xserver 01/11] dix: Rename (and retype) PixmapPerDepth[1] to defaultStipple

2016-12-09 Thread Adam Jackson
Signed-off-by: Adam Jackson --- dix/gc.c | 13 ++--- hw/dmx/dmxextension.c | 10 +- hw/xnest/Screen.c | 2 +- include/scrnintstr.h | 12 +--- 4 files changed, 17 insertions(+), 20 deletions(-) diff --git a/dix/gc.c b/dix/gc.c index 78f3be2..960e868 1006

[PATCH xserver 00/11] Misc format and shadow cleanups

2016-12-09 Thread Adam Jackson
Just some stuff that bothered me while iterating the 24bpp series. This all stands on its own, nothing too controversial. 34 files changed, 385 insertions(+), 570 deletions(-) - ajax ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists

Re: [PATCH xserver 0/5] Remove (most) 24bpp support

2016-12-09 Thread Emil Velikov
On 8 December 2016 at 19:23, Mark Kettenis wrote: > The KMS drivers in question are unfortunately GPL'ed, which for some > non-Linux OSes is a bit of an issue. I've mentioned this on dri-devel@ so mostly repeating it here - consider reaching out to driver author(s)/maintainers wrt re-licensing re

Re: [PATCH xf86-video-dummy 0/4] Cleanups Redux

2016-12-09 Thread Emil Velikov
Hi Bob, On 6 December 2016 at 22:41, Bob Terek wrote: > Resubmitting some of Aaron Plattner's cleanup patches to xf86-video-dummy: > > https://lists.x.org/archives/xorg-devel/2015-January/045395.html > > This series contains Aaron's patches numbered 2 - 5, as patch 1 has already > been applied,

[PATCH xserver v2 4/5] xwayland: fix 'buffer' may be used uninitialized warning

2016-12-09 Thread Pekka Paalanen
From: Pekka Paalanen Fix the following warning due to --disable-glamor: CC Xwayland-xwayland.o In file included from /home/pq/local/include/wayland-client.h:40:0, from xwayland.h:35, from xwayland.c:26: xwayland.c: In function ‘block_handler’: /home/pq/l

[PATCH xserver v2 5/5] xwayland: use _XWAYLAND_ALLOW_COMMITS property

2016-12-09 Thread Pekka Paalanen
From: Pekka Paalanen The X11 window manager (XWM) of a Wayland compositor can use the _XWAYLAND_ALLOW_COMMITS property to control when Xwayland sends wl_surface.commit requests. If the property is not set, the behaviour remains what it was. XWM uses the property to inhibit commits until the wind

[PATCH xserver v2 1/5] dix: Pass the whole property into deliverPropertyNotifyEvent

2016-12-09 Thread Pekka Paalanen
From: Adam Jackson Instead of just the atom. No functional change. Signed-off-by: Adam Jackson Signed-off-by: Pekka Paalanen --- dix/property.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/dix/property.c b/dix/property.c index fa4da2d..3447bca 100644 ---

[PATCH xserver v2 3/5] xwayland: refactor into xwl_window_post_damage()

2016-12-09 Thread Pekka Paalanen
From: Pekka Paalanen Refactor xwl_screen_post_damage() and split the window specific code into a new function xwl_window_post_damage(). This is a pure refactoring, there are no behavioral changes. An assert is added to xwl_window_post_damage() to ensure frame callbacks are not leaked if a future

[PATCH xserver v2 2/5] dix: Add a callback chain for window property state change

2016-12-09 Thread Pekka Paalanen
From: Adam Jackson This will be used by in-server features that need to react to property changes. The first one will be _XWAYLAND_ALLOW_COMMITS. Signed-off-by: Adam Jackson [Pekka: add commit message body] Signed-off-by: Pekka Paalanen --- dix/property.c | 9 + include/property.h

[PATCH xserver v2 0/5] Allow XWM to control Xwayland commits

2016-12-09 Thread Pekka Paalanen
From: Pekka Paalanen Hi, this is v2 of _XWAYLAND_ALLOW_COMMITS property support in Xwayland to allow the window manager to control when wl_surface.commits occur. The first revision was posted here: https://lists.x.org/archives/xorg-devel/2016-November/051913.html The fundamental problem is that

2 equal touches at same time

2016-12-09 Thread Johann Obermayr
Hello, we use evdev-drv version 2.7.3 (yocto) and X-Server 1.14.0 we have connect 2 equal touches over usb at your cpu (i.mx6 cpu) now we see, that evdev_drv mix the events from both touches. example: touch two sends ABS_X 101 ABS_Y 100 BTN_TOUCH 1 sync touch one sends ABS_X