diff mbox

[v3,1/2] dt-bindings: i2c: add binding for Socionext SynQuacer I2C

Message ID 20180227110107.30259-2-ard.biesheuvel@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Ard Biesheuvel Feb. 27, 2018, 11:01 a.m. UTC
Add a binding for the I2C controller that can be found in the Socionext
SynQuacer SoC. Note that this is the exact same IP as the Fujitsu F_I2C.

Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
 Documentation/devicetree/bindings/i2c/i2c-synquacer.txt | 29 ++++++++++++++++++++
 1 file changed, 29 insertions(+)

Comments

Rob Herring March 1, 2018, 2:28 p.m. UTC | #1
On Tue, Feb 27, 2018 at 5:01 AM, Ard Biesheuvel
<ard.biesheuvel@linaro.org> wrote:
> Add a binding for the I2C controller that can be found in the Socionext
> SynQuacer SoC. Note that this is the exact same IP as the Fujitsu F_I2C.

Is there a binding for that? Should be the same doc if so.

> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
>  Documentation/devicetree/bindings/i2c/i2c-synquacer.txt | 29 ++++++++++++++++++++
>  1 file changed, 29 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt b/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt
> new file mode 100644
> index 000000000000..4d0bb6a71211
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt
> @@ -0,0 +1,29 @@
> +Socionext SynQuacer I2C
> +
> +Required properties:
> +- compatible      : Must be "socionext,synquacer-i2c"
> +- reg             : Offset and length of the register set for the device
> +- interrupts      : the interrupt specifier

How many?

> +- #address-cells  : Must be <1>;
> +- #size-cells     : Must be <0>;
> +- clock-names     : Must contain "pclk".
> +- clocks          : Must contain an entry for each name in clock-names.
> +                    (See the common clock bindings.)
> +
> +Optional properties:
> +- clock-frequency : Desired I2C bus clock frequency in Hz. As only Normal and
> +                    Fast modes are supported, possible values are 100000 and
> +                    400000.
> +
> +Example :
> +
> +    i2c@51210000 {
> +        compatible = "socionext,synquacer-i2c";
> +        reg = <0x51210000 0x1000>;
> +        interrupts = <GIC_SPI 165 IRQ_TYPE_LEVEL_HIGH>;
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +        clock-names = "pclk";
> +        clocks = <&clk_i2c>;
> +        clock-frequency = <400000>;
> +    };
> --
> 2.11.0
>
Ard Biesheuvel March 1, 2018, 2:30 p.m. UTC | #2
On 1 March 2018 at 14:28, Rob Herring <robh+dt@kernel.org> wrote:
> On Tue, Feb 27, 2018 at 5:01 AM, Ard Biesheuvel
> <ard.biesheuvel@linaro.org> wrote:
>> Add a binding for the I2C controller that can be found in the Socionext
>> SynQuacer SoC. Note that this is the exact same IP as the Fujitsu F_I2C.
>
> Is there a binding for that? Should be the same doc if so.
>

No, there isn't. That support never made it upstream, but I thought
it'd be useful to mention it given the Fujitsu -> Socionext name
change.

>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> ---
>>  Documentation/devicetree/bindings/i2c/i2c-synquacer.txt | 29 ++++++++++++++++++++
>>  1 file changed, 29 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt b/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt
>> new file mode 100644
>> index 000000000000..4d0bb6a71211
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt
>> @@ -0,0 +1,29 @@
>> +Socionext SynQuacer I2C
>> +
>> +Required properties:
>> +- compatible      : Must be "socionext,synquacer-i2c"
>> +- reg             : Offset and length of the register set for the device
>> +- interrupts      : the interrupt specifier
>
> How many?
>

A single one, will change

>> +- #address-cells  : Must be <1>;
>> +- #size-cells     : Must be <0>;
>> +- clock-names     : Must contain "pclk".
>> +- clocks          : Must contain an entry for each name in clock-names.
>> +                    (See the common clock bindings.)
>> +
>> +Optional properties:
>> +- clock-frequency : Desired I2C bus clock frequency in Hz. As only Normal and
>> +                    Fast modes are supported, possible values are 100000 and
>> +                    400000.
>> +
>> +Example :
>> +
>> +    i2c@51210000 {
>> +        compatible = "socionext,synquacer-i2c";
>> +        reg = <0x51210000 0x1000>;
>> +        interrupts = <GIC_SPI 165 IRQ_TYPE_LEVEL_HIGH>;
>> +        #address-cells = <1>;
>> +        #size-cells = <0>;
>> +        clock-names = "pclk";
>> +        clocks = <&clk_i2c>;
>> +        clock-frequency = <400000>;
>> +    };
>> --
>> 2.11.0
>>
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt b/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt
new file mode 100644
index 000000000000..4d0bb6a71211
--- /dev/null
+++ b/Documentation/devicetree/bindings/i2c/i2c-synquacer.txt
@@ -0,0 +1,29 @@ 
+Socionext SynQuacer I2C
+
+Required properties:
+- compatible      : Must be "socionext,synquacer-i2c"
+- reg             : Offset and length of the register set for the device
+- interrupts      : the interrupt specifier
+- #address-cells  : Must be <1>;
+- #size-cells     : Must be <0>;
+- clock-names     : Must contain "pclk".
+- clocks          : Must contain an entry for each name in clock-names.
+                    (See the common clock bindings.)
+
+Optional properties:
+- clock-frequency : Desired I2C bus clock frequency in Hz. As only Normal and
+                    Fast modes are supported, possible values are 100000 and
+                    400000.
+
+Example :
+
+    i2c@51210000 {
+        compatible = "socionext,synquacer-i2c";
+        reg = <0x51210000 0x1000>;
+        interrupts = <GIC_SPI 165 IRQ_TYPE_LEVEL_HIGH>;
+        #address-cells = <1>;
+        #size-cells = <0>;
+        clock-names = "pclk";
+        clocks = <&clk_i2c>;
+        clock-frequency = <400000>;
+    };