All of lore.kernel.org
 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 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.