Re: [PATCH xserver 08/10] xf86-video-modesetting: Create CONNECTOR_ID properties for outputs

2018-01-23 Thread Keith Packard
Adam Jackson writes: > I think I do like an enum better, even if only for the xrandr output > formatting: > > CONNECTOR_ID: 58 > range: (0, 2147483647) > non-desktop: 0 > supported: 0, 1 > > If CONNECTOR_ID were an enum that'd just be "supported" with onl

Re: [PATCH xserver 08/10] xf86-video-modesetting: Create CONNECTOR_ID properties for outputs

2018-01-23 Thread Adam Jackson
On Tue, 2018-01-23 at 11:32 +1100, Keith Packard wrote: > Adam Jackson writes: > > > > +if (name != BAD_RESOURCE) { > > > +err = RRConfigureOutputProperty(output->randr_output, name, > > > +FALSE, TRUE, TRUE, > > > +

Re: [PATCH xserver 08/10] xf86-video-modesetting: Create CONNECTOR_ID properties for outputs

2018-01-22 Thread Keith Packard
Adam Jackson writes: >> +if (name != BAD_RESOURCE) { >> +err = RRConfigureOutputProperty(output->randr_output, name, >> +FALSE, TRUE, TRUE, >> +2, prop_range); > > There's no good reason to

Re: [PATCH xserver 08/10] xf86-video-modesetting: Create CONNECTOR_ID properties for outputs

2018-01-22 Thread Adam Jackson
On Thu, 2017-12-21 at 18:54 -0800, Keith Packard wrote: > diff --git a/hw/xfree86/drivers/modesetting/drmmode_display.c > b/hw/xfree86/drivers/modesetting/drmmode_display.c > index 1ec232f70..ef3dba4cb 100644 > --- a/hw/xfree86/drivers/modesetting/drmmode_display.c > +++ b/hw/xfree86/drivers/mode

[PATCH xserver 08/10] xf86-video-modesetting: Create CONNECTOR_ID properties for outputs

2017-12-21 Thread Keith Packard
This lets a DRM client map between X outputs and kernel connectors. Signed-off-by: Keith Packard --- hw/xfree86/drivers/modesetting/drmmode_display.c | 24 1 file changed, 24 insertions(+) diff --git a/hw/xfree86/drivers/modesetting/drmmode_display.c b/hw/xfree86/drive