[PATCH v1 08/15] dt-bindings: phy: phy-cadence-torrent: Add clock bindings

[Maxime Ripard]

On Tue, Dec 03, 2019 at 11:13:18AM +0100, Yuti Amonkar wrote:
> Add Torrent PHY reference clock bindings.
>
> Signed-off-by: Yuti Amonkar <yamonkar at cadence.com>
> ---
>  .../devicetree/bindings/phy/phy-cadence-torrent.yaml         | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
> index d0037bc..6e1d71e5 100644
> --- a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
> +++ b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml
> @@ -16,6 +16,14 @@ properties:
>    compatible:
>      const: cdns,torrent-phy
>
> +  clocks:
> +    maxItems: 1
> +    description:
> +      PHY reference clock. Must contain an entry in clock-names.
> +
> +  clock-names:
> +    const: "refclk"
> +
>    reg:
>      items:
>        - description: Offset of the DPTX PHY configuration registers.
> @@ -36,6 +44,8 @@ properties:
>
>  required:
>    - compatible
> +  - clocks
> +  - clock-names

This also breaks previous DT that wouldn't have the clocks properties.

What you can do to support what you seem to be doing (changing the
properties and making the clocks properties required for the new one)
is:
  - Add a new compatible to the binding (without removing the older one)
  - Making the older one deprecated
  - Introduce the clocks properties to the binding
  - Making the clocks properties required for the new binding only

Maxime
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20191213/80c5e061/attachment-0001.sig>