Skip to contentSkip to conversation

ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

PATCH New Stephen Warren 4 participants 14 messages Aug 22, 2014 – Oct 15, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

From: Stephen Warren <[email protected]>

This pinmux tables currently omit any configuration for PCIe clk_req, wake, and rst pins, which in turn causes intermittent failures in U-Boot's PCIe support. Import an updated version of the pinmux tables which rectifies this.

---
 board/nvidia/jetson-tk1/pinmux-config-jetson-tk1.h | 5 +++++
 1 file changed, 5 insertions(+)
board/nvidia/jetson-tk1/pinmux-config-jetson-tk1.h +5 -0
index d338818a64eb..de4eb355982c 100644
@@ -283,6 +283,11 @@ static const struct pmux_pingrp_config jetson_tk1_pingrps[] = {
 	PINCFG(PCC2,                   DEFAULT,      DOWN,   NORMAL,   INPUT,   DEFAULT, DEFAULT),
 	PINCFG(SDMMC4_CLK_PCC4,        SDMMC4,       NORMAL, NORMAL,   INPUT,   DEFAULT, DEFAULT),
 	PINCFG(CLK2_REQ_PCC5,          DEFAULT,      NORMAL, NORMAL,   OUTPUT,  DEFAULT, DEFAULT),
+	PINCFG(PEX_L0_RST_N_PDD1,      PE0,          NORMAL, NORMAL,   OUTPUT,  DEFAULT, DEFAULT),
+	PINCFG(PEX_L0_CLKREQ_N_PDD2,   PE0,          UP,     NORMAL,   INPUT,   DEFAULT, DEFAULT),
+	PINCFG(PEX_WAKE_N_PDD3,        PE,           UP,     NORMAL,   INPUT,   DEFAULT, DEFAULT),
+	PINCFG(PEX_L1_RST_N_PDD5,      PE1,          NORMAL, NORMAL,   OUTPUT,  DEFAULT, DEFAULT),
+	PINCFG(PEX_L1_CLKREQ_N_PDD6,   PE1,          UP,     NORMAL,   INPUT,   DEFAULT, DEFAULT),
 	PINCFG(CLK3_OUT_PEE0,          EXTPERIPH3,   NORMAL, NORMAL,   OUTPUT,  DEFAULT, DEFAULT),
 	PINCFG(CLK3_REQ_PEE1,          DEFAULT,      NORMAL, NORMAL,   OUTPUT,  DEFAULT, DEFAULT),
 	PINCFG(DAP_MCLK1_REQ_PEE2,     SATA,         NORMAL, NORMAL,   OUTPUT,  DEFAULT, DEFAULT),
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Quoted 8 lines · Stephen Warren

From: Stephen Warren <[email protected]>

This pinmux tables currently omit any configuration for PCIe clk_req, wake, and rst pins, which in turn causes intermittent failures in U-Boot's PCIe support. Import an updated version of the pinmux tables which rectifies this.

Could this please make it into the imminent v2014.10 release?

#2 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

I take a look today.

Quoted 12 lines

-----Original Message-----
From: Stephen Warren [mailto:swarren at wwwdotorg.org]
Sent: Wednesday, September 17, 2014 9:17 AM
To: Tom Warren
Cc: u-boot at lists.denx.de; Simon Glass; Stephen Warren; Thierry Reding; Tom Rini
Subject: Re: [U-Boot] [PATCH] ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Quoted 2 lines · Stephen Warren

From: Stephen Warren <[email protected]>

Quoted 8 lines
Quoted 6 lines

This pinmux tables currently omit any configuration for PCIe clk_req, wake, and rst pins, which in turn causes intermittent failures in U-Boot's PCIe support. Import an updated version of the pinmux tables which rectifies this.

Could this please make it into the imminent v2014.10 release?

--
nvpublic

#3 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

In reply to Stephen Warren
Quoted 11 lines · Stephen Warren
Quoted 8 lines · Stephen Warren

From: Stephen Warren <[email protected]>

This pinmux tables currently omit any configuration for PCIe clk_req, wake, and rst pins, which in turn causes intermittent failures in U-Boot's PCIe support. Import an updated version of the pinmux tables which rectifies this.

Could this please make it into the imminent v2014.10 release?

I see this in u-boot-tegra/master, but it doesn't seem to be in u-boot-arm/master or u-boot/master.

Did a pull request get sent for this commit? I'd really like to see this bugfix get applied to the main U-Boot repo ASAP...

#4 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

My gmail account has been in flux for a few weeks, so I can't say for sure whether I sent a PR or not.

I'll resend one today to Albert.

Tom

Quoted 19 lines

-----Original Message-----
From: Stephen Warren [mailto:swarren at wwwdotorg.org]
Sent: Tuesday, October 14, 2014 8:25 AM
To: Tom Warren; Tom Rini; albert.u.boot at aribaud.net
Cc: u-boot at lists.denx.de; Simon Glass; Stephen Warren; Thierry Reding
Subject: Re: [U-Boot] [PATCH] ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Quoted 10 lines · Stephen Warren
Quoted 8 lines · Stephen Warren

From: Stephen Warren <[email protected]>

This pinmux tables currently omit any configuration for PCIe clk_req, wake, and rst pins, which in turn causes intermittent failures in U-Boot's PCIe support. Import an updated version of the pinmux tables which rectifies this.

Quoted 7 lines
Quoted 1 line

Could this please make it into the imminent v2014.10 release?

I see this in u-boot-tegra/master, but it doesn't seem to be in u-boot- arm/master or u-boot/master.

Did a pull request get sent for this commit? I'd really like to see this bugfix get applied to the main U-Boot repo ASAP...

-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

#5 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi Tom,

Quoted 1 line · Tom Warren

My gmail account has been in flux for a few weeks, so I can't say for sure whether I sent a PR or not.

I've gone through the U-Boot archives, and the last tegra PR sent was on aug 20, applied aug 29.

Quoted 1 line

I'll resend one today to Albert.

I'll make sue to apply it as soon as I get it in my inbox.

Quoted 1 line

Tom

Amicalement,

Signature · 1 line
Albert.
#6 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

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

 arch/arm/cpu/armv7/tegra30/Kconfig                 |   4 +
 arch/arm/dts/Makefile                              |   1 +
 arch/arm/dts/tegra30-apalis.dts                    | 300 ++++++++++++++++++
 arch/arm/include/asm/mach-types.h                  |  13 +
 board/nvidia/jetson-tk1/pinmux-config-jetson-tk1.h |   5 +
 board/toradex/apalis_t30/Kconfig                   |  15 +
 board/toradex/apalis_t30/MAINTAINERS               |   7 +
 board/toradex/apalis_t30/Makefile                  |   6 +
 board/toradex/apalis_t30/apalis_t30.c              |  92 ++++++
 .../toradex/apalis_t30/pinmux-config-apalis_t30.h  | 347 +++++++++++++++++++++
 configs/apalis_t30_defconfig                       |   4 +
 include/configs/apalis_t30.h                       |  80 +++++
 12 files changed, 874 insertions(+)
 create mode 100644 arch/arm/dts/tegra30-apalis.dts
 create mode 100644 board/toradex/apalis_t30/Kconfig
 create mode 100644 board/toradex/apalis_t30/MAINTAINERS
 create mode 100644 board/toradex/apalis_t30/Makefile
 create mode 100644 board/toradex/apalis_t30/apalis_t30.c
 create mode 100644 board/toradex/apalis_t30/pinmux-config-apalis_t30.h
 create mode 100644 configs/apalis_t30_defconfig
 create mode 100644 include/configs/apalis_t30.h

--
nvpublic

Quoted 29 lines

-----Original Message-----
From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
Sent: Tuesday, October 14, 2014 3:28 PM
To: Tom Warren
Cc: Stephen Warren; Tom Rini; u-boot at lists.denx.de; Simon Glass; Stephen Warren; Thierry Reding
Subject: Re: [U-Boot] [PATCH] ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi Tom,

Quoted 1 line · Tom Warren

My gmail account has been in flux for a few weeks, so I can't say for sure

whether I sent a PR or not.

I've gone through the U-Boot archives, and the last tegra PR sent was on aug 20, applied aug 29.

Quoted 1 line

I'll resend one today to Albert.

I'll make sue to apply it as soon as I get it in my inbox.

Quoted 1 line

Tom

Amicalement,
--
Albert.

#7 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi Tom,

Quoted 14 lines · Tom Warren

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:

Quoted 8 lines

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.

Amicalement,

Signature · 1 line
Albert.
#8 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Quoted 41 lines · Albert ARIBAUD

Hi Tom,

Quoted 14 lines · Tom Warren

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:

Quoted 8 lines

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)

