OSDN Git Service

i2c: rk3x: fix bug that cause measured high_ns doesn't meet I2C specification
authoraddy ke <addy.ke@rock-chips.com>
Thu, 11 Dec 2014 11:02:40 +0000 (19:02 +0800)
committerWolfram Sang <wsa@the-dreams.de>
Tue, 13 Jan 2015 15:21:04 +0000 (16:21 +0100)
commit1330e29105a3ad0a2a88d7a37ddd29d3f70675cf
tree1bd1859e23c0d2caf6cadf8a5052c644ff693770
parent9fae82e1acda8d4a6881be63cc38521b84007ba9
i2c: rk3x: fix bug that cause measured high_ns doesn't meet I2C specification

The number of clock cycles to be written into the CLKDIV register
that determines the I2C clk high phase includes the rise time.
So to meet the timing requirements defined in the I2C specification
which defines the minimal time SCL has to be high, the rise time
has to taken into account. The same applies to the low phase with
falling time.

In my test on RK3288-Pink2 board, which is not an upstream board yet,
if external pull-up resistor is 4.7K, rise_ns is about 700ns.
So the measured high_ns is about 3900ns, which is less than 4000ns
(the minimum high_ns in I2C specification for Standard-mode).

To fix this bug min_low_ns should include fall time and min_high_ns
should include rise time.

This patch merged the patch from chromium project which can get the
rise and fall times for signals from the device tree. This allows us
to more accurately calculate timings. see:
https://chromium-review.googlesource.com/#/c/232774/

Signed-off-by: Addy Ke <addy.ke@rock-chips.com>
Reviewed-by: Doug Anderson <dianders@chromium.org>
Tested-by: Doug Anderson <dianders@chromium.org>
[wsa: fixed a typo in the docs]
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Documentation/devicetree/bindings/i2c/i2c-rk3x.txt
drivers/i2c/busses/i2c-rk3x.c