RE: [PATCH v2 1/2] dt-bindings: i2c: renesas,rzv2m: Fix SoC specific string

From: Fabrizio Castro
Date: Mon Jan 16 2023 - 11:28:25 EST


Dear All,

This patch has been reviewed by both Geert Uytterhoeven and Krzysztof
Kozlowski. The corresponding device tree patch has already been
taken, I was hoping this patch could be applied to v6.3?

Thanks,
Fab

>
> On 07/11/2022 17:50, Fabrizio Castro wrote:
> > 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: ba7a4d15e2c4 ("dt-bindings: i2c: Document RZ/V2M I2C controller")
> > Signed-off-by: Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx>
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> Best regards,
> Krzysztof