On 12/12/2023 12:48, Swapnil Jakhade wrote: > Torrent PHY can have two input reference clocks. Update bindings It already supports two. > to support dual reference clock multilink configurations. > > Signed-off-by: Swapnil Jakhade <sjakhade@xxxxxxxxxxx> > --- > .../devicetree/bindings/phy/phy-cadence-torrent.yaml | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml > index dfb31314face..98946f549895 100644 > --- a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml > +++ b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml > @@ -35,14 +35,14 @@ properties: > minItems: 1 > maxItems: 2 > description: > - PHY reference clock for 1 item. Must contain an entry in clock-names. > - Optional Parent to enable output reference clock. > + PHY input reference clocks - refclk & pll1_refclk (optional). > + Optional Parent to enable output reference clock (phy_en_refclk). So third clock? But you allow only two? Confusing. > > clock-names: > minItems: 1 > items: > - const: refclk > - - const: phy_en_refclk > + - enum: [ pll1_refclk, phy_en_refclk ] This does not match your commit msg. You already had two clocks there. Best regards, Krzysztof