Re: [PATCH 16/16] dt-bindings: phy: cdns,dphy: add power-domains property

Pratyush Yadav
2021-04-06T18:24:14Z

On 02/04/21 01:35PM, Laurent Pinchart wrote:
> Hi Pratyush,
>
> Thank you for the patch.
>
> On Tue, Mar 30, 2021 at 11:03:48PM +0530, Pratyush Yadav wrote:
> > This property is needed on TI platforms to enable the PD of the DPHY
> > before it can be used.
> >
> > Signed-off-by: Pratyush Yadav <p.yadav@ti.com>
> > ---
> > Documentation/devicetree/bindings/phy/cdns,dphy.yaml | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml
> > index 0807ba68284d..ddcd4de0aef6 100644
> > --- a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml
> > +++ b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml
> > @@ -30,6 +30,9 @@ properties:
> > "#phy-cells":
> > const: 0
> >
> > + power-domains:
> > + maxItems: 1
> > +
>
> Would it be useful to add power-domains to the example ?

Ok. Will add it in v2.

>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Thanks.

>
> > required:
> > - compatible
> > - reg
>
> --
> Regards,
>
> Laurent Pinchart

--
Regards,
Pratyush Yadav
Texas Instruments Inc.