From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Warren <swarren@wwwdotorg.org>
Date: Fri, 19 Jul 2013 13:14:13 -0600
Subject: [U-Boot] [Ac100] [PATCH 3/3] ARM: tegra: paz00: enable nvec
	keyboard support
In-Reply-To: <1374223663-8576-4-git-send-email-danindrey@mail.ru>
References: <1374223663-8576-1-git-send-email-danindrey@mail.ru>
	<1374223663-8576-4-git-send-email-danindrey@mail.ru>
Message-ID: <51E99005.1090008@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 07/19/2013 02:47 AM, Andrey Danin wrote:
> Signed-off-by: Andrey Danin <danindrey@mail.ru>

(Some patch descriptions would be useful)

> diff --git a/board/compal/dts/tegra20-paz00.dts b/board/compal/dts/tegra20-paz00.dts

> +	nvec {
> +		compatible = "nvidia,tegra20-nvec";
> +		reg = <0x7000c500 0x100>;
> +		clock-frequency = <80000>;
> +		request-gpios = <&gpio 170 0>; /* gpio PV2 */
> +		slave-addr = <138>;
> +	};

I would rather not propagate this DT binding. We need to fix the binding
to clearly separate the concepts of:

a) The I2C slave controller (which should be a standalone driver for the
Tegra I2C slave HW).

b) The protocol sent over the I2C slave channel (which would be specific
to NVEC, implement the GPIO hand-shaking, etc.).

c) The devices that communicate over the protocol (keyboard in this case).

I suspect we need separate DT nodes/sub-nodes for all of those, and a
method of hooking them all together.