2023-06-01 10:21:03

by Oleksij Rempel

Subject: [PATCH v1 5/7] dt-bindings: clock: imx6ul: Support optional enet*_ref_pad clocks

Extend the 'clocks' and 'clock-names' properties to support optional
'enet1_ref_pad' and 'enet2_ref_pad' clocks to resolve the following
dtbs_check warning:
imx6ul-prti6g.dtb: clock-controller@20c4000: clocks: [[17], [18], [19],
[20], [21]] is too long
From schema: Documentation/devicetree/bindings/clock/imx6ul-clock.yaml

imx6ul-prti6g.dtb: clock-controller@20c4000: clock-names: ['ckil',
'osc', 'ipp_di0', 'ipp_di1', 'enet1_ref_pad'] is too long
From schema: Documentation/devicetree/bindings/clock/imx6ul-clock.yaml

Signed-off-by: Oleksij Rempel <[email protected]>
---
Documentation/devicetree/bindings/clock/imx6ul-clock.yaml | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml b/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
index be54d4df5afa..d6a36fe575d3 100644
--- a/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
@@ -28,18 +28,24 @@ properties:
const: 1

clocks:
+ minItems: 4
items:
- description: 32k osc
- description: 24m osc
- description: ipp_di0 clock input
- description: ipp_di1 clock input
+ - description: Optional lenet1_ref_pad or enet2_ref_pad clocks
+ - description: Optional lenet1_ref_pad or enet2_ref_pad clocks

clock-names:
+ minItems: 4
items:
- const: ckil
- const: osc
- const: ipp_di0
- const: ipp_di1
+ - enum: [enet1_ref_pad, enet2_ref_pad]
+ - enum: [enet1_ref_pad, enet2_ref_pad]

required:
- compatible
--
2.39.2

2023-06-14 19:22:40

by Rob Herring (Arm)

Subject: Re: [PATCH v1 5/7] dt-bindings: clock: imx6ul: Support optional enet*_ref_pad clocks

On Thu, Jun 01, 2023 at 12:14:49PM +0200, Oleksij Rempel wrote:
> Extend the 'clocks' and 'clock-names' properties to support optional
> 'enet1_ref_pad' and 'enet2_ref_pad' clocks to resolve the following
> dtbs_check warning:
> imx6ul-prti6g.dtb: clock-controller@20c4000: clocks: [[17], [18], [19],
> [20], [21]] is too long
> From schema: Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
>
> imx6ul-prti6g.dtb: clock-controller@20c4000: clock-names: ['ckil',
> 'osc', 'ipp_di0', 'ipp_di1', 'enet1_ref_pad'] is too long
> From schema: Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
>
> Signed-off-by: Oleksij Rempel <[email protected]>
> ---
> Documentation/devicetree/bindings/clock/imx6ul-clock.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml b/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
> index be54d4df5afa..d6a36fe575d3 100644
> --- a/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
> +++ b/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
> @@ -28,18 +28,24 @@ properties:
> const: 1
>
> clocks:
> + minItems: 4
> items:
> - description: 32k osc
> - description: 24m osc
> - description: ipp_di0 clock input
> - description: ipp_di1 clock input
> + - description: Optional lenet1_ref_pad or enet2_ref_pad clocks
> + - description: Optional lenet1_ref_pad or enet2_ref_pad clocks
>
> clock-names:
> + minItems: 4
> items:
> - const: ckil
> - const: osc
> - const: ipp_di0
> - const: ipp_di1
> + - enum: [enet1_ref_pad, enet2_ref_pad]
> + - enum: [enet1_ref_pad, enet2_ref_pad]

pattern: '^enet[12]_ref_pad$'

Rob