From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Wed, 20 Aug 2014 12:51:35 -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: <53F4EE37.3020003@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

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

> +	regulators {

> +		/* Molex power connector */
> +		vdd_5v0_sata: regulator at 13 {
...
> +		vdd_12v0_sata: regulator at 14 {

Those two aren't in the kernel DT yet. I'll assume they match what's 
been proposed for the SATA patches:-)

> diff --git a/board/nvidia/jetson-tk1/jetson-tk1.c b/board/nvidia/jetson-tk1/jetson-tk1.c

A lot of the AS3722 code here duplicates what's in 
board/nvidia/venice2/as3722_init.[ch]. It'd be good to put it all in one 
place. Personally I'd be OK with putting it into the existing Venice2 
code, but I suppose as Simon mentioned, a "real" AS3722 driver in 
drivers/power/pmic would be a good idea.