Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Koen Kooi <koen@dominion.thruhere.net>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH 3/3] libgles-omap3: Remove SGX Fixup for TI816x.
Date: Sun, 17 Apr 2011 20:35:19 +0200	[thread overview]
Message-ID: <iofbt7$hur$1@dough.gmane.org> (raw)
In-Reply-To: <1302328581-30367-3-git-send-email-heroor@gmail.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09-04-11 07:56, Siddharth Heroor wrote:
> From: Siddharth Heroor <heroor@ti.com>
> 
> * Unlike OMAP3, the SGX core has not undergone revisions in TI816x.
>   The libraries for TI816x are also fixed and do not need fixup based
>   on revision of the SGX core.
> 
> Signed-off-by: Siddharth Heroor <heroor@ti.com>

So, I'm back from travelling now and I'll have to NAK this patch:

1) it assumes netra will never have an sgx core update and makes it
diverge from the other platforms.

2) it adds a useless 2 second boot delay

So the net result is a harder to maintain slower version of the scripts,
which is not acceptable.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFNqzLnMkyGM64RGpERAgHyAKCXhRIKrqjA1FMKG+nJ8U2d+rAfYgCfSxEf
cywcK4FZnaGd12H1+B9fk4Q=
=goBu
-----END PGP SIGNATURE-----




  parent reply	other threads:[~2011-04-17 18:37 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-09  5:56 [PATCH 1/3] omap3-sgx-modules: Update TI816x Module Location and Platform variables Siddharth Heroor
2011-04-09  5:56 ` [PATCH 2/3] libgles-omap3: Correct libraries for TI816x Siddharth Heroor
2011-04-09  5:56 ` [PATCH 3/3] libgles-omap3: Remove SGX Fixup " Siddharth Heroor
2011-04-10  0:33   ` Denys Dmytriyenko
2011-04-10 12:09     ` Siddharth Heroor
2011-04-17 18:35   ` Koen Kooi [this message]
2011-04-18  2:56     ` Siddharth Heroor

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='iofbt7$hur$1@dough.gmane.org' \
    --to=koen@dominion.thruhere.net \
    --cc=openembedded-devel@lists.openembedded.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox