Commit 0a4eecf9 authored by Fabrizio Castro's avatar Fabrizio Castro Committed by Wolfram Sang
Browse files

dt-bindings: i2c: renesas,rzv2m: Fix SoC specific string 



The preferred form for Renesas' compatible strings is:
"<vendor>,<family>-<module>"

Somehow the compatible string for the r9a09g011 I2C IP was upstreamed
as renesas,i2c-r9a09g011 instead of renesas,r9a09g011-i2c, which
is really confusing, especially considering the generic fallback
is renesas,rzv2m-i2c.

The first user of renesas,i2c-r9a09g011 in the kernel is not yet in
a kernel release, it will be in v6.1, therefore it can still be
fixed in v6.1.
Even if we don't fix it before v6.2, I don't think there is any
harm in making such a change.

s/renesas,i2c-r9a09g011/renesas,r9a09g011-i2c/g for consistency.

Fixes: ba7a4d15 ("dt-bindings: i2c: Document RZ/V2M I2C controller")
Signed-off-by: default avatarFabrizio Castro <fabrizio.castro.jz@renesas.com>
Reviewed-by: default avatarGeert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: default avatarWolfram Sang <wsa@kernel.org>
parent 78a4471f

Documentation/devicetree/bindings/i2c/renesas,rzv2m.yaml

+2 −2
Original line number Diff line number Diff line
@@ -16,7 +16,7 @@ properties: 
  compatible:

    items:

      - enum:

          - renesas,i2c-r9a09g011  # RZ/V2M

          - renesas,r9a09g011-i2c  # RZ/V2M

      - const: renesas,rzv2m-i2c



  reg:
@@ -66,7 +66,7 @@ examples: 
    #include <dt-bindings/interrupt-controller/arm-gic.h>



    i2c0: i2c@a4030000 {

        compatible = "renesas,i2c-r9a09g011", "renesas,rzv2m-i2c";

        compatible = "renesas,r9a09g011-i2c", "renesas,rzv2m-i2c";

        reg = <0xa4030000 0x80>;

        interrupts = <GIC_SPI 232 IRQ_TYPE_EDGE_RISING>,

                     <GIC_SPI 236 IRQ_TYPE_EDGE_RISING>;