All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Stiffler, Jacob" <j-stiffler@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [EXTERNAL] [master][PATCH 2/4] toolchain: remove secondary toolchain hack
Date: Mon, 4 May 2020 17:16:48 -0400	[thread overview]
Message-ID: <20200504211648.GE31934@beryl> (raw)
In-Reply-To: <6fc6e6c601214fc083d666da2a7aff60@ti.com>

On Mon, May 04, 2020 at 09:06:32PM +0000, Stiffler, Jacob wrote:
> This was for adding the R5 toolchain into the devkit so that we can build 
> the R5 u-boot from the SDK.

Which would have to be done properly now, following multiconfig changes in 
meta-ti, not as a hack, which won't work and conflicts with those changes...

This is unfortunate, but is necessary for many existing and upcoming features.


> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> > Sent: Monday, May 04, 2020 4:55 PM
> > To: meta-arago@arago-project.org
> > Subject: [EXTERNAL] [meta-arago] [master][PATCH 2/4] toolchain: remove
> > secondary toolchain hack
> > 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > With K3 R5F support in meta-ti now, this hack is no longer needed.
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  meta-arago-distro/conf/distro/include/toolchain-arm.inc        |  6 ------
> >  .../recipes-core/meta/external-arm-secondary-sdk-toolchain.bb  |  4 ----
> >  .../packagegroups/packagegroup-cross-canadian.bbappend         | 10 ---------
> > -
> >  3 files changed, 20 deletions(-)
> >  delete mode 100644 meta-arago-distro/recipes-core/meta/external-arm-
> > secondary-sdk-toolchain.bb
> >  delete mode 100644 meta-arago-distro/recipes-
> > core/packagegroups/packagegroup-cross-canadian.bbappend
> > 
> > diff --git a/meta-arago-distro/conf/distro/include/toolchain-arm.inc
> > b/meta-arago-distro/conf/distro/include/toolchain-arm.inc
> > index d2c2b2f..fd63256 100644
> > --- a/meta-arago-distro/conf/distro/include/toolchain-arm.inc
> > +++ b/meta-arago-distro/conf/distro/include/toolchain-arm.inc
> > @@ -9,9 +9,6 @@ EAT_TARGET_SYS_arm = "${EAT_TARGET_SYS_ARMV5}"
> >  EAT_TARGET_SYS_armv7a = "${EAT_TARGET_SYS_ARMV7}"
> >  EAT_TARGET_SYS_aarch64 = "${EAT_TARGET_SYS_ARMV8}"
> > 
> > -SECONDARY_TARGET_ARCH = ""
> > -SECONDARY_TARGET_ARCH_k3 = "armv7a"
> > -
> >  TARGET_VENDOR = ""
> > 
> >  TOOLCHAIN_BASE ?= "/opt"
> > @@ -34,9 +31,6 @@ PREFERRED_PROVIDER_gcc-cross-canadian-
> > ${TRANSLATED_TARGET_ARCH} ?= "external-arm
> >  PREFERRED_PROVIDER_binutils-cross-canadian-
> > ${TRANSLATED_TARGET_ARCH} ?= "external-arm-sdk-toolchain"
> >  #PREFERRED_PROVIDER_gdb-cross-canadian-
> > ${TRANSLATED_TARGET_ARCH} = "external-arm-sdk-toolchain"
> > 
> > -PREFERRED_PROVIDER_gcc-cross-canadian-${SECONDARY_TARGET_ARCH}
> > ?= "external-arm-secondary-sdk-toolchain"
> > -PREFERRED_PROVIDER_binutils-cross-canadian-
> > ${SECONDARY_TARGET_ARCH} ?= "external-arm-secondary-sdk-toolchain"
> > -
> >  # Special case for gdb to be built as part of canadian-cross-sdk, instead of
> > packaged from external toolchain
> >  bindir_pn-gdb-cross-canadian-${TRANSLATED_TARGET_ARCH} =
> > "${exec_prefix}/bin"
> >  PREFERRED_PROVIDER_gdb-cross-canadian-${TRANSLATED_TARGET_ARCH}
> > = "gdb-cross-canadian-${TRANSLATED_TARGET_ARCH}"
> > diff --git a/meta-arago-distro/recipes-core/meta/external-arm-secondary-
> > sdk-toolchain.bb b/meta-arago-distro/recipes-core/meta/external-arm-
> > secondary-sdk-toolchain.bb
> > deleted file mode 100644
> > index 22a8aa0..0000000
> > --- a/meta-arago-distro/recipes-core/meta/external-arm-secondary-sdk-
> > toolchain.bb
> > +++ /dev/null
> > @@ -1,4 +0,0 @@
> > -TARGET_ARCH := "${SECONDARY_TARGET_ARCH}"
> > -MACHINEOVERRIDES := "${SECONDARY_TARGET_ARCH}"
> > -
> > -require recipes-core/meta/external-arm-sdk-toolchain.bb
> > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-
> > cross-canadian.bbappend b/meta-arago-distro/recipes-
> > core/packagegroups/packagegroup-cross-canadian.bbappend
> > deleted file mode 100644
> > index fb728e0..0000000
> > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-cross-
> > canadian.bbappend
> > +++ /dev/null
> > @@ -1,10 +0,0 @@
> > -PR_append = ".arago0"
> > -
> > -SECONDARY_TARGET_ARCH ?= ""
> > -
> > -SECONDARY_TOOLCHAIN = "gcc-cross-canadian-
> > ${SECONDARY_TARGET_ARCH} \
> > -                       binutils-cross-canadian-${SECONDARY_TARGET_ARCH}"
> > -
> > -RDEPENDS_${PN}_append = " \
> > -    ${@oe.utils.conditional('SECONDARY_TARGET_ARCH', '', '',
> > '${SECONDARY_TOOLCHAIN}', d)} \
> > -"
> > --
> > 2.7.4
> > 
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


  reply	other threads:[~2020-05-04 21:19 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-04 20:54 [master][PATCH 1/4] recipes-bsp: meta-ti now builds multiconfig u-boot/ti-sci-fw for R5 Denys Dmytriyenko
2020-05-04 20:54 ` [master][PATCH 2/4] toolchain: remove secondary toolchain hack Denys Dmytriyenko
2020-05-04 21:06   ` [EXTERNAL] " Stiffler, Jacob
2020-05-04 21:16     ` Denys Dmytriyenko [this message]
2020-05-05 11:12       ` Jacob Stiffler
2020-05-04 20:54 ` [master][PATCH 3/4] packagegroup: ti-sci-fw is only compatible with k3r5, not main k3 Denys Dmytriyenko
2020-05-04 20:54 ` [master][PATCH 4/4] arago.conf: only need to separate TMPDIR by TCMODE, not ARCH 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=20200504211648.GE31934@beryl \
    --to=denys@ti.com \
    --cc=j-stiffler@ti.com \
    --cc=meta-arago@arago-project.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.