Re: [2011.03-maintenance] Pull request for TI matrix-gui, ipc and sgx/gles

[Tom Rini <tom_rini@mentor.com>]

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

On 04/17/2011 12:03 PM, Denys Dmytriyenko wrote:
> On Sun, Apr 17, 2011 at 08:54:53PM +0200, Koen Kooi wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> On 17-04-11 20:21, Denys Dmytriyenko wrote:
>>> On Sun, Apr 17, 2011 at 11:09:48AM -0700, Tom Rini wrote:
>>>> On 04/17/2011 12:00 AM, Koen Kooi wrote:
>>>>> On 17-04-11 04:28, Tom Rini wrote:
>>>>>> On 04/14/2011 12:34 PM, Denys Dmytriyenko wrote:
>>>>>>> Tom,
>>>>>
>>>>>>> Please pull these 5 commits for TI related changes - matrix-gui rev bump,
>>>>>>> new ti-ipc version and sgx/gles fixes for ti816x.
>>>>>>> All are cherry picked from oe.dev
>>>>>
>>>>>
>>>>>>> Pull URL: git://arago-project.org/git/people/denix/openembedded.git
>>>>>>>   Branch: 2011.03-maintenance
>>>>>>>   Browse: http://arago-project.org/git/people/?p=denix/openembedded.git;a=shortlog;h=refs/heads/2011.03-maintenance
>>>>>
>>>>>>> Thanks,
>>>>>>>     Denys Dmytriyenko <denys@ti.com>
>>>>>
>>>>>> Applied, thanks.
>>>>>
>>>>> Tom, can you please unmerge it? The SGX commit is bogus, I have already
>>>>> asked Denys to revert it from .dev. I do not want crap like that in
>>>>> maintenance.
>>>>
>>>> All of the parts of the merge or just a specific one?
>>>
>>> Tom,
>>>
>>> Can we hold off with unmerging/reverting for now? I'm trying to understand the 
>>> damage level and see the best course of action... But the commit in question 
>>> is "ligles-omap3: Remove SGX Fixup for TI816x"
>>
>> Right, only the "ligles-omap3: Remove SGX Fixup for TI816x" needs to get
>> reverted.
>>
>>>
>>> Koen,
>>>
>>> I replied to your email. Please let me know if you still prefer the complete 
>>> revert or if it can be worked around, as it still works, while may not be 
>>> optimal, and doesn't break anything else. Thanks.
>>
>> As I said in my response to the patch:
>>
>> - ------
>> 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.
>> - ------
>>
>> So I'd very much like to get this patch reverted both from .dev and
>> 2011.03-maintenance. Especially considering I'm the actual maintainer
>> for that recipe.
>>
>> For fun, try booting on omap3 and moving the SD to netra and see what
>> happens. This is what we do at the office and customers do as well.
> 
> Ok, I'll revert this specific patch from .dev
> 
> Tom,
> 
> Please go ahead and unmerge/revert it from the 2011.03-maintenance. Sorry for 
> the inconvenience.

Done.

- -- 
Tom Rini
Mentor Graphics Corporation
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJNq0hOAAoJEI4NMjfc3nJxyrgH/2f374sesrN0/9quWm6jE2M5
4VDq2xOFw5YtZzfiYq/KRMeCM1uHaAVVojHP88HMhzICmx37fPVWx5/AKQmwy+0S
I5859/ct4UcprpKtyqc2eUGiTJGWxcvR0L/FCST6hod0iMDC9Azz6/UFjJsXkpJB
I5IF098ej7nbCQxova9fGlfxU9Vi7xwyLWr8T0r5ygwUYaeXZLL898NpkzWWlGxZ
0utU5po0zQ0IDchRcm2QY4o9eQn8Y1Lh833CGkqTLmQGR5oLBpW/r1ZnqyE5ayTv
TL2fMR1yYVcjZ6lPSW+oESo6AAkUEtRSk+HkFTcU+vj+VU+hE/IJsRwJr9wOLS4=
=cDor
-----END PGP SIGNATURE-----