[PATCH v1 08/15] dt-bindings: phy: phy-cadence-torrent: Add clock bindings
Maxime Ripard
maxime at cerno.tech
Fri Dec 13 09:59:56 UTC 2019
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>
More information about the dri-devel
mailing list