Hi Stephen,
On Thu, Jan 19, 2012 at 12:56 PM, Stephen Warren <swar...@nvidia.com> wrote:
> On 01/12/2012 12:00 PM, Simon Glass wrote:
>> Select the port ordering for I2C on Seaboard.
>>
>> Signed-off-by: Simon Glass <s...@chromium.org>
>> Acked-by: Stephen Warren <swar...@nvidia.com>
>
> This isn't the patch that I ack'd.
Sorry, I added the disable.
>
>> diff --git a/board/nvidia/dts/tegra2-seaboard.dts
>> b/board/nvidia/dts/tegra2-seaboard.dts
>
> Unrelated to this patch, but shouldn't that be tegra-seaboard.dts not
> tegra2-seaboard.dts to match the naming in the kernel?
>
>> @@ -44,4 +49,9 @@
>> usb@c5004000 {
>> status = "disabled";
>> };
>> +
>> + i2c@7000c400 {
>> + status = "disabled";
>> + };
>> +
>> };
>
> That chunk wasn't in the original patch, and doesn't match the kernel's
> .dts file (and I believe that I2C controller really is in use, so
> shouldn't be disabled).
It cannot be used - remember the discussion about pinmux? We elected
to disable I2C1 at present since you didn't like my nvidia,pinmux
binding for selecting which value to pass to funcmux. The fix is to
pass 1 instead of 0 for that port, but we have no clean way to specify
this.
Rather disable it than leave it enabled and not working.
Regards,
Simon
>
> --
> nvpublic
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
