Hi, On Tue, Dec 03, 2019 at 11:13:12AM +0100, Yuti Amonkar wrote: > - Convert the MHDP PHY devicetree bindings to yaml schemas. > - Rename DP PHY to have generic Torrent PHY nomrnclature. > - Rename compatible string from "cdns,dp-phy" to "cdns,torrent-phy". > > Signed-off-by: Yuti Amonkar <yamonkar@xxxxxxxxxxx> > --- > .../devicetree/bindings/phy/phy-cadence-dp.txt | 30 ------------- > .../bindings/phy/phy-cadence-torrent.yaml | 52 ++++++++++++++++++++++ > 2 files changed, 52 insertions(+), 30 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/phy/phy-cadence-dp.txt > create mode 100644 Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml > > diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-dp.txt b/Documentation/devicetree/bindings/phy/phy-cadence-dp.txt > deleted file mode 100644 > index 7f49fd54e..0000000 > --- a/Documentation/devicetree/bindings/phy/phy-cadence-dp.txt > +++ /dev/null > @@ -1,30 +0,0 @@ > -Cadence MHDP DisplayPort SD0801 PHY binding > -=========================================== > - > -This binding describes the Cadence SD0801 PHY hardware included with > -the Cadence MHDP DisplayPort controller. > - > -------------------------------------------------------------------------------- > -Required properties (controller (parent) node): > -- compatible : Should be "cdns,dp-phy" > -- reg : Defines the following sets of registers in the parent > - mhdp device: > - - Offset of the DPTX PHY configuration registers > - - Offset of the SD0801 PHY configuration registers > -- #phy-cells : from the generic PHY bindings, must be 0. > - > -Optional properties: > -- num_lanes : Number of DisplayPort lanes to use (1, 2 or 4) > -- max_bit_rate : Maximum DisplayPort link bit rate to use, in Mbps (2160, > - 2430, 2700, 3240, 4320, 5400 or 8100) > -------------------------------------------------------------------------------- > - > -Example: > - dp_phy: phy@f0fb030a00 { > - compatible = "cdns,dp-phy"; > - reg = <0xf0 0xfb030a00 0x0 0x00000040>, > - <0xf0 0xfb500000 0x0 0x00100000>; > - num_lanes = <4>; > - max_bit_rate = <8100>; > - #phy-cells = <0>; > - }; > diff --git a/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml > new file mode 100644 > index 0000000..d0037bc > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml > @@ -0,0 +1,52 @@ > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/phy/phy-cadence-torrent.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Cadence Torrent SD0801 PHY binding for DisplayPort > + > +description: > + This binding describes the Cadence SD0801 PHY hardware included with > + the Cadence MHDP DisplayPort controller. > + > +maintainers: > + - Kishon Vijay Abraham I <kishon@xxxxxx> > + > +properties: > + compatible: > + const: cdns,torrent-phy > + > + reg: > + items: > + - description: Offset of the DPTX PHY configuration registers. > + - description: Offset of the SD0801 PHY configuration registers. > + > + "#phy-cells": > + const: 0 > + > + num_lanes: > + maxItems: 1 > + description: > + Number of DisplayPort lanes to use (1, 2 or 4) > + > + max_bit_rate: > + maxItems: 1 > + description: > + Maximum DisplayPort link bit rate to use, in Mbps (2160, 2430, 2700, 3240, 4320, 5400 or 8100) > + > +required: > + - compatible > + - reg > + - "#phy-cells" Setting additionalProperties: false here would be great, it prevents additional properties from slipping in (or typos in optional properties that would be unnoticed otherwise). Maxime
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel