From mboxrd@z Thu Jan 1 00:00:00 1970 From: ben.dooks@codethink.co.uk (Ben Dooks) Date: Mon, 28 May 2012 11:55:56 +0100 Subject: [PATCH 1/2] MXS: Set I2C timing registers for mxs-i2c In-Reply-To: <1338084656-11961-1-git-send-email-marex@denx.de> References: <1338084656-11961-1-git-send-email-marex@denx.de> Message-ID: <4FC359BC.8080606@codethink.co.uk> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 27/05/12 03:10, Marek Vasut wrote: > This patch configures the I2C bus timing registers according > to information passed via DT. Currently, 100kHz and 400kHz > modes are supported. > > Signed-off-by: Marek Vasut > Cc: Detlev Zundel > CC: Dong Aisheng > CC: Fabio Estevam > Cc: Linux ARM kernel > Cc: linux-i2c at vger.kernel.org > CC: Sascha Hauer > CC: Shawn Guo > Cc: Stefano Babic > CC: Uwe Kleine-K?nig > Cc: Wolfgang Denk > Cc: Wolfram Sang > --- > Documentation/devicetree/bindings/i2c/i2c-mxs.txt | 1 + > arch/arm/boot/dts/imx28.dtsi | 2 + > drivers/i2c/busses/i2c-mxs.c | 64 +++++++++++++++++++++ > 3 files changed, 67 insertions(+) > > diff --git a/Documentation/devicetree/bindings/i2c/i2c-mxs.txt b/Documentation/devicetree/bindings/i2c/i2c-mxs.txt > index 1bfc02d..790b5c6 100644 > --- a/Documentation/devicetree/bindings/i2c/i2c-mxs.txt > +++ b/Documentation/devicetree/bindings/i2c/i2c-mxs.txt > @@ -4,6 +4,7 @@ Required properties: > - compatible: Should be "fsl,-i2c" > - reg: Should contain registers location and length > - interrupts: Should contain ERROR and DMA interrupts > +- speed: Speed of the bus in kHz (400 or 100 are supported) > > Examples: > > diff --git a/arch/arm/boot/dts/imx28.dtsi b/arch/arm/boot/dts/imx28.dtsi > index 4634cb8..d927155 100644 > --- a/arch/arm/boot/dts/imx28.dtsi > +++ b/arch/arm/boot/dts/imx28.dtsi > @@ -381,6 +381,7 @@ > compatible = "fsl,imx28-i2c"; > reg =<0x80058000 2000>; > interrupts =<111 68>; > + fsl,speed =<400>; > status = "disabled"; > }; > > @@ -390,6 +391,7 @@ > compatible = "fsl,imx28-i2c"; > reg =<0x8005a000 2000>; > interrupts =<110 69>; > + fsl,speed =<400>; > status = "disabled"; > }; Is there not a standard speed setting in the i2c binding? -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius