From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Wed, 25 Feb 2015 16:15:45 -0700
Subject: [U-Boot] [PATCH 07/20] dm: tegra: cros_ec: Enable Chrome OS EC
	on Nyan-big
In-Reply-To: <1424212195-7501-8-git-send-email-sjg@chromium.org>
References: <1424212195-7501-1-git-send-email-sjg@chromium.org>
 <1424212195-7501-8-git-send-email-sjg@chromium.org>
Message-ID: <54EE57A1.4080700@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 02/17/2015 03:29 PM, Simon Glass wrote:
> Enable the EC and keyboard, using the SPI bus.
>
> The EC driver requires a particular format and a deactivation delay. Also
> U-Boot does not support interrupts.
>
> For now, adjust the device tree to comply. At some point we should tidy
> this up to support interrupts and make tegra and exynos use the same setup.

> diff --git a/arch/arm/dts/tegra124-nyan-big.dts b/arch/arm/dts/tegra124-nyan-big.dts

>   	spi at 7000d400 {
>   		status = "okay";
> +		spi-deactivate-delay = <200>;
> +		spi-max-frequency = <3000000>;
>
>   		cros_ec: cros-ec at 0 {
> -			compatible = "google,cros-ec-spi";
> -			spi-max-frequency = <3000000>;
> +			compatible = "google,cros-ec";

I recall some discussion that implied both the SPI bus/controller node 
and the SPI device node both needed an spi-max-frequency property. Can 
you double-check this change conforms with the DT binding, and whatever 
Linux has in its DT files?