| Message ID | 20200103094821.13185-2-codrin.ciubotariu@microchip.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | i2c bus recovery for Microchip SoCs | expand |
On Fri, 3 Jan 2020 09:49:06 +0000, <Codrin.Ciubotariu@microchip.com> wrote: > > From: Kamel Bouhara <kamel.bouhara@bootlin.com> > > The at91 I2C controller can support bus recovery by re-assigning SCL > and SDA to gpios. Add the optional pinctrl and gpio properties to do > so. > > Signed-off-by: Kamel Bouhara <kamel.bouhara@bootlin.com> > --- > > Changes in v2: > - none > > Documentation/devicetree/bindings/i2c/i2c-at91.txt | 10 ++++++++++ > 1 file changed, 10 insertions(+) > Please add Acked-by/Reviewed-by tags when posting new versions. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for acks received on the version they apply. If a tag was not added on purpose, please state why and what changed.
On 04.01.2020 00:29, Rob Herring wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On Fri, 3 Jan 2020 09:49:06 +0000, <Codrin.Ciubotariu@microchip.com> wrote: >> >> From: Kamel Bouhara <kamel.bouhara@bootlin.com> >> >> The at91 I2C controller can support bus recovery by re-assigning SCL >> and SDA to gpios. Add the optional pinctrl and gpio properties to do >> so. >> >> Signed-off-by: Kamel Bouhara <kamel.bouhara@bootlin.com> >> --- >> >> Changes in v2: >> - none >> >> Documentation/devicetree/bindings/i2c/i2c-at91.txt | 10 ++++++++++ >> 1 file changed, 10 insertions(+) >> > > Please add Acked-by/Reviewed-by tags when posting new versions. However, > there's no need to repost patches *only* to add the tags. The upstream > maintainer will do that for acks received on the version they apply. > > If a tag was not added on purpose, please state why and what changed. > Sorry Rob, I forgot to add your Acked-by. There is nothing changed in this version. How should I proceed, should I resend it with your tag? Best regards, Codrin
diff --git a/Documentation/devicetree/bindings/i2c/i2c-at91.txt b/Documentation/devicetree/bindings/i2c/i2c-at91.txt index 2210f4359c45..551ddabb566b 100644 --- a/Documentation/devicetree/bindings/i2c/i2c-at91.txt +++ b/Documentation/devicetree/bindings/i2c/i2c-at91.txt @@ -20,8 +20,13 @@ Optional properties: capable I2C controllers. - i2c-sda-hold-time-ns: TWD hold time, only available for "atmel,sama5d4-i2c" and "atmel,sama5d2-i2c". +- scl-gpios: specify the gpio related to SCL pin +- sda-gpios: specify the gpio related to SDA pin +- pinctrl: add extra pinctrl to configure i2c pins to gpio function for i2c + bus recovery, call it "gpio" state - Child nodes conforming to i2c bus binding + Examples : i2c0: i2c@fff84000 { @@ -56,6 +61,11 @@ i2c0: i2c@f8034600 { clocks = <&flx0>; atmel,fifo-size = <16>; i2c-sda-hold-time-ns = <336>; + pinctrl-names = "default", "gpio"; + pinctrl-0 = <&pinctrl_i2c0>; + pinctrl-1 = <&pinctrl_i2c0_gpio>; + sda-gpios = <&pioA 30 GPIO_ACTIVE_HIGH>; + scl-gpios = <&pioA 31 GPIO_ACTIVE_HIGH>; wm8731: wm8731@1a { compatible = "wm8731";