[PATCH weston 2/2] compositor-drm: remove connector option

[Ucan, Emre (ADITG/ESB)]

yes

Best regards

Emre Ucan
Engineering Software Base (ADITG/ESB)

Tel. +49 5121 49 6937

> -----Original Message-----
> From: wayland-devel [mailto:wayland-devel-
> bounces at lists.freedesktop.org] On Behalf Of Pekka Paalanen
> Sent: Montag, 4. September 2017 14:20
> To: Ucan, Emre (ADITG/ESB)
> Cc: wayland-devel at lists.freedesktop.org
> Subject: Re: [PATCH weston 2/2] compositor-drm: remove connector option
> 
> On Mon, 4 Sep 2017 12:14:16 +0000
> "Ucan, Emre (ADITG/ESB)" <eucan at de.adit-jv.com> wrote:
> 
> > Hi Pekka,
> >
> > You can use it now in your work.
> 
> Hi Emre,
> 
> yes, naturally, but is it ok to land it upstream without patch 1? :-)
> 
> If it's ok, I can land it today.
> 
> 
> Thanks,
> pq
> 
> > > -----Original Message-----
> > > From: Pekka Paalanen [mailto:ppaalanen at gmail.com]
> > > Sent: Montag, 4. September 2017 13:59
> > > To: Ucan, Emre (ADITG/ESB)
> > > Cc: wayland-devel at lists.freedesktop.org
> > > Subject: Re: [PATCH weston 2/2] compositor-drm: remove connector
> option
> > >
> > > On Mon, 24 Jul 2017 15:01:27 +0300
> > > Pekka Paalanen <ppaalanen at gmail.com> wrote:
> > >
> > > > On Wed, 8 Mar 2017 15:43:18 +0000
> > > > "Ucan, Emre (ADITG/SW1)" <eucan at de.adit-jv.com> wrote:
> > > >
> > > > > Remove the option, because it is hard to use.
> > > > > Drm connector ids are hard to reach for users,
> > > > > and they can change when kernel or device tree
> > > > > is modified.
> > > > >
> > > > > If someone wants to enable just one output and
> > > > > disable all others, he can set "require-output-config".
> > > > > Then, all unconfigured outputs will be disabled
> > > > > automatically.
> > > > >
> > > > > Signed-off-by: Emre Ucan <eucan at de.adit-jv.com>
> > > > > ---
> > > > >  compositor/main.c          |    2 --
> > > > >  libweston/compositor-drm.c |   12 +-----------
> > > > >  libweston/compositor-drm.h |    6 ------
> > > > >  man/weston-drm.man         |    5 -----
> > > > >  4 files changed, 1 insertion(+), 24 deletions(-)
> > > >
> > > > > diff --git a/libweston/compositor-drm.h b/libweston/compositor-
> drm.h
> > > > > index 0870719..6f37735 100644
> > > > > --- a/libweston/compositor-drm.h
> > > > > +++ b/libweston/compositor-drm.h
> > > > > @@ -98,12 +98,6 @@ weston_drm_output_get_api(struct
> > > weston_compositor *compositor)
> > > > >  struct weston_drm_backend_config {
> > > > >  	struct weston_backend_config base;
> > > > >
> > > > > -	/** The connector id of the output to be initialized.
> > > > > -	 *
> > > > > -	 * A value of 0 will enable all available outputs.
> > > > > -	 */
> > > > > -	int connector;
> > > > > -
> > > >
> > > > Hi,
> > > >
> > > > you are removing a field from struct weston_drm_backend_config,
> > > > breaking ABI. This should be accompanied by a bump to
> > > > WESTON_DRM_BACKEND_CONFIG_VERSION.
> > > >
> > > > Otherwise the patch looks good, but I managed to look in here too late.
> > > > Since it is removing a command line option from Weston, I think this
> > > > has to wait a bit more for the next development cycle.
> > > >
> > > > Acked-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> > >
> > > Hi Emre,
> > >
> > > I'm doing some work which would benefit from this patch being in. Do
> > > you want to revise patch 1/2 first or can I just take patch 2/2 and
> > > land it with the required modifications?
> > >
> > > There's no hurry yet, I can just pick your patch in my development
> > > branch which depends on the atomic modesetting series as well.
> > >
> > >
> > > Thanks,
> > > pq