Hi Stephen,
On Thu, Jan 19, 2012 at 1:08 PM, Stephen Warren <swar...@nvidia.com> wrote:
> On 01/12/2012 12:00 PM, Simon Glass wrote:
>> From: Yen Lin <ye...@nvidia.com>
>>
>> Add basic i2c driver for Tegra2 with 8- and 16-bit address support.
>> The driver requires CONFIG_OF_CONTROL to obtain its configuration
>> from the device tree.
>>
>> (Simon Glass: s...@chromium.org modified for upstream)
>>
>> Signed-off-by: Simon Glass <s...@chromium.org>
>
> Why isn't Yen's S-o-b line here, since he's listed as the patch author
> in git?
It will work if he replies to this thread or the new patch that I sent
out. I cannot add it for him.
>
>> diff --git a/drivers/i2c/tegra2_i2c.c b/drivers/i2c/tegra2_i2c.c
>
>> +static struct i2c_bus i2c_controllers[CONFIG_SYS_MAX_I2C_BUS];
>
> I still don't like using the wrong define here, but since I see you
> pushing the multi-bus stuff upstream, so I assume you'll come back and
> fix this when it makes a difference, so I won't object too much about this.
I have changed this over.
>
>> +void i2c_init(int speed, int slaveaddr)
>> +{
>> + debug("i2c_init(speed=%u, slaveaddr=0x%x)\n", speed, slaveaddr);
>> +}
>
> What was the resolution on making this do something correct instead of
> ignoring the request?
I am just going to change the speed, thus forgetting the speed that
the fdt specified. There is no correct answer here - only the least
wrong. I will flip it the other way and see what people think.
Regards,
Simon
>
> --
> nvpublic
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
