Re: [PATCH xserver] configure.ac: unconditionally enable kdrive

2017-11-01 Thread Matt Turner
On Tue, Oct 24, 2017 at 10:27 AM, Matt Turner wrote: > Reviewed-by: Matt Turner Please merge. :) ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info:

Re: [PATCH v2 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2017-11-01 Thread Giuseppe Bilotta
On Wed, Nov 1, 2017 at 9:35 PM, Adam Jackson wrote: > On Wed, 2017-11-01 at 21:04 +0100, Giuseppe Bilotta wrote: > >> (Heck, I would even argue that the core DPI should be set to the >> primary monitor DPI by default, regardless of whether the system has >> one or more than one

[PATCH xserver] fixup: Actually add the glamor_init compat function

2017-11-01 Thread Adam Jackson
Signed-off-by: Adam Jackson --- glamor/glamor.h | 4 glamor/glamor_egl.c | 6 ++ 2 files changed, 10 insertions(+) diff --git a/glamor/glamor.h b/glamor/glamor.h index 44ad48788..7b2225b0a 100644 --- a/glamor/glamor.h +++ b/glamor/glamor.h @@ -350,6 +350,10 @@

Re: [PATCH v2 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2017-11-01 Thread Adam Jackson
On Wed, 2017-11-01 at 21:04 +0100, Giuseppe Bilotta wrote: > (Heck, I would even argue that the core DPI should be set to the > primary monitor DPI by default, regardless of whether the system has > one or more than one monitor.) The primary monitor can change at runtime. The core DPI is only

[PATCH xserver 7/8] glamor: Add glamor_get_screen_context

2017-11-01 Thread Adam Jackson
Also, fold glamor_context.h into glamor.h, and use the standard Xnest trick so we can include server headers in glamor_glx.c. Signed-off-by: Adam Jackson --- glamor/Makefile.am| 1 - glamor/glamor.c | 6 ++ glamor/glamor.h | 21

[PATCH xserver 8/8] glamor: Eliminate EGL stubs

2017-11-01 Thread Adam Jackson
glamor_init is now spelled glamor_api_init, and takes as a new parameter the winsys API initialization function. A compatibility function is added for radeon's sake since it's using glamor_init currently; it only works for EGL, which is fine since Xephyr doesn't need it. glamor_*_from_pixmap are

[PATCH xserver 2/8] dmx: Use noGlxExtension like other DDXes

2017-11-01 Thread Adam Jackson
Signed-off-by: Adam Jackson --- hw/dmx/dmxinit.c | 16 +--- hw/dmx/glxProxy/glxext.c | 2 ++ 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/hw/dmx/dmxinit.c b/hw/dmx/dmxinit.c index 24bb875fb..d1ffcc538 100644 --- a/hw/dmx/dmxinit.c +++

[PATCH xserver 3/8] xwayland: Stop printing the EGL version

2017-11-01 Thread Adam Jackson
It doesn't matter, none of this matters. Signed-off-by: Adam Jackson --- hw/xwayland/xwayland-glamor.c | 4 1 file changed, 4 deletions(-) diff --git a/hw/xwayland/xwayland-glamor.c b/hw/xwayland/xwayland-glamor.c index 260e4e0d8..8ffb40d6f 100644 ---

[PATCH xserver 4/8] glamor: Split out glamor_xv.h

2017-11-01 Thread Adam Jackson
Signed-off-by: Adam Jackson --- glamor/Makefile.am | 3 +- glamor/glamor_priv.h | 52 glamor/glamor_xv.c | 2 +- glamor/glamor_xv.h | 86 ++

[PATCH xserver 6/8] glamor: Remove unused glamor_create_screen_resources

2017-11-01 Thread Adam Jackson
Signed-off-by: Adam Jackson --- glamor/glamor.c | 25 - glamor/glamor_priv.h | 1 - 2 files changed, 26 deletions(-) diff --git a/glamor/glamor.c b/glamor/glamor.c index cf4a76a7f..9fd110ed2 100644 --- a/glamor/glamor.c +++ b/glamor/glamor.c @@

[PATCH xserver 1/8] miinitext: General cleanup

2017-11-01 Thread Adam Jackson
This really just wants to be the list of disable booleans and initialization functions, and nothing else. Stop including the protocol headers from extinit.h, remove a stray mention of xgl, and move an XInput declaration to a better place. Signed-off-by: Adam Jackson ---

[PATCH xserver 5/8] glamor: Un-inline glamor_[gs]et_screen_private

2017-11-01 Thread Adam Jackson
textdata bss dec hex filename 2134764 45210 128704 2308678 233a46 build/hw/kdrive/ephyr/Xephyr.before 2129972 45210 128704 2303886 23278e build/hw/kdrive/ephyr/Xephyr.after Signed-off-by: Adam Jackson --- glamor/glamor.c | 13 +

Re: [PATCH v2 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2017-11-01 Thread Giuseppe Bilotta
On Wed, Nov 1, 2017 at 9:14 PM, Pali Rohár wrote: > On Wednesday 01 November 2017 21:04:17 Giuseppe Bilotta wrote: >> >> Personally, I disagree with the statement that the core DPI value has >> no useful meaning in multi-monitor configuration. > > Hi! I did not write such

Re: [PATCH v2 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2017-11-01 Thread Pali Rohár
On Wednesday 01 November 2017 21:04:17 Giuseppe Bilotta wrote: > On Wed, Nov 1, 2017 at 11:25 AM, Pali Rohár wrote: > > Hello, can somebody process this my patch? > > > > There are no objections for a long time and Walter already reviewed it. > > > > On Monday 07 August 2017

Re: [PATCH v2 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2017-11-01 Thread Giuseppe Bilotta
On Wed, Nov 1, 2017 at 11:25 AM, Pali Rohár wrote: > Hello, can somebody process this my patch? > > There are no objections for a long time and Walter already reviewed it. > > On Monday 07 August 2017 18:45:22 walter harms wrote: >> Sorry, i had the idea you already got my

Re: [PATCH] modesetting: fail PreInit() if the device has zero connectors

2017-11-01 Thread Tobias Jakobi
Dave Airlie wrote: > On 24 October 2017 at 02:42, Tobias Jakobi > wrote: >> Hello Hans, >> >> >> Hans de Goede wrote: >>> Hi, >>> >>> On 20-10-17 19:08, tobias.jako...@uni-bielefeld.de wrote: On laptop systems with a dedicated (powerful) GPU A, you usually

Re: [PATCH v2 app/xrandr] Document that --dpi and --fbmm options set DPI of whole X screen

2017-11-01 Thread Pali Rohár
Hello, can somebody process this my patch? There are no objections for a long time and Walter already reviewed it. On Monday 07 August 2017 18:45:22 walter harms wrote: > Sorry, i had the idea you already got my ack. > > Reviewed-by: Walter Harms wha...@bfs.de > > Am 07.08.2017 12:56, schrieb

Re: [PATCH xserver] modesetting: setup colormap

2017-11-01 Thread Rinat Ibragimov
>Вторник, 24 октября 2017, 1:44 +03:00 от Rinat Ibragimov >: > >Signed-off-by: Rinat Ibragimov < ibragimovri...@mail.ru > >Reviewed-by: Michel Dänzer >--- > hw/xfree86/drivers/modesetting/driver.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git

Re: [PATCH xserver] xfree86: fix gamma compute when palette_size > 256

2017-11-01 Thread Yu, Qiang
>> 2. the palette_size can be bigger than crtc->gamma_size, [...] > > Can it really? In Aaron's example, both have 1024 entries. Isn't it 2^11 for palette_size but 2^10 gamma_size? >> So according to the above understanding, I think there're two solution to >> the problem: >> 1. DDX is

Re: [PATCH xserver] xfree86: fix gamma compute when palette_size > 256

2017-11-01 Thread Michel Dänzer
On 01/11/17 02:20 AM, Yu, Qiang wrote: > Aaron, Michel, thanks for your comments which make me get more clear about > the notation involved in this patch. As my understanding so far: > 1. the crtc->gamma_size should be a hardware setting that reflect the hardware > capability of max gamma CLUT

Re: [PATCH] modesetting: fail PreInit() if the device has zero connectors

2017-11-01 Thread Dave Airlie
On 24 October 2017 at 02:42, Tobias Jakobi wrote: > Hello Hans, > > > Hans de Goede wrote: >> Hi, >> >> On 20-10-17 19:08, tobias.jako...@uni-bielefeld.de wrote: >>> On laptop systems with a dedicated (powerful) GPU A, you usually >>> have all connectors routed to

Re: [PATCH] modesetting: fail PreInit() if the device has zero connectors

2017-11-01 Thread Dave Airlie
On 21 October 2017 at 03:08, wrote: > On laptop systems with a dedicated (powerful) GPU A, you usually > have all connectors routed to another (less-powerful) GPU B. > > With my setup (GPU A = Nvidia, GPU B = Intel) I keep GPU A switched > off by not loading the