All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Franklin S. Cooper Jr" <fcooper@ti.com>
Cc: meta-ti@yoctoproject.org
Subject: Re: [PATCH 3/3] machine config: Use Graphics SDK 05.00.00.01 by default on AM335x and AM43x
Date: Fri, 22 Nov 2013 14:22:45 -0500	[thread overview]
Message-ID: <20131122192245.GD690@edge> (raw)
In-Reply-To: <1385136047-15830-3-git-send-email-fcooper@ti.com>

On Fri, Nov 22, 2013 at 10:00:47AM -0600, Franklin S. Cooper Jr wrote:
> * The latest Graphics SDK is required for the 3.12 kernel being used by AM335x
>   and AM43x machines.
> * Set the preferred version to insure that the proper version of the graphics
>   sdk is being used.
> 
> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> ---
>  conf/machine/include/ti33x.inc |    3 +++
>  conf/machine/include/ti43x.inc |    3 +++
>  2 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/conf/machine/include/ti33x.inc b/conf/machine/include/ti33x.inc
> index af340d2..8cdf580 100644
> --- a/conf/machine/include/ti33x.inc
> +++ b/conf/machine/include/ti33x.inc
> @@ -23,6 +23,9 @@ PREFERRED_PROVIDER_virtual/kernel = "linux-ti-staging"
>  PREFERRED_PROVIDER_virtual/bootloader = "u-boot-ti-staging"
>  PREFERRED_PROVIDER_u-boot = "u-boot-ti-staging"
>  
> +PREFERRED_VERSION_libgles-omap3 = "5.00.00.01"
> +PREFERRED_VERSION_omap3-sgx-modules = "5.00.00.01"

I'm worried about setting specific preferred versions in the machine configs - 
it doesn't look very scalable...

I'm willing to suggest to drop all DEFAULT_PREFERENCE = "-1" from the 
corresponding recipes and let the build pick the latest "compatible" version, 
unless overwritten by a distro with own PREFERRED_VERSION.

The only drawback I see is that some people still use 4.05 Gfx SDK on omap3 
and beagleboard, as that version had the best x11 support (or the least 
broken).

So, we can fix that particular issue by adding DEFAULT_PREFERENCE_omap3 = "99" 
to that recipe to bump it up.

Or, if not removing DEFAULT_PREFERENCE = "-1" from all Gfx SDK recipes, then 
just add overrides for ti33x and ti43x to bump the preference in 5.00.00.01 
recipe only...

Thoughts?

-- 
Denys


>  KERNEL_IMAGETYPE = "zImage"
>  
>  UBOOT_ARCH = "arm"
> diff --git a/conf/machine/include/ti43x.inc b/conf/machine/include/ti43x.inc
> index d9bfd25..948fd99 100644
> --- a/conf/machine/include/ti43x.inc
> +++ b/conf/machine/include/ti43x.inc
> @@ -23,6 +23,9 @@ PREFERRED_PROVIDER_virtual/kernel = "linux-ti-staging"
>  PREFERRED_PROVIDER_virtual/bootloader = "u-boot-ti-staging"
>  PREFERRED_PROVIDER_u-boot = "u-boot-ti-staging"
>  
> +PREFERRED_VERSION_libgles-omap3 = "5.00.00.01"
> +PREFERRED_VERSION_omap3-sgx-modules = "5.00.00.01"
> +
>  KERNEL_IMAGETYPE = "zImage"
>  
>  UBOOT_ARCH = "arm"
> -- 
> 1.7.0.4
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


  reply	other threads:[~2013-11-22 19:22 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-22 16:00 [PATCH 1/3] ti43x.inc: Add SGX as MACHINE_FEATURES Franklin S. Cooper Jr
2013-11-22 16:00 ` [PATCH 2/3] linux-ti-staging_3.12: Pull in commit that includes SGX DT patches Franklin S. Cooper Jr
2013-11-22 15:02   ` Denys Dmytriyenko
2013-11-22 16:00 ` [PATCH 3/3] machine config: Use Graphics SDK 05.00.00.01 by default on AM335x and AM43x Franklin S. Cooper Jr
2013-11-22 19:22   ` Denys Dmytriyenko [this message]
2013-11-22 19:29     ` Cooper Jr., Franklin
2013-11-22 19:34       ` Cooper Jr., Franklin

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=20131122192245.GD690@edge \
    --to=denys@ti.com \
    --cc=fcooper@ti.com \
    --cc=meta-ti@yoctoproject.org \
    /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.