Skip to content

Commit

Permalink
dt-bindings: phy: ti,phy-j721e-wiz: fix bindings for torrent phy
Browse files Browse the repository at this point in the history
When WIZ wraps a Cadence Torrent PHY (instead of Cadence Sierra PHY)
there is a difference in the refclk-dig node: Torrent only has two
clocks instead of Sierra's four clocks. Add minItems: 2 to solve this.

Additionally, in our use case we only need to use assigned-clock for a
single clock, but the current binding requires either no assigned-clocks
or two. Fix this by adding minItems: 1 to all the assigned-clock
properties.

There was also an extra trailing whitespace, which this patch removes.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Link: https://lore.kernel.org/r/20200918083743.213874-2-tomi.valkeinen@ti.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
  • Loading branch information
Tomi Valkeinen authored and Vinod Koul committed Sep 23, 2020
1 parent 4feac94 commit b713228
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,15 @@ properties:
ranges: true

assigned-clocks:
minItems: 1
maxItems: 2

assigned-clock-parents:
minItems: 1
maxItems: 2

assigned-clock-rates:
minItems: 1
maxItems: 2

typec-dir-gpios:
Expand Down Expand Up @@ -119,9 +125,10 @@ patternProperties:
logic.
properties:
clocks:
minItems: 2
maxItems: 4
description: Phandle to four clock nodes representing the inputs to
refclk_dig
description: Phandle to two (Torrent) or four (Sierra) clock nodes representing
the inputs to refclk_dig

"#clock-cells":
const: 0
Expand Down Expand Up @@ -203,7 +210,7 @@ examples:
};
refclk-dig {
clocks = <&k3_clks 292 11>, <&k3_clks 292 0>,
clocks = <&k3_clks 292 11>, <&k3_clks 292 0>,
<&dummy_cmn_refclk>, <&dummy_cmn_refclk1>;
#clock-cells = <0>;
assigned-clocks = <&wiz0_refclk_dig>;
Expand Down

0 comments on commit b713228

Please sign in to comment.