Re: [PATCH v2] dt-bindings: mfd: maxim,max77693: Add USB connector

[Chanwoo Choi <chanwoo@xxxxxxxxxx>]

On 23. 7. 23. 23:28, Krzysztof Kozlowski wrote:
> Add micro-USB connector under "muic" node to properly represent the
> hardware.  Deprecate also the old "max77693-muic" in favor of generic
> "muic" (this is max77693, so there is no need to state it in its child
> nodes).  This "muic" node is used only to instantiate MUIC driver by
> compatible, thus actual Linux driver changes are needed.  Extend the
> example with this new code.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> 
> ---
> 
> Changes in v2:
> 1. Put connector node in the muic node (Henrik).
> 2. Rename max77693-muic -> muic.
> 3. Extend example.
> 
> DTS change using this:
> https://lore.kernel.org/linux-samsung-soc/20230723142417.97734-1-krzysztof.kozlowski@xxxxxxxxxx/T/#u
> ---
>  .../bindings/mfd/maxim,max77693.yaml          | 48 +++++++++++++++++++
>  1 file changed, 48 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77693.yaml b/Documentation/devicetree/bindings/mfd/maxim,max77693.yaml
> index 1b06a77ec798..6a6f222b868f 100644
> --- a/Documentation/devicetree/bindings/mfd/maxim,max77693.yaml
> +++ b/Documentation/devicetree/bindings/mfd/maxim,max77693.yaml
> @@ -37,6 +37,7 @@ properties:
>    max77693-muic:
>      type: object
>      additionalProperties: false
> +    deprecated: true
>  
>      properties:
>        compatible:
> @@ -45,6 +46,21 @@ properties:
>      required:
>        - compatible
>  
> +  muic:
> +    type: object
> +    additionalProperties: false
> +
> +    properties:
> +      compatible:
> +        const: maxim,max77693-muic
> +
> +      connector:
> +        $ref: /schemas/connector/usb-connector.yaml#
> +        unevaluatedProperties: false
> +
> +    required:
> +      - compatible
> +
>    motor-driver:
>      type: object
>      additionalProperties: false
> @@ -107,6 +123,38 @@ examples:
>                  };
>              };
>  
> +            muic {
> +                compatible = "maxim,max77693-muic";
> +
> +                connector {
> +                    compatible = "samsung,usb-connector-11pin",
> +                                 "usb-b-connector";
> +                    label = "micro-USB";
> +                    type = "micro";
> +
> +                    ports {
> +                        #address-cells = <1>;
> +                        #size-cells = <0>;
> +
> +                        port@0 {
> +                            reg = <0>;
> +
> +                            muic_to_usb: endpoint {
> +                                remote-endpoint = <&usb_to_muic>;
> +                            };
> +                        };
> +
> +                        port@3 {
> +                            reg = <3>;
> +
> +                            muic_to_mhl: endpoint {
> +                                remote-endpoint = <&mhl_to_muic>;
> +                            };
> +                        };
> +                    };
> +                };
> +            };
> +
>              motor-driver {
>                  compatible = "maxim,max77693-haptic";
>                  haptic-supply = <&ldo26_reg>;

Reviewed-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx>

Thanks.

-- 
Best Regards,
Samsung Electronics
Chanwoo Choi