From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Wed, 20 Aug 2014 12:54:23 -0600
Subject: [U-Boot] [PATCH 23/23] ARM: tegra: Enable PCIe on Jetson TK1
In-Reply-To: <1408346196-30419-24-git-send-email-thierry.reding@gmail.com>
References: <1408346196-30419-1-git-send-email-thierry.reding@gmail.com>
	<1408346196-30419-24-git-send-email-thierry.reding@gmail.com>
Message-ID: <53F4EEDF.8030908@wwwdotorg.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 08/18/2014 01:16 AM, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> The Jetson TK1 has an ethernet NIC connected to the PCIe bus and routes
> the second root port to a miniPCIe slot. Enable the PCIe controller and
> the network driver to allow the device to boot over the network.

> diff --git a/arch/arm/dts/tegra124-jetson-tk1.dts b/arch/arm/dts/tegra124-jetson-tk1.dts

>   	i2c at 7000d000 {

> +		pmic: pmic at 40 {

> +			regulators {

> +				sd1 {

> +					ams,ext-control = <1>;

According to an internal bug, we need to remove that property in order 
to get correct sequencing of the power rails at shutdown time. We need 
to fix that for the kernel too. However, we need to do more than just 
remove that property, or suspend won't actually turn off the rails:-/ So 
I suppose we should wait until the kernel patches land before resolving 
that; your patch does match what the kernel does currently.