All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denis@denix.org>
To: Koen Kooi <koen@dominion.thruhere.net>
Cc: "meta-ti@yoctoproject.org" <meta-ti@yoctoproject.org>
Subject: Re: Denys Dmytriyenko : beaglebone: disable THUMB2_KERNEL in defconfig for newer binutils
Date: Thu, 03 May 2012 13:34:31 -0400	[thread overview]
Message-ID: <20120503173431.GK2681@denix.org> (raw)
In-Reply-To: <914CC77F-F9A0-4FCB-B112-46649335EAA3@dominion.thruhere.net>

On Thu, May 03, 2012 at 07:12:05PM +0200, Koen Kooi wrote:
> 
> 
> Op 3 mei 2012 om 18:21 heeft Denys Dmytriyenko <denis@denix.org> het volgende geschreven:
> 
> > On Thu, May 03, 2012 at 08:25:11AM +0200, Koen Kooi wrote:
> >> 
> >> Op 3 mei 2012, om 08:09 heeft Denys Dmytriyenko het volgende geschreven:
> >> 
> >>> On Thu, May 03, 2012 at 08:02:00AM +0200, Koen Kooi wrote:
> >>>> 
> >>>> Op 3 mei 2012, om 02:43 heeft Arago Project git het volgende geschreven:
> >>>> 
> >>>>> Module: meta-ti
> >>>>> Branch: master
> >>>>> Commit: 3c693c26087aa148b944166083b5336f05d1c15b
> >>>>> URL:    http://arago-project.org/git/meta-ti.git?a=commit;h=3c693c26087aa148b944166083b5336f05d1c15b
> >>>>> 
> >>>>> Author: Denys Dmytriyenko <denys@ti.com>
> >>>>> Date:   Wed May  2 20:40:58 2012 -0400
> >>>>> 
> >>>>> beaglebone: disable THUMB2_KERNEL in defconfig for newer binutils
> >>>> 
> >>>> Wait, what? Can we actually send these changes for review first? T2 is 
> >>>> needed to meet the boot time goal, so please revert this one!
> >>> 
> >>> Have another solution to get it building under recent binutils?
> >> 
> >> Patch binutils or patch the kernel.
> > 
> > Like I said, do you have a solution? Do you have said patches?
> > 
> > Since gcc-4.5 and binutils-2.20 are scheduled to be removed from 
> > meta-oe/toolchain-layer in a few days, I had to fix it some way. That's the 
> > best I could do under this circumstances. I can't have beaglebone broken with 
> > any  oher toolchain, to only gain few milliseconds of boot time.
> 
> It's in the range of half a second when I last tested it. 

If you care to provide binutils-2.20 with it forever, the change can be 
reverted.

Again, your decision to drop gcc-4.5 and binutils-2.20 from meta-oe drove me 
to push this fix.


> >> Changing the defconfig without even sending the patch for review is NOT the 
> >> way to do it.
> > 
> > I heard you the first time! No need to repeat it every time.
> 
> I'll repeat it some more if you think meta-ti should be TIs playground 
> without regard for the beagleboard.org project. Your action put a serious 
> dent in my belief in meta-ti.

I can say exactly the same to you - do you think meta-ti should be 
beagleboard.org's playground w/o regard for other TI goals to support 
different platforms and toolchains?

Anyway, it's a childish reaction and fight - you know very well it wasn't 
a conspiracy against you and affecting boot time was incidental. Calm down, 
react sane and eveything can be fixed and worked out.

-- 
Denys


  reply	other threads:[~2012-05-03 17:35 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-03  0:43 Denys Dmytriyenko : beaglebone: disable THUMB2_KERNEL in defconfig for newer binutils Arago Project git
2012-05-03  6:02 ` Koen Kooi
2012-05-03  6:09   ` Denys Dmytriyenko
2012-05-03  6:25     ` Koen Kooi
2012-05-03 16:21       ` Denys Dmytriyenko
2012-05-03 17:12         ` Koen Kooi
2012-05-03 17:34           ` Denys Dmytriyenko [this message]
2012-05-03 20:28             ` Denys Dmytriyenko

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=20120503173431.GK2681@denix.org \
    --to=denis@denix.org \
    --cc=koen@dominion.thruhere.net \
    --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.