On Mon, Dec 10, 2018 at 01:13:53PM +0100, Geert Uytterhoeven wrote:
> Hi Biju,
>
> On Mon, Dec 10, 2018 at 1:06 PM Biju Das <biju.das@xxxxxxxxxxxxxx> wrote:
> > The iic3 block on RZ/G1M does not support automatic transmission, unlike
> > other R-Car SoC's. So dropping the compatibility with the generic version.
> >
> > Fixes: f523405f2a22cc0c307 ("ARM: dts: r8a7743: Add IIC cores to dtsi")
>
> Thanks for going through the other affected SoCs!
>
> > Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
Thanks, applied for v4.22.
> > --- a/arch/arm/boot/dts/r8a7743.dtsi
> > +++ b/arch/arm/boot/dts/r8a7743.dtsi
> > @@ -564,9 +564,7 @@
> > /* doesn't need pinmux */
> > #address-cells = <1>;
> > #size-cells = <0>;
> > - compatible = "renesas,iic-r8a7743",
> > - "renesas,rcar-gen2-iic",
> > - "renesas,rmobile-iic";
> > + compatible = "renesas,iic-r8a7743";
> > reg = <0 0xe60b0000 0 0x425>;
> > interrupts = <GIC_SPI 173 IRQ_TYPE_LEVEL_HIGH>;
> > clocks = <&cpg CPG_MOD 926>;
>
> Note that this is still a bit odd, as there are other iic nodes that do declare
> compatibility with the generic version, as they do support automatic
> transmission. But at least we'll have an easier job to support this in the
> driver[*], if ever needed.
>
> [*] The code handling r8a7743 (and r8a7744) can call of_device_is_compatible()
> to find out if the instance is compatible with the generic version.
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
>