Quoted 8 lines

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?

#9 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi Stephen,

Quoted 53 lines · Stephen Warren
Quoted 41 lines · Albert ARIBAUD

Hi Tom,

Quoted 14 lines · Tom Warren

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:

Quoted 8 lines

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)

Quoted 8 lines

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:

Quoted 7 lines

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,

Signature · 1 line
Albert.
#10 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Albert,

I did a Makeall -s tegra before sending the PR (as I always do), and I saw no failures. But I haven't rebased against ARM master in a few weeks.

I think the Jetson TK1 change is still valid and should go in.

Thanks,

Tom

Quoted 81 lines

-----Original Message-----
From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
Sent: Wednesday, October 15, 2014 8:40 AM
To: Stephen Warren
Cc: Tom Warren; u-boot at lists.denx.de; Simon Glass; Tom Rini; Stephen Warren; Thierry Reding
Subject: Re: [U-Boot] [PATCH] ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi Stephen,

Quoted 11 lines · Stephen Warren
Quoted 10 lines · Albert ARIBAUD

Hi Tom,

Quoted 5 lines · Tom Warren

Albert,

Please pull u-boot-tegra/master into ARM/master. Thanks!

The following changes since commit

a7f99bf139b3aaa0d5494693fd0395084355e41a:

Quoted 9 lines
Quoted 9 lines
Quoted 9 lines
   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:

