[U-Boot] [PATCH 4/4 V4] I2C: mxc_i2c rework
Jason Hui
jason.hui at linaro.org
Thu Sep 22 06:54:22 CEST 2011
Hi, Marek,
On Thu, Sep 22, 2011 at 11:43 AM, Marek Vasut <marek.vasut at gmail.com> wrote:
> On Thursday, September 22, 2011 04:45:20 AM Jason Hui wrote:
>> Hi, Marek,
>>
>> On Tue, Sep 20, 2011 at 10:35 AM, Marek Vasut <marek.vasut at gmail.com> wrote:
>> > Rewrite the mxc_i2c driver.
>> > * This version is much closer to Linux implementation.
>> > * Fixes IPG_PERCLK being incorrectly used as clock source
>> > * Fixes behaviour of the driver on iMX51
>> > * Clean up coding style a bit ;-)
>> >
>> > Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
>> > Cc: Stefano Babic <sbabic at denx.de>
>> > Cc: Heiko Schocher <hs at denx.de>
>> > Cc: Jason Hui <jason.hui at linaro.org>
>> > ---
>> > drivers/i2c/mxc_i2c.c | 422
>> > +++++++++++++++++++++++++++++++++---------------- 1 files changed, 289
>> > insertions(+), 133 deletions(-)
>> >
>> > V2: Use PERCLK as a source.
>> >
>> > V3: Remove forgotten unused variables.
>> >
>> > V4: Add missing Cc field to commit message
>>
>> Test result on i.mx53evk, it shows that this patch does not work well,
>> here is the log:
>>
>> MX53EVK U-Boot > pmic dump 10
>> PMIC ID: 0x0000ffff [Rev: unknown]
>>
>> 0x00: 0001ffff 00ffffff 0039ffff 0000ffff 00ffffff 0000ffff 0000ffff
>> 0000ffff 0x08: 0000ffff 0000ffff 0000ffff 0000ffff 0000ffff 0000ffff
>> 0000ffff 0040ffff
>>
>
> It's strange, reading the registers with i2c md seems to work.
Here is i2c md output:
MX53EVK U-Boot > i2c md 8 0
0000: 01 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ................
seems not ok.
Jason Liu
>
More information about the U-Boot
mailing list