public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables
Date: Wed, 15 Oct 2014 17:39:36 +0200	[thread overview]
Message-ID: <E1XeQfp-0002GG-Mc@janus> (raw)
In-Reply-To: <543E9419.4010200@wwwdotorg.org>

Hi Stephen,

On Wed, 15 Oct 2014 09:34:49 -0600, Stephen Warren
<swarren@wwwdotorg.org> wrote:

> On 10/15/2014 05:43 AM, Albert ARIBAUD wrote:
> > Hi Tom,
> >
> > On Tue, 14 Oct 2014 23:14:24 +0000, Tom Warren <TWarren@nvidia.com>
> > wrote:
> >
> >> Albert,
> >>
> >> Please pull u-boot-tegra/master into ARM/master. Thanks!
> >>
> >> The following changes since commit a7f99bf139b3aaa0d5494693fd0395084355e41a:
> >>
> >>    arm: Fix _start for CONFIG_SYS_DV_NOR_BOOT_CFG (2014-09-11 18:04:39 +0200)
> >>
> >> are available in the git repository at:
> >>
> >>    git://git.denx.de/u-boot-tegra.git master
> >>
> >> for you to fetch changes up to 90a565acf3a18c61170ec0e0b2046c98cb7ecc85:
> >>
> >>    arm: tegra: initial support for apalis t30 (2014-09-17 11:15:04 -0700)
> >>
> >> ----------------------------------------------------------------
> >> Marcel Ziswiler (1):
> >>        arm: tegra: initial support for apalis t30
> >>
> >> Stephen Warren (1):
> >>        ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables
> >
> > This new board fails when I merge this branch to u-boot-arm/master,
> > with the following diagnostics:
> >
> > include/configs/apalis_t30.h:17:0: warning: "CONFIG_OF_SEPARATE"
> > redefined [enabled by default]
> > In file included
> > from /home/albert.u.boot/src/u-boot-arm/include/linux/kconfig.h:4:0,
> >                   from <command-line>:0:
> > include/generated/autoconf.h:8:0: note: this is the location of the
> > previous definition
> >
> > (actually, there is a lot of these, for various config options)
> >
> > Device Tree Source is not correctly specified.
> > Please define 'CONFIG_DEFAULT_DEVICE_TREE'
> > or build with 'DEVICE_TREE=<device_tree>' argument
> >
> > (This one is fatal even though it does not explicitly state "error:")
> >
> > This failure does not happen on the tegra branch; it is obviously due
> > to the Kbuild stuff.
> 
> This issue should be fixed by:
> 
> https://patchwork.ozlabs.org/patch/398793/
> [U-Boot] tegra: apalis_t30: master revamp
> 
> However, I would assert that at this point during the release cycle, we 
> probably shouldn't be adding new boards/features, but just fixing bugs. 
> Perhaps we should drop the Apalis patch completely and re-apply it once 
> the release is out?

Ok. Should I still apply the TK1 pinmux patch, or should I consider the
whole PR to be for -next?

Amicalement,
-- 
Albert.

  reply	other threads:[~2014-10-15 15:39 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-22 21:04 [U-Boot] [PATCH] ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables Stephen Warren
2014-09-17 16:16 ` Stephen Warren
2014-09-17 16:19   ` Tom Warren
2014-10-14 15:24   ` Stephen Warren
2014-10-14 18:05     ` Tom Warren
2014-10-14 22:28       ` Albert ARIBAUD
2014-10-14 23:14         ` Tom Warren
2014-10-15 11:43           ` Albert ARIBAUD
2014-10-15 15:34             ` Stephen Warren
2014-10-15 15:39               ` Albert ARIBAUD [this message]
2014-10-15 15:59                 ` Tom Warren
2014-10-15 16:34                   ` Albert ARIBAUD
2014-10-15 16:38                 ` Stephen Warren
2014-10-15 16:41                   ` Simon Glass

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=E1XeQfp-0002GG-Mc@janus \
    --to=albert.u.boot@aribaud.net \
    --cc=u-boot@lists.denx.de \
    /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