| Message ID | 20230404190115.546973-1-krzysztof.kozlowski@linaro.org |
|---|---|
| State | Accepted |
| Commit | 60671c4feee550e5597565866a9d8e1001ffa770 |
| Headers | show |
| Series | [1/5] dt-bindings: phy: cadence-sierra: drop assigned-clocks | expand |
On Tue, 04 Apr 2023 21:01:11 +0200, Krzysztof Kozlowski wrote: > The meta schema from DT schema already defines assigned-clocks, so there > is no need for device schema to mention it at all. There are also no > benefits of having it here and a board could actually need more of clock > assignments than the schema allows. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 8 -------- > 1 file changed, 8 deletions(-) > Reviewed-by: Rob Herring <robh@kernel.org>
On 04-04-23, 21:01, Krzysztof Kozlowski wrote: > The meta schema from DT schema already defines assigned-clocks, so there > is no need for device schema to mention it at all. There are also no > benefits of having it here and a board could actually need more of clock > assignments than the schema allows. Applied all, thanks > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 8 -------- > 1 file changed, 8 deletions(-) > > diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml > index 6a09472740ed..1de699fe5306 100644 > --- a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml > +++ b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml > @@ -61,14 +61,6 @@ properties: > - const: pll0_refclk > - const: pll1_refclk > > - assigned-clocks: > - minItems: 1 > - maxItems: 2 > - > - assigned-clock-parents: > - minItems: 1 > - maxItems: 2 > - > cdns,autoconf: > type: boolean > description: > -- > 2.34.1
diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml index 6a09472740ed..1de699fe5306 100644 --- a/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml +++ b/Documentation/devicetree/bindings/phy/phy-cadence-sierra.yaml @@ -61,14 +61,6 @@ properties: - const: pll0_refclk - const: pll1_refclk - assigned-clocks: - minItems: 1 - maxItems: 2 - - assigned-clock-parents: - minItems: 1 - maxItems: 2 - cdns,autoconf: type: boolean description:
The meta schema from DT schema already defines assigned-clocks, so there is no need for device schema to mention it at all. There are also no benefits of having it here and a board could actually need more of clock assignments than the schema allows. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../devicetree/bindings/phy/phy-cadence-sierra.yaml | 8 -------- 1 file changed, 8 deletions(-)