Re: [PATCH] dt-bindings: i2c: convert i2c-mt65xx to json-schema

[Miles Chen]

Hi AngeloGioacchino,

> Convert I2C binding for MediaTek SoCs to Devicetree schema.
> 
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
> ---
>  .../devicetree/bindings/i2c/i2c-mt65xx.txt    |  53 --------
>  .../devicetree/bindings/i2c/i2c-mt65xx.yaml   | 118 ++++++++++++++++++
>  MAINTAINERS                                   |   2 +-
>  3 files changed, 119 insertions(+), 54 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/i2c/i2c-mt65xx.txt
>  create mode 100644 Documentation/devicetree/bindings/i2c/i2c-mt65xx.yaml
> 
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mt65xx.txt b/Documentation/devicetree/bindings/i2c/i2c-mt65xx.txt
> deleted file mode 100644
> index 026985b8f61a..000000000000
> --- a/Documentation/devicetree/bindings/i2c/i2c-mt65xx.txt
> +++ /dev/null
> @@ -1,53 +0,0 @@
> -* MediaTek's I2C controller
> -
> -The MediaTek's I2C controller is used to interface with I2C devices.
> -
> -Required properties:
> -  - compatible: value should be either of the following.
> -      "mediatek,mt2701-i2c", "mediatek,mt6577-i2c": for MediaTek MT2701
> -      "mediatek,mt2712-i2c": for MediaTek MT2712
> -      "mediatek,mt6577-i2c": for MediaTek MT6577
> -      "mediatek,mt6589-i2c": for MediaTek MT6589
> -      "mediatek,mt6797-i2c", "mediatek,mt6577-i2c": for MediaTek MT6797
> -      "mediatek,mt7622-i2c": for MediaTek MT7622
> -      "mediatek,mt7623-i2c", "mediatek,mt6577-i2c": for MediaTek MT7623
> -      "mediatek,mt7629-i2c", "mediatek,mt2712-i2c": for MediaTek MT7629
> -      "mediatek,mt8168-i2c": for MediaTek MT8168
> -      "mediatek,mt8173-i2c": for MediaTek MT8173
> -      "mediatek,mt8183-i2c": for MediaTek MT8183
> -      "mediatek,mt8186-i2c": for MediaTek MT8186
> -      "mediatek,mt8192-i2c": for MediaTek MT8192
> -      "mediatek,mt8195-i2c", "mediatek,mt8192-i2c": for MediaTek MT8195
> -      "mediatek,mt8516-i2c", "mediatek,mt2712-i2c": for MediaTek MT8516
> -  - reg: physical base address of the controller and dma base, length of memory
> -    mapped region.
> -  - interrupts: interrupt number to the cpu.
> -  - clock-div: the fixed value for frequency divider of clock source in i2c
> -    module. Each IC may be different.
> -  - clocks: clock name from clock manager
>-  - clock-names: Must include "main" and "dma", "arb" is for multi-master that
>-    one bus has more than two i2c controllers, if enable have-pmic need include
>-    "pmic" extra.

snip

>+properties:
>+  compatible:
>+    oneOf:
>+      - const: mediatek,mt2712-i2c
>+      - const: mediatek,mt6577-i2c
>+      - const: mediatek,mt6589-i2c
>+      - const: mediatek,mt7622-i2c
>+      - const: mediatek,mt8168-i2c
>+      - const: mediatek,mt8173-i2c
>+      - const: mediatek,mt8183-i2c
>+      - const: mediatek,mt8186-i2c
>+      - const: mediatek,mt8192-i2c
>+      - items:
>+          - enum:
>+              - mediatek,mt7629-i2c
>+              - mediatek,mt8516-i2c
>+          - const: mediatek,mt2712-i2c
>+      - items:
>+          - enum:
>+              - mediatek,mt2701-i2c
>+              - mediatek,mt6797-i2c
>+              - mediatek,mt7623-i2c
>+          - const: mediatek,mt6577-i2c
>+      - items:
>+          - enum:
>+              - mediatek,mt8195-i2c
>+          - const: mediatek,mt8192-i2c
>+
>+  reg:
>+    items:
>+      - description: Physical base address
>+      - description: DMA base address
>+
>+  interrupts:
>+    maxItems: 1
>+
>+  clocks:
>+    minItems: 1
>+    items:
>+      - description: Main clock for I2C bus
>+      - description: Clock for I2C via DMA
>+      - description: Bus arbitrator clock
>+      - description: Clock for I2C from PMIC
>+
>+  clock-names:
>+    minItems: 1
>+    items:
>+      - const: main
>+      - const: dma
>+      - const: arb
>+      - const: pmic

I read "clock-names: Must include "main" and "dma"" from i2c-mt65xx.txt.
Does it means that we should have at least 2 clock-names (minItems: 2)?

Thanks,
Miles

>+
>+  clock-div:
>+    $ref: /schemas/types.yaml#/definitions/uint32
>+    description: Frequency divider of clock source in I2C module
>+
>+  clock-frequency:
>+    default: 100000
>+    description:
>+      SCL frequency to use (in Hz). If omitted, 100kHz is used.
>+
>+  mediatek,have-pmic:
>+    description: Platform controls I2C from PMIC side
>+    type: boolean
>+
>+  mediatek,use-push-pull:
>+    description: Use push-pull mode I/O config
>+    type: boolean
>+
>+  vbus-supply:
>+    description: Phandle to the regulator providing power to SCL/SDA
>+
>+required:
>+  - compatible
>+  - reg
>+  - clocks
>+  - clock-names
>+  - clock-div
>+  - interrupts
>+
>+unevaluatedProperties: false
>+
>+examples:
>+  - |
>+    #include <dt-bindings/interrupt-controller/arm-gic.h>
>+    #include <dt-bindings/interrupt-controller/irq.h>
>+
>+    i2c0: i2c@1100d000 {
>+      compatible = "mediatek,mt6577-i2c";
>+      reg = <0x1100d000 0x70>, <0x11000300 0x80>;
>+      interrupts = <GIC_SPI 44 IRQ_TYPE_LEVEL_LOW>;
>+      clocks = <&i2c0_ck>, <&ap_dma_ck>;
>+      clock-names = "main", "dma";
>+      clock-div = <16>;
>+      clock-frequency = <400000>;
>+      mediatek,have-pmic;
>+
>+      #address-cells = <1>;
>+      #size-cells = <0>;
>+    };
>diff --git a/MAINTAINERS b/MAINTAINERS
>index ee0b5a70aff1..28a17d4c8d12 100644
>--- a/MAINTAINERS
>+++ b/MAINTAINERS
>@@ -12361,7 +12361,7 @@ MEDIATEK I2C CONTROLLER DRIVER
> M:	Qii Wang <qii.wang@xxxxxxxxxxxx>
> L:	linux-i2c@xxxxxxxxxxxxxxx
> S:	Maintained
>-F:	Documentation/devicetree/bindings/i2c/i2c-mt65xx.txt
>+F:	Documentation/devicetree/bindings/i2c/i2c-mt65xx.yaml
> F:	drivers/i2c/busses/i2c-mt65xx.c
> 
> MEDIATEK IOMMU DRIVER
>-- 
>2.35.1
>
>