Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread addy ke
On 2014/9/26 10:08, Doug Anderson wrote: > Addy, > > On Thu, Sep 25, 2014 at 6:40 PM, addy ke wrote: >> Hi, Doug >> >> On 2014/9/26 5:52, Doug Anderson wrote: >>> Addy, >>> >>> On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson >>> wrote: Addy, On Wed, Sep 24, 2014 at 6:56 PM,

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread Doug Anderson
Addy, On Thu, Sep 25, 2014 at 6:40 PM, addy ke wrote: > Hi, Doug > > On 2014/9/26 5:52, Doug Anderson wrote: >> Addy, >> >> On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson wrote: >>> Addy, >>> >>> On Wed, Sep 24, 2014 at 6:56 PM, addy ke wrote: In my measurement,all paramter but "Data hold

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread addy ke
Hi, Doug On 2014/9/26 5:52, Doug Anderson wrote: > Addy, > > On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson wrote: >> Addy, >> >> On Wed, Sep 24, 2014 at 6:56 PM, addy ke wrote: >>> In my measurement,all paramter but "Data hold time" are match the >>> characteristics of SCL bus line. >>> the

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread Doug Anderson
Addy, On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson wrote: > Addy, > > On Wed, Sep 24, 2014 at 6:56 PM, addy ke wrote: >> In my measurement,all paramter but "Data hold time" are match the >> characteristics of SCL bus line. >> the measured value is 0.928us("data hold time on RK3X" ~= "the

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread Doug Anderson
Addy, On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson diand...@chromium.org wrote: Addy, On Wed, Sep 24, 2014 at 6:56 PM, addy ke addy...@rock-chips.com wrote: In my measurement,all paramter but Data hold time are match the characteristics of SCL bus line. the measured value is 0.928us(data

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread addy ke
Hi, Doug On 2014/9/26 5:52, Doug Anderson wrote: Addy, On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson diand...@chromium.org wrote: Addy, On Wed, Sep 24, 2014 at 6:56 PM, addy ke addy...@rock-chips.com wrote: In my measurement,all paramter but Data hold time are match the characteristics

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread Doug Anderson
Addy, On Thu, Sep 25, 2014 at 6:40 PM, addy ke addy...@rock-chips.com wrote: Hi, Doug On 2014/9/26 5:52, Doug Anderson wrote: Addy, On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson diand...@chromium.org wrote: Addy, On Wed, Sep 24, 2014 at 6:56 PM, addy ke addy...@rock-chips.com wrote: In

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-25 Thread addy ke
On 2014/9/26 10:08, Doug Anderson wrote: Addy, On Thu, Sep 25, 2014 at 6:40 PM, addy ke addy...@rock-chips.com wrote: Hi, Doug On 2014/9/26 5:52, Doug Anderson wrote: Addy, On Wed, Sep 24, 2014 at 9:36 PM, Doug Anderson diand...@chromium.org wrote: Addy, On Wed, Sep 24, 2014 at

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread Doug Anderson
Addy, On Wed, Sep 24, 2014 at 6:56 PM, addy ke wrote: > In my measurement,all paramter but "Data hold time" are match the > characteristics of SCL bus line. > the measured value is 0.928us("data hold time on RK3X" ~= "the low period / > 2") > but the maximum value described in table is 0.9us

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread addy ke
Hi, Doug On 2014/9/25 1:13, Doug Anderson wrote: > Addy, > > On Wed, Sep 24, 2014 at 1:23 AM, addy ke wrote: >> >> >> On 2014/9/24 12:10, Doug Anderson wrote: >>> Addy, >>> >>> On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke wrote: As show in I2C specification: - Standard-mode: the

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread Doug Anderson
Addy, On Wed, Sep 24, 2014 at 1:23 AM, addy ke wrote: > > > On 2014/9/24 12:10, Doug Anderson wrote: >> Addy, >> >> On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke wrote: >>> As show in I2C specification: >>> - Standard-mode: >>> the minimum HIGH period of the scl clock is 4.0us >>> the minimum

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread addy ke
On 2014/9/24 12:10, Doug Anderson wrote: > Addy, > > On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke wrote: >> As show in I2C specification: >> - Standard-mode: >> the minimum HIGH period of the scl clock is 4.0us >> the minimum LOW period of the scl clock is 4.7us >> - Fast-mode: >> the

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread addy ke
On 2014/9/24 12:10, Doug Anderson wrote: Addy, On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke addy...@rock-chips.com wrote: As show in I2C specification: - Standard-mode: the minimum HIGH period of the scl clock is 4.0us the minimum LOW period of the scl clock is 4.7us - Fast-mode: the

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread Doug Anderson
Addy, On Wed, Sep 24, 2014 at 1:23 AM, addy ke addy...@rock-chips.com wrote: On 2014/9/24 12:10, Doug Anderson wrote: Addy, On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke addy...@rock-chips.com wrote: As show in I2C specification: - Standard-mode: the minimum HIGH period of the scl clock is

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread addy ke
Hi, Doug On 2014/9/25 1:13, Doug Anderson wrote: Addy, On Wed, Sep 24, 2014 at 1:23 AM, addy ke addy...@rock-chips.com wrote: On 2014/9/24 12:10, Doug Anderson wrote: Addy, On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke addy...@rock-chips.com wrote: As show in I2C specification: -

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-24 Thread Doug Anderson
Addy, On Wed, Sep 24, 2014 at 6:56 PM, addy ke addy...@rock-chips.com wrote: In my measurement,all paramter but Data hold time are match the characteristics of SCL bus line. the measured value is 0.928us(data hold time on RK3X ~= the low period / 2) but the maximum value described in

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-23 Thread Doug Anderson
Addy, On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke wrote: > As show in I2C specification: > - Standard-mode: > the minimum HIGH period of the scl clock is 4.0us > the minimum LOW period of the scl clock is 4.7us > - Fast-mode: > the minimum HIGH period of the scl clock is 0.6us > the minimum

[PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-23 Thread Addy Ke
As show in I2C specification: - Standard-mode: the minimum HIGH period of the scl clock is 4.0us the minimum LOW period of the scl clock is 4.7us - Fast-mode: the minimum HIGH period of the scl clock is 0.6us the minimum LOW period of the scl clock is 1.3us - Fast-mode plus: the minimum

[PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-23 Thread Addy Ke
As show in I2C specification: - Standard-mode: the minimum HIGH period of the scl clock is 4.0us the minimum LOW period of the scl clock is 4.7us - Fast-mode: the minimum HIGH period of the scl clock is 0.6us the minimum LOW period of the scl clock is 1.3us - Fast-mode plus: the minimum

Re: [PATCH] i2c: rk3x: adjust the LOW divison based on characteristics of SCL

2014-09-23 Thread Doug Anderson
Addy, On Tue, Sep 23, 2014 at 6:55 PM, Addy Ke addy...@rock-chips.com wrote: As show in I2C specification: - Standard-mode: the minimum HIGH period of the scl clock is 4.0us the minimum LOW period of the scl clock is 4.7us - Fast-mode: the minimum HIGH period of the scl clock is 0.6us