From mboxrd@z Thu Jan  1 00:00:00 1970
From: Dan Murphy <dmurphy@ti.com>
Date: Fri, 12 Jul 2013 12:30:36 -0500
Subject: [U-Boot] [[PATCH v3 2/2] gpio: omap5-uevm: Configure the
 tca6424 gpio expander
In-Reply-To: <CAGo_u6rTY_f5bU4RDMwrKvZQr04JOGYk_vbCLr-Eaj=w2OQa6A@mail.gmail.com>
References: <1373486763-13091-1-git-send-email-dmurphy@ti.com>
	<1373486763-13091-2-git-send-email-dmurphy@ti.com>
	<51DDF49F.2050805@ti.com> <51DED56C.5080602@ti.com>
	<CAGo_u6rTY_f5bU4RDMwrKvZQr04JOGYk_vbCLr-Eaj=w2OQa6A@mail.gmail.com>
Message-ID: <51E03D3C.8040104@ti.com>
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

NM

On 07/11/2013 01:02 PM, Nishanth Menon wrote:
<snip>
>>> diff --git a/board/ti/omap5_uevm/mux_data.h
> b/board/ti/omap5_uevm/mux_data.h
>>>> index a82795d..7e6415e 100644
>>>> --- a/board/ti/omap5_uevm/mux_data.h
>>>> +++ b/board/ti/omap5_uevm/mux_data.h
>>>> @@ -56,6 +56,8 @@ const struct pad_conf_entry
> core_padconf_array_essential[] = {
>>>>       {USBD0_HS_DP, (IEN | M0)},    /*  USBD0_HS_DP */
>>>>       {USBD0_HS_DM, (IEN | M0)},    /*  USBD0_HS_DM */
>>>>       {USBD0_SS_RX, (IEN | M0)},    /*  USBD0_SS_RX */
>>>> +    {I2C5_SCL, (IEN | M0)}, /* I2C5_SCL */
>>> nit pick -> SCL (or i2c clk) is out put from master - always. IEN
> enables full duplex, but anyways.. just a nitpick :)
>> OK I changed this and did a test and reads fail without the IEN set.
>> This is to sync the client clocks.
> Scl? I2c clock? Does not work without input- something is fishy no?
Nope.  There is clock synchronization and clock arbitration that happens and the I2C master needs to see the SCL as an input.

Kernel configures the I2C mux for 1 and 5 the same way.
<snip>

-- 
------------------
Dan Murphy