Messages in this thread | | | Date | Mon, 23 Aug 2021 04:40:40 +0300 | From | Laurent Pinchart <> | Subject | Re: [PATCH v4 4/6] phy: dt-bindings: cdns,dphy: make clocks optional |
| |
Hi Pratyush,
Thank you for the patch.
On Sat, Aug 21, 2021 at 12:33:44AM +0530, Pratyush Yadav wrote: > The clocks are not used by the DPHY when used in Rx mode so make them > optional. > > Signed-off-by: Pratyush Yadav <p.yadav@ti.com> > Acked-by: Rob Herring <robh@kernel.org> > > --- > > (no changes since v3) > > Changes in v3: > - Add Rob's Ack. > > Changes in v2: > - Re-order subject prefixes. > > Documentation/devicetree/bindings/phy/cdns,dphy.yaml | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml > index b90a58773bf2..3bb5be05e825 100644 > --- a/Documentation/devicetree/bindings/phy/cdns,dphy.yaml > +++ b/Documentation/devicetree/bindings/phy/cdns,dphy.yaml > @@ -33,8 +33,6 @@ properties: > required: > - compatible > - reg > - - clocks > - - clock-names
Could you turn this into a conditional requirement based on the compat string, as the clocks are needed in the TX case ?
> - "#phy-cells" > > additionalProperties: false
-- Regards,
Laurent Pinchart
| |