Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-12 Thread Geert Uytterhoeven
On Wed, Nov 12, 2014 at 9:53 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: i2c_shmobile support.) in v2.6.37. Without this fix i2c may not operate correctly on the sh73a0/kzm9g. Thanks, that's correct. git tag --contains b028f94b76319e1b8 says v2.6.38. Not that you're gonna

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-12 Thread Geert Uytterhoeven
Hi Simon, On Wed, Nov 12, 2014 at 2:44 AM, Simon Horman ho...@verge.net.au wrote: On Fri, Nov 07, 2014 at 06:24:21AM +0100, Wolfram Sang wrote: On Thu, Nov 06, 2014 at 12:52:06PM +0100, Geert Uytterhoeven wrote: On sh73a0/kzm9g-legacy, probing of the i2c masters fails with:

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-12 Thread Simon Horman
On Wed, Nov 12, 2014 at 09:53:27AM +0100, Geert Uytterhoeven wrote: Hi Simon, On Wed, Nov 12, 2014 at 2:44 AM, Simon Horman ho...@verge.net.au wrote: On Fri, Nov 07, 2014 at 06:24:21AM +0100, Wolfram Sang wrote: On Thu, Nov 06, 2014 at 12:52:06PM +0100, Geert Uytterhoeven wrote: On

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-12 Thread Simon Horman
On Wed, Nov 12, 2014 at 09:55:58AM +0100, Geert Uytterhoeven wrote: On Wed, Nov 12, 2014 at 9:53 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: i2c_shmobile support.) in v2.6.37. Without this fix i2c may not operate correctly on the sh73a0/kzm9g. Thanks, that's correct. git

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-11 Thread Simon Horman
On Fri, Nov 07, 2014 at 06:24:21AM +0100, Wolfram Sang wrote: On Thu, Nov 06, 2014 at 12:52:06PM +0100, Geert Uytterhoeven wrote: On sh73a0/kzm9g-legacy, probing of the i2c masters fails with: i2c-sh_mobile i2c-sh_mobile.0: timing values out of range: L/H=0x208/0x1bf

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-09 Thread Simon Horman
On Fri, Nov 07, 2014 at 06:24:21AM +0100, Wolfram Sang wrote: On Thu, Nov 06, 2014 at 12:52:06PM +0100, Geert Uytterhoeven wrote: On sh73a0/kzm9g-legacy, probing of the i2c masters fails with: i2c-sh_mobile i2c-sh_mobile.0: timing values out of range: L/H=0x208/0x1bf

[PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-06 Thread Geert Uytterhoeven
On sh73a0/kzm9g-legacy, probing of the i2c masters fails with: i2c-sh_mobile i2c-sh_mobile.0: timing values out of range: L/H=0x208/0x1bf sh_mobile: probe of i2c-sh_mobile.0 failed with error -22 According to the datasheet, the transfer rate is derived from the HP clock (which runs at

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-06 Thread Wolfram Sang
On Thu, Nov 06, 2014 at 12:52:06PM +0100, Geert Uytterhoeven wrote: On sh73a0/kzm9g-legacy, probing of the i2c masters fails with: i2c-sh_mobile i2c-sh_mobile.0: timing values out of range: L/H=0x208/0x1bf sh_mobile: probe of i2c-sh_mobile.0 failed with error -22 Yay, so the warning

Re: [PATCH 1/5] ARM: shmobile: kzm9g legacy: Set i2c clks_per_count to 2

2014-11-06 Thread Geert Uytterhoeven
Hi Wolfram, On Fri, Nov 7, 2014 at 6:24 AM, Wolfram Sang w...@the-dreams.de wrote: Note that the L/H values still differ from the Transfer Rate Settings example in Table 19.3 of the datasheet, which suggests 0x121/0xe7. Is the formula different or is it rounding errors? The formula is