[Buildroot] [RFC v2 1/1] ti-gfx: add new package

Spenser Gilliland spenser at gillilanding.com
Wed Jun 26 17:53:00 UTC 2013

On Tue, 25 Jun 2013 23:42:58 +0200

Dear Arnout,

> > This is WIP of adding the TI Graphics SDK to Buildroot
>   It's not really _the_ graphics SDK, since it only supports a subset
> of TI platforms (no OMAP1, OMAP2 or Davinci, and I'm not sure about
> OMAP5 either).
> >
> > Current Status:
> >     Kernel Drivers load without errror.
> >     pvrsrvctl starts with no errors.
> >     Some demos run, others do not.
> >     gstreamer videotestsrc ! eglglessink does not work.
> >
> > TODO:
> >     Install startup files for sgx core.
> >     Create .pc files to simplify additional packaging.
>   We don't usually create pc files if the package doesn't provide
> them. It's not really needed, because packages that use the library
> will anyway not use pkg-config to get the package info.

This was done for the rpi-userland component. It's a sensible thing to
do as many packages require egl or gles libraries.  It's easier to
just add an PKG_CHECK_MODULES to configure.ac then to add a bunch of
different ifdefs for each and every opengl implementation.  Thus, I
believe that this should be included.

> >     Hard Float Binaries
>   Doesn't the package contain soft-float binary libraries? Then 
> hard-float won't be possible, right?

TI provides a set of hard float binaries as well; however, they are in a
different download.

> >     Testing, lots of testing.
> >
> > Additional Info:
> >     I've been using the 3.9.6-x3 tag of the kernel at
> >     https://github.com/RobertCNelson/stable-kernel  by use of the
> >
> >     You must use a soft-float toolchain (ie Code Sourcery) as the
> > binaries provided by the Graphics SDK are soft-float. (hard float
> > is on the TODO list)
> >
> >     To load the driver manually use the following:
> >        modprobe omaplfb
> >        modprobe bufferclass-ti
> >        pvrsrvctl --start --no-module
> >
> > Signed-off-by: Spenser Gilliland<spenser at gillilanding.com>
>   Please keep a changelog of the history of your patch in the commit 
> message, so we can keep track of which review comments you've already 
> taken into account.

Will do.
>   In this particular case, I notice that Thomas repeats a lot of
> review comments that I've made a week ago, so writing "review
> comments from Arnout not taken into account" would have helped :-) Or
> maybe you missed those?

Sorry about that, they were good comments but I was just trying to get
it functional and most of the comments were cosmetic in nature.  I'll
get them all fixed up soon.


More information about the buildroot mailing list