* [PATCH 1/2] dt-bindings: phy: ti,phy-j721e-wiz: fix bindings for torrent phy
@ 2020-09-16 12:47 Tomi Valkeinen
2020-09-16 12:47 ` Tomi Valkeinen
2020-09-16 12:47 ` [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names Tomi Valkeinen
0 siblings, 2 replies; 5+ messages in thread
From: Tomi Valkeinen @ 2020-09-16 12:47 UTC (permalink / raw)
To: Kishon Vijay Abraham I, Vinod Koul, Rob Herring, Swapnil Jakhade,
Yuti Amonkar
Cc: linux-kernel, devicetree, Sekhar Nori, Tomi Valkeinen
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>
---
.../devicetree/bindings/phy/ti,phy-j721e-wiz.yaml | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml b/Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml
index 5ffc95c62909..c33e9bc79521 100644
--- a/Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml
+++ b/Documentation/devicetree/bindings/phy/ti,phy-j721e-wiz.yaml
@@ -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:
@@ -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
@@ -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>;
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 1/2] dt-bindings: phy: ti,phy-j721e-wiz: fix bindings for torrent phy
2020-09-16 12:47 [PATCH 1/2] dt-bindings: phy: ti,phy-j721e-wiz: fix bindings for torrent phy Tomi Valkeinen
@ 2020-09-16 12:47 ` Tomi Valkeinen
2020-09-16 12:47 ` [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names Tomi Valkeinen
1 sibling, 0 replies; 5+ messages in thread
From: Tomi Valkeinen @ 2020-09-16 12:47 UTC (permalink / raw)
To: Kishon Vijay Abraham I, Vinod Koul, Rob Herring, Swapnil Jakhade,
Yuti Amonkar
Cc: linux-kernel, devicetree, Sekhar Nori, Tomi Valkeinen
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names
2020-09-16 12:47 [PATCH 1/2] dt-bindings: phy: ti,phy-j721e-wiz: fix bindings for torrent phy Tomi Valkeinen
2020-09-16 12:47 ` Tomi Valkeinen
@ 2020-09-16 12:47 ` Tomi Valkeinen
2020-09-16 12:47 ` Tomi Valkeinen
2020-09-18 5:22 ` Vinod Koul
1 sibling, 2 replies; 5+ messages in thread
From: Tomi Valkeinen @ 2020-09-16 12:47 UTC (permalink / raw)
To: Kishon Vijay Abraham I, Vinod Koul, Rob Herring, Swapnil Jakhade,
Yuti Amonkar
Cc: linux-kernel, devicetree, Sekhar Nori, Tomi Valkeinen
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names
2020-09-16 12:47 ` [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names Tomi Valkeinen
@ 2020-09-16 12:47 ` Tomi Valkeinen
2020-09-18 5:22 ` Vinod Koul
1 sibling, 0 replies; 5+ messages in thread
From: Tomi Valkeinen @ 2020-09-16 12:47 UTC (permalink / raw)
To: Kishon Vijay Abraham I, Vinod Koul, Rob Herring, Swapnil Jakhade,
Yuti Amonkar
Cc: linux-kernel, devicetree, Sekhar Nori, Tomi Valkeinen
Add reset-names as a required property.
There are no dts files using torrent phy yet, so it is safe to add a new
required property.
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
---
.../devicetree/bindings/phy/phy-cadence-torrent.yaml | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
index 4071438be2ba..12ce022e4764 100644
--- a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
+++ b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
@@ -54,6 +54,10 @@ properties:
Torrent PHY reset.
See Documentation/devicetree/bindings/reset/reset.txt
+ reset-names:
+ items:
+ - const: torrent_reset
+
patternProperties:
'^phy@[0-7]+$':
type: object
@@ -111,6 +115,7 @@ required:
- reg
- reg-names
- resets
+ - reset-names
additionalProperties: false
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names
2020-09-16 12:47 ` [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names Tomi Valkeinen
2020-09-16 12:47 ` Tomi Valkeinen
@ 2020-09-18 5:22 ` Vinod Koul
1 sibling, 0 replies; 5+ messages in thread
From: Vinod Koul @ 2020-09-18 5:22 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: Kishon Vijay Abraham I, Rob Herring, Swapnil Jakhade,
Yuti Amonkar, linux-kernel, devicetree, Sekhar Nori
On 16-09-20, 15:47, Tomi Valkeinen wrote:
> Add reset-names as a required property.
>
> There are no dts files using torrent phy yet, so it is safe to add a new
> required property.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> ---
> .../devicetree/bindings/phy/phy-cadence-torrent.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
> index 4071438be2ba..12ce022e4764 100644
> --- a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
> +++ b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
> @@ -54,6 +54,10 @@ properties:
> Torrent PHY reset.
> See Documentation/devicetree/bindings/reset/reset.txt
>
> + reset-names:
> + items:
> + - const: torrent_reset
> +
> patternProperties:
> '^phy@[0-7]+$':
> type: object
> @@ -111,6 +115,7 @@ required:
> - reg
> - reg-names
> - resets
> + - reset-names
Update the example as well please.
>
> additionalProperties: false
>
> --
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
--
~Vinod
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-09-18 5:22 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-16 12:47 [PATCH 1/2] dt-bindings: phy: ti,phy-j721e-wiz: fix bindings for torrent phy Tomi Valkeinen
2020-09-16 12:47 ` Tomi Valkeinen
2020-09-16 12:47 ` [PATCH 2/2] dt-bindings: phy: cdns,torrent-phy: add reset-names Tomi Valkeinen
2020-09-16 12:47 ` Tomi Valkeinen
2020-09-18 5:22 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).