As stated in [1], ETEK ET7304 is functionally identical to Richtek RT1715,
so reflect it in the bindings via a fallback compatible.

As there are various TCPCI chips by different vendors reimplementing the
registers and behavior of the RT1711H/RT1715, fallback compatibles will
scale better.

[1]
Link: https://lore.kernel.org/all/20260220-et7304-v3-2-ede2d9634957@gmail.com/
Signed-off-by: Alexey Charkov <alchark@flipper.net>
---
 Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml b/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
index 1eb611f35998..62169daddb4c 100644
--- a/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
+++ b/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
@@ -18,10 +18,13 @@ description: |
 
 properties:
   compatible:
-    enum:
-      - etekmicro,et7304
-      - richtek,rt1711h
-      - richtek,rt1715
+    oneOf:
+      - const: richtek,rt1711h
+      - const: richtek,rt1715
+      - items:
+          - enum:
+              - etekmicro,et7304
+          - const: richtek,rt1715
     description:
       RT1711H support PD20, ET7304 and RT1715 support PD30 except Fast Role Swap.
 

-- 
2.52.0

On 18/03/2026 10:23, Alexey Charkov wrote:
> As stated in [1], ETEK ET7304 is functionally identical to Richtek RT1715,

Functionally sounds like its functions/features. You mean the register
layout is identical?

> so reflect it in the bindings via a fallback compatible.
> 
> As there are various TCPCI chips by different vendors reimplementing the
> registers and behavior of the RT1711H/RT1715, fallback compatibles will
> scale better.
> 
> [1]
> Link: https://lore.kernel.org/all/20260220-et7304-v3-2-ede2d9634957@gmail.com/

[1] should be after the URL, rather.

> Signed-off-by: Alexey Charkov <alchark@flipper.net>
> ---
>  Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml | 11 +++++++----
>  1 file changed, 7 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml b/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
> index 1eb611f35998..62169daddb4c 100644
> --- a/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
> +++ b/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
> @@ -18,10 +18,13 @@ description: |
>  
>  properties:
>    compatible:
> -    enum:
> -      - etekmicro,et7304
> -      - richtek,rt1711h
> -      - richtek,rt1715
> +    oneOf:
> +      - const: richtek,rt1711h
> +      - const: richtek,rt1715

That's still an enum. Don't change the format.

> +      - items:
> +          - enum:
> +              - etekmicro,et7304
> +          - const: richtek,rt1715
>      description:
>        RT1711H support PD20, ET7304 and RT1715 support PD30 except Fast Role Swap.

Best regards,
Krzysztof

On Wed, Mar 18, 2026 at 1:27 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 18/03/2026 10:23, Alexey Charkov wrote:
> > As stated in [1], ETEK ET7304 is functionally identical to Richtek RT1715,
>
> Functionally sounds like its functions/features. You mean the register
> layout is identical?

Both actually. The only difference is the VID reported in registers.
Will reflect in the wording.

> > so reflect it in the bindings via a fallback compatible.
> >
> > As there are various TCPCI chips by different vendors reimplementing the
> > registers and behavior of the RT1711H/RT1715, fallback compatibles will
> > scale better.
> >
> > [1]
> > Link: https://lore.kernel.org/all/20260220-et7304-v3-2-ede2d9634957@gmail.com/
>
> [1] should be after the URL, rather.

Noted, thanks, will adjust.

> > Signed-off-by: Alexey Charkov <alchark@flipper.net>
> > ---
> >  Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml | 11 +++++++----
> >  1 file changed, 7 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml b/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
> > index 1eb611f35998..62169daddb4c 100644
> > --- a/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
> > +++ b/Documentation/devicetree/bindings/usb/richtek,rt1711h.yaml
> > @@ -18,10 +18,13 @@ description: |
> >
> >  properties:
> >    compatible:
> > -    enum:
> > -      - etekmicro,et7304
> > -      - richtek,rt1711h
> > -      - richtek,rt1715
> > +    oneOf:
> > +      - const: richtek,rt1711h
> > +      - const: richtek,rt1715
>
> That's still an enum. Don't change the format.

Ack, will make it oneOf: enum: ..., items: ...

Thanks a lot,
Alexey