All of lore.kernel.org
 help / color / mirror / Atom feed
From: Emil Velikov <emil.l.velikov@gmail.com>
To: Chih-Wei Huang <cwhuang@android-x86.org>
Cc: "Jiyong Park" <jiyong@google.com>,
	"Rob Herring" <robherring2@gmail.com>,
	"ML dri-devel" <dri-devel@lists.freedesktop.org>,
	"Sean Paul" <seanpaul@google.com>,
	"Stéphane Marchesin" <marcheu@google.com>
Subject: Re: [PATCH libdrm] android: make libdrm*.so available to the vendor partition
Date: Mon, 11 Dec 2017 19:28:28 +0000	[thread overview]
Message-ID: <CACvgo51t23GmFV-QCWKDPMU9A5d8Bic8vPGErMGf9Kt9KHpoGQ@mail.gmail.com> (raw)
In-Reply-To: <CAKc24n3RrL9SCOxt6h1euaZsEE2r+LreBFJLLwoNp8UOc0rRjw@mail.gmail.com>

On 11 December 2017 at 03:51, Chih-Wei Huang <cwhuang@android-x86.org> wrote:

> Please rebase to the latest master and re-submit the patch.
> In particular, please consider the master already has
> a Rob Herring's patch which moved libdrm* to /vendor.
>
This please.

>
>
> 2017-07-26 18:08 GMT+08:00 Jiyong Park <jiyong@google.com>:
>> libdrm_<vendor>.so are moved to the vendor partition (/vendor/lib or
>> /system/vendor/lib if there is no dedicated vendor partition), since
>> they are vendor-specific extension that must not be in the system
>> partition which should be generic.
>>
>> libdrm.so (which is generic) is built/installed twice: once to
>> /vendor/lib to satisfy the dependency for the libdrm_<vendor>.so libs
>> and once to /system/lib for platform clients such as the recovery
>> executable.
>>
Strange I though we already have both issues addressed?
Admittedly the /vendor move was recent, although the static build of
libdrm (for recovery) has been around for ages.

Аre you saying that despite that it's around latest AOSP does not pick it up?
Can you share a bit more light on the topic?

Thanks
Emil
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2017-12-11 19:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-26 10:08 [PATCH libdrm] android: make libdrm*.so available to the vendor partition Jiyong Park
2017-12-11  3:51 ` Chih-Wei Huang
2017-12-11 19:28   ` Emil Velikov [this message]
2017-12-12  9:12     ` Jiyong Park
2017-12-12 10:43       ` Jiyong Park

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=CACvgo51t23GmFV-QCWKDPMU9A5d8Bic8vPGErMGf9Kt9KHpoGQ@mail.gmail.com \
    --to=emil.l.velikov@gmail.com \
    --cc=cwhuang@android-x86.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jiyong@google.com \
    --cc=marcheu@google.com \
    --cc=robherring2@gmail.com \
    --cc=seanpaul@google.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.