All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Sweet <asweet@thegoodpenguin.co.uk>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package/cog: add BR2_PACKAGE_COG_USE_SYSTEM_DBUS
Date: Mon, 1 Mar 2021 16:55:49 +0000	[thread overview]
Message-ID: <CAH=7X1AyXmWKnV+57tX9w_Smgz413jsYYVJfzv1X8arz7Bs1zA@mail.gmail.com> (raw)
In-Reply-To: <20210204223323.GB441747@momiji>

All,
Is there anything preventing this being merged? Can I help with any issues?
Thanks, Adrian

On Thu, 4 Feb 2021 at 20:33, Adrian Perez de Castro <aperez@igalia.com>
wrote:

> Hi Adrian,
>
> Thanks for the patch, I think adding the option is fine given that there
> are people around who would use it?which seems to be the case.
>
> Unless somebody else has an opinion against, I think we could merge this
> change =]
>
> On Tue, 02 Feb 2021 11:57:21 +0000 Adrian Sweet <
> asweet at thegoodpenguin.co.uk> wrote:
> > Added config option to expose browser remote control on the D-Bus system
> bus
> >
> > Signed-off-by: Adrian Sweet <asweet@thegoodpenguin.co.uk>
>
> Reviewed-by: Adrian Perez de Castro <aperez@igalia.com>
>
> > ---
> >  package/cog/Config.in | 6 ++++++
> >  package/cog/cog.mk    | 6 ++++++
> >  2 files changed, 12 insertions(+)
> >
> > diff --git a/package/cog/Config.in b/package/cog/Config.in
> > index a9ca7473a9..c19f1c200e 100644
> > --- a/package/cog/Config.in
> > +++ b/package/cog/Config.in
> > @@ -47,6 +47,12 @@ config BR2_PACKAGE_COG_PLATFORM_DRM
> >         with video drivers that support kernel mode-setting (KMS)
> >         via the DRM user-space API.
> >
> > +config BR2_PACKAGE_COG_USE_SYSTEM_DBUS
> > +     bool "Use D-Bus system bus"
> > +     default n
> > +     help
> > +       Expose remote control interface on system bus
> > +
> >  comment "DRM platform needs mesa3d w/ EGL driver and GBM"
> >       depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL
> >
> > diff --git a/package/cog/cog.mk b/package/cog/cog.mk
> > index 47a5573dda..c5a1cc0df2 100644
> > --- a/package/cog/cog.mk
> > +++ b/package/cog/cog.mk
> > @@ -31,4 +31,10 @@ else
> >  COG_CONF_OPTS += -DCOG_PLATFORM_DRM=OFF
> >  endif
> >
> > +ifeq ($(BR2_PACKAGE_COG_USE_SYSTEM_DBUS),y)
> > +COG_CONF_OPTS += -DCOG_DBUS_SYSTEM_BUS=ON
> > +else
> > +COG_CONF_OPTS += -DCOG_DBUS_SYSTEM_BUS=OFF
> > +endif
> > +
> >  $(eval $(cmake-package))
> > --
> > 2.25.1
> >
>


-- 
Adrian Sweet, Software Engineer
https://www.thegoodpenguin.co.uk

The Good Penguin Ltd is a company registered in England and Wales with
company number 12374667 and VAT number 341687879. Registered office: The
Good Penguin Ltd, Westcott, Glasllwch Lane, Newport, NP20 3PS.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20210301/b9687bea/attachment.html>

  reply	other threads:[~2021-03-01 16:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-02 11:57 [Buildroot] [PATCH 1/1] package/cog: add BR2_PACKAGE_COG_USE_SYSTEM_DBUS Adrian Sweet
2021-02-04 20:33 ` Adrian Perez de Castro
2021-03-01 16:55   ` Adrian Sweet [this message]
2021-03-04 15:25     ` Adrian Perez de Castro
2021-03-04 16:18       ` Bartosz Bilas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAH=7X1AyXmWKnV+57tX9w_Smgz413jsYYVJfzv1X8arz7Bs1zA@mail.gmail.com' \
    --to=asweet@thegoodpenguin.co.uk \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.