Quoted 37 lines
Quoted 29 lines
Quoted 11 lines

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

Quoted 5 lines

(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:

Quoted 7 lines

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?

Quoted 6 lines

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

Amicalement,
--
Albert.

-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

#11 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi Tom,

Quoted 5 lines · Tom Warren

Albert,

I did a Makeall -s tegra before sending the PR (as I always do), and I saw no failures. But I haven't rebased against ARM master in a few weeks.

I think the Jetson TK1 change is still valid and should go in.

Ok. Seeing as this is a single patch to pick and board-, not arm-, specific, maybe Tom (Rini, promoted to To:) can pick it up directly instead of going through an ARM PR?

Quoted 3 lines

Thanks,

Tom

Amicalement,

Signature · 1 line
Albert.
#12 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

In reply to Albert ARIBAUD
Quoted 61 lines · Albert ARIBAUD

Hi Stephen,

Quoted 56 lines · Stephen Warren
Quoted 41 lines · Albert ARIBAUD

Hi Tom,

Quoted 14 lines · Tom Warren

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:

Quoted 8 lines
    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)

Quoted 8 lines

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

Quoted 7 lines
Quoted 4 lines

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?

I consider the Jetson TK1 pinmux patch a bug-fix; it adds entries that always should have been there.

#13 lore raw

PATCH 1/1 ARM: tegra: add PCIe-related pins to the Jetson TK1 pinmux tables

Hi,

Quoted 77 lines · Stephen Warren
Quoted 76 lines · Albert ARIBAUD

Hi Stephen,

Quoted 68 lines · Stephen Warren
Quoted 48 lines · Albert ARIBAUD

Hi Tom,

Quoted 20 lines · Tom Warren

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)

Quoted 7 lines

----------------------------------------------------------------
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)

Quoted 8 lines

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?

Quoted 6 lines
Quoted 2 lines

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

I consider the Jetson TK1 pinmux patch a bug-fix; it adds entries that always should have been there.

BTW the release has already gone out. Also what is the status of the Norrin patch? Will that go into Tegra soon?

Regards,
Simon