All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: Martin Jansa <martin.jansa@gmail.com>
Cc: Christophe Chapuis <chris.chapuis@gmail.com>,
	openembedded-devel@lists.openembedded.org,
	Khem Raj <raj.khem@gmail.com>, Fabio Estevam <festevam@denx.de>
Subject: Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include
Date: Sun, 17 Mar 2024 17:02:08 +0100	[thread overview]
Message-ID: <cbb4d19a-48de-46cc-993b-794ab2c2a583@denx.de> (raw)
In-Reply-To: <CA+chaQdg+Qz2RVBS3jc16=WhNPBzRGEcV2RURzOcuFJTuTfjJw@mail.gmail.com>

On 3/17/24 8:46 AM, Martin Jansa wrote:
> The extra -I flag won't harm even when libdrm isn't there.
> 
> lvgl-demo-fb also doesn't do it conditionally based on drm PACKAGECONFIG.
> 
> If someone want's to fix this properly then should also use pkg-config
> (and tests/FindLibDRM.cmake in the lvgl repo), but lets not block this
> fix based on that.

The CMake fix is already being worked on upstream.

For completeness:

Reviewed-by: Marek Vasut <marex@denx.de>


  reply	other threads:[~2024-03-17 16:22 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-16 10:00 [meta-oe][PATCH v4 1/7] lvgl: fix typo in lv-conf.inc Christophe Chapuis
2024-03-16 10:00 ` [meta-oe][PATCH v4 2/7] lvgl: install lv_conf.h Christophe Chapuis
2024-03-16 20:41   ` Marek Vasut
2024-03-16 10:00 ` [meta-oe][PATCH v4 3/7] lvgl: remove useless FILES include Christophe Chapuis
2024-03-16 20:41   ` Marek Vasut
2024-03-16 10:00 ` [meta-oe][PATCH v4 4/7] lvgl: cleanup sed instructions in lv-conf.inc Christophe Chapuis
2024-03-16 20:43   ` Marek Vasut
2024-03-16 10:00 ` [meta-oe][PATCH v4 5/7] lvgl: add more variables to lv-conf.inc Christophe Chapuis
2024-03-16 20:45   ` Marek Vasut
2024-03-16 10:00 ` [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include Christophe Chapuis
2024-03-16 16:08   ` Khem Raj
2024-03-16 16:17     ` Christophe Chapuis
2024-03-16 16:30       ` Khem Raj
2024-03-16 20:42   ` Marek Vasut
2024-03-16 21:55     ` Christophe Chapuis
2024-03-17  5:44       ` Marek Vasut
2024-03-17  7:46         ` [oe] " Martin Jansa
2024-03-17 16:02           ` Marek Vasut [this message]
2024-03-16 10:00 ` [meta-oe][PATCH v4 7/7] lvgl: lv-conf.inc: generalize sed instructions Christophe Chapuis
2024-03-16 20:44   ` Marek Vasut
2024-03-18 13:10   ` [oe] " Peter Kjellerstedt
2024-03-18 14:01     ` Marek Vasut
2024-03-18 15:42       ` Christophe Chapuis
2024-03-18 21:59         ` Marek Vasut
2024-03-21 20:49           ` Peter Kjellerstedt
2024-03-16 20:41 ` [meta-oe][PATCH v4 1/7] lvgl: fix typo in lv-conf.inc Marek Vasut

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=cbb4d19a-48de-46cc-993b-794ab2c2a583@denx.de \
    --to=marex@denx.de \
    --cc=chris.chapuis@gmail.com \
    --cc=festevam@denx.de \
    --cc=martin.jansa@gmail.com \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=raj.khem@gmail.com \
    /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.