Re: [PATCH v2 v2 1/3] dt-bindings: i2c: extend existing opencore bindings.

From: Rob Herring
Date: Mon May 13 2019 - 16:57:55 EST


On Tue, May 07, 2019 at 08:45:06PM +0530, Sagar Shrikant Kadam wrote:
> Add FU540-C000 specific device tree bindings to already
> available i2-ocores file. This device is available on
> HiFive Unleashed Rev A00 board.
>
> Signed-off-by: Sagar Shrikant Kadam <sagar.kadam@xxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/i2c/i2c-ocores.txt | 20 ++++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-ocores.txt b/Documentation/devicetree/bindings/i2c/i2c-ocores.txt
> index 17bef9a..f6bcf90 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-ocores.txt
> +++ b/Documentation/devicetree/bindings/i2c/i2c-ocores.txt
> @@ -2,6 +2,7 @@ Device tree configuration for i2c-ocores
>
> Required properties:
> - compatible : "opencores,i2c-ocores" or "aeroflexgaisler,i2cmst"
> + "sifive,fu540-c000-i2c" or "sifive,i2c0"

If this is Opencores IP, does it really follow the Sifive versioning
convention? If so, please reference sifive-blocks-ip-versioning.txt
(which appears to have missed going upstream). Also, referencing the IP
repository would be good too. If this IP block doesn't follow the same
convention, then don't try using it for this binding.

> - reg : bus address start and address range size of device
> - interrupts : interrupt number
> - clocks : handle to the controller clock; see the note below.
> @@ -67,3 +68,22 @@ or
> reg = <0x60>;
> };
> };
> +or

Just a new compatible isn't really a reason to add an example.

> + /*
> + An Opencore based I2C node in FU540-C000 chip from SiFive
> + This chip has a hardware erratum for broken IRQ
> + so it's recommended not to define interrupt in the device node

Then interrupts needs to be optional.

> + */
> + i2c@10030000 {
> + compatible = "sifive,i2c0","sifive,fu540-c000-i2c";
> + reg = <0x0 0x10030000 0x0 0x1000>;
> + reg-names = "i2c-control";

Not doucmented.

> + clocks = <&tlclk>;
> + clock-frequency = <100000>;
> +
> + reg-shift = <2>;
> + reg-io-width = <1>;
> +
> + #address-cells = <1>;
> + #size-cells = <0>;
> + };
> --
> 1.9.1
>
>
> --
> The information transmitted is intended only for the person or entity to
> which it is addressed and may contain confidential and/or privileged
> material. If you are not the intended recipient of this message please do
> not read, copy, use or disclose this communication and notify the sender
> immediately. It should be noted that any review, retransmission,
> dissemination or other use of, or taking action or reliance upon, this
> information by persons or entities other than the intended recipient is
> prohibited.