mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Markus Niebel <niebelm@tqsc.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH v4] libdrm: version bump 2.7.74 -> 2.7.75
Date: Fri, 17 Mar 2017 11:15:12 +0100	[thread overview]
Message-ID: <1489745712.31279.3.camel@niebel-ws> (raw)
In-Reply-To: <20170317092912.xucf6h7slhk4d4da@pengutronix.de>

Am Freitag, den 17.03.2017, 10:29 +0100 schrieb Michael Olbrich:
> On Thu, Mar 16, 2017 at 03:47:58PM +0100, Markus Niebel wrote:
> > From: Markus Niebel <Markus.Niebel@tq-group.com>
> > 
> > - drop not longer nneded libudev dependency
> > - make udev support optional (udev is not needed to build but
> >   if compiled with udev support udev is expected to handle device
> >   nodes)
> > 
> > Signed-off-by: Markus Niebel <Markus.Niebel@tq-group.com>
> > ---
> > changes since v3:
> > 
> > - cleanup leftovers from git merge conflicts (again)
> > 
> > changes since v2:
> > - handle as single patch, part of original series was applied
> > - fix udev dependency
> 
> I meant an unconditional dependency. I looked at this some more. The
> '--enable-udev' options is not named correctly. What the corresponding code
> does is if a drm device is missing, then either wait for it (--enable-udev)
> or create it (--disable-udev).
> 
> So for any system that hat either udev or devtmpfs '--enable-udev' should
> be used.
> So unless you have a use-case where this is not the case, I will just fixup
> the patch and drop the udev dependency entirely and set --enable-udev
> unconditionally.

Thank you for explaining it. I read the relevant code, was misguided by
--enable-udev and did not thought of devtmpfs.

Markus

> 
> Michael
> 
> > - fix commit message for new version
> > 
> > changes since v1:
> > 
> > - cleanup leftovers from git merge conflicts
> > 
> > 
> >  rules/libdrm.in   | 10 ++++++++--
> >  rules/libdrm.make |  6 +++---
> >  2 files changed, 11 insertions(+), 5 deletions(-)
> > 
> > diff --git a/rules/libdrm.in b/rules/libdrm.in
> > index 5e77665..385af35 100644
> > --- a/rules/libdrm.in
> > +++ b/rules/libdrm.in
> > @@ -4,8 +4,7 @@ menuconfig LIBDRM
> >  	tristate
> >  	prompt "libdrm                        "
> >  	select LIBC_RT
> > -	select UDEV
> > -	select UDEV_LIBUDEV
> > +	select UDEV if LIBDRM_USE_UDEV && RUNTIME
> >  	select LIBPTHREAD_STUBS
> >  	select LIBPCIACCESS	if LIBDRM_INTEL
> >  	help
> > @@ -14,6 +13,13 @@ menuconfig LIBDRM
> >  
> >  if LIBDRM
> >  
> > +config LIBDRM_USE_UDEV
> > +	bool
> > +	prompt "Enable support for udev"
> > +	help
> > +	  Enable support for using udev instead of mknod. If enabled, udev is
> > +	  expected to manage device nodes for probed modules
> > +
> >  config LIBDRM_LIBKMS
> >  	bool
> >  	prompt "build libkms"
> > diff --git a/rules/libdrm.make b/rules/libdrm.make
> > index ec02cc7..7a2f2be 100644
> > --- a/rules/libdrm.make
> > +++ b/rules/libdrm.make
> > @@ -17,8 +17,8 @@ PACKAGES-$(PTXCONF_LIBDRM) += libdrm
> >  #
> >  # Paths and names
> >  #
> > -LIBDRM_VERSION	:= 2.4.74
> > -LIBDRM_MD5	:= b661a54514109caad3de3b520680b98e
> > +LIBDRM_VERSION	:= 2.4.75
> > +LIBDRM_MD5	:= 743c16109d91a2539dfc9cc56130d695
> >  LIBDRM		:= libdrm-$(LIBDRM_VERSION)
> >  LIBDRM_SUFFIX	:= tar.gz
> >  LIBDRM_URL	:= http://dri.freedesktop.org/libdrm/$(LIBDRM).$(LIBDRM_SUFFIX)
> > @@ -61,7 +61,7 @@ LIBDRM_BACKENDSL-y += $(LIBDRM_BACKENDS-y)
> >  LIBDRM_CONF_TOOL := autoconf
> >  LIBDRM_CONF_OPT := \
> >  	$(CROSS_AUTOCONF_USR) \
> > -	--enable-udev \
> > +	--$(call ptx/endis, PTXCONF_LIBDRM_USE_UDEV)-udev \
> >  	--$(call ptx/endis, PTXCONF_LIBDRM_LIBKMS)-libkms \
> >  	$(addprefix --enable-,$(LIBDRM_BACKENDSC-y)) \
> >  	$(addprefix --disable-,$(LIBDRM_BACKENDSC-)) \
> > -- 
> > 1.9.1
> > 
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> 



_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

      reply	other threads:[~2017-03-17 10:15 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-16 14:47 Markus Niebel
2017-03-17  9:29 ` Michael Olbrich
2017-03-17 10:15   ` Markus Niebel [this message]

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=1489745712.31279.3.camel@niebel-ws \
    --to=niebelm@tqsc.de \
    --cc=ptxdist@pengutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox