From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 09/14 v6] package/nvidia-driver: add NVidia's OpenGL binary blob
Date: Fri, 6 Mar 2015 23:47:47 +0100 [thread overview]
Message-ID: <20150306224747.GC4220@free.fr> (raw)
In-Reply-To: <20150221183902.7a37db02@free-electrons.com>
Thomas, All,
On 2015-02-21 18:39 +0100, Thomas Petazzoni spake thusly:
> On Sat, 24 Jan 2015 00:24:38 +0100, Yann E. MORIN wrote:
> > This patch only adds the userland part. Unless other such other packages
> > (which we named like: rpi-userland), we do not replicate this naming
> > scheme with this package, as a future patch will also enable building
> > the kernel part of the driver. So, it is better to just name that
> > package with -driver, rather than with -userland and renaming it
> > afterwards.
> >
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> I've applied to next, after doing some changes:
[--SNIP--]
> However, one thing that I find a bit unclear and that might need
> improvement is that you are making this package depend on X.org while
> it does provide an EGL implementation. So you install the 36 MB
> libnvidia-eglcore.so.346.35 unconditionally, even though I believe it's
> probably unused when X.org is used.
>
> Same for libGLES: do we really need to install both libGL and libGLES ?
> Though I agree libGLES* are very small.
Well, I just did reproduce what the standard install procedure of the
package does: it installs everything. Agreed, I did not even try to
think those things apart.
> So I believe at least the EGL stuff should be separated out: either you
> do X.org, or you do EGL. But I don't think both can be used at the same
> time. Doing this would allow to drastically cut the installed size of
> this huge package.
Well, it is indeed quite huge, but also consider the type of systems it
is supposed to run on: desktop-class machines. The only embedded-class
board I know of that has an NVidia GPU is the newly introduced Jetson
TK1, and it has 16GiB of on-board eMMC...
> So now, the diff of the changes I made:
Thanks for this cleanup pass! :-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2015-03-06 22:47 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-23 23:24 [Buildroot] [PATCH 0/14 v6] NVidia GPU binary blobs for x86 and ARM (branch yem/gfx) Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 01/14 v6] package/python-lxml: new host package Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 02/14 v6] package/python-lxml: add target variant Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 03/14 v6] package/khronos-registry-khrplatform: new package Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 04/14 v6] package/khronos-registry-openkode: " Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 05/14 v6] package/mesa3d-headers: " Yann E. MORIN
2015-01-26 8:54 ` Jérôme Pouiller
2015-01-26 18:26 ` Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 06/14 v6] package/khronos-registry-opengl: " Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 07/14 v6] package/khronos-registry-opengl: download and install extra headers Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 08/14 v6] package/khronos-registry-openmaxil: new package Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 09/14 v6] package/nvidia-driver: add NVidia's OpenGL binary blob Yann E. MORIN
2015-02-21 17:39 ` Thomas Petazzoni
2015-03-06 22:47 ` Yann E. MORIN [this message]
2015-01-23 23:24 ` [Buildroot] [PATCH 10/14 v6] package/nvidia-driver: Add gl.pc Yann E. MORIN
2015-02-21 17:40 ` Thomas Petazzoni
2015-01-23 23:24 ` [Buildroot] [PATCH 11/14 v6] package/nvidia-driver: build the kernel module Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 12/14 v6] package/nvidia-tegra23-binaries: new package Yann E. MORIN
2015-01-26 13:31 ` Ben Ben
2015-01-26 18:19 ` Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 13/14 v6] package.nvidia-tegra23-binaries: add gstreamer plugins Yann E. MORIN
2015-01-23 23:24 ` [Buildroot] [PATCH 14/14 v6] package/nvidia-tegra23-codecs: new package Yann E. MORIN
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=20150306224747.GC4220@free.fr \
--to=yann.morin.1998@free.fr \
--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.