lkml.org 
[lkml]   [2022]   [Nov]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 2/6] dt-bindings: net: marvell,dfx-server: Convert to yaml
    Date
    Even though this description is not used anywhere upstream (no matching
    driver), while on this file I decided I would try a conversion to yaml
    in order to clarify the prestera family description.

    I cannot keep the nodename dfx-server@xxxx so I switched to dfx-bus@xxxx
    which matches simple-bus.yaml. Otherwise I took the example context from
    the only user of this compatible: armada-xp-98dx3236.dtsi, which is a
    rather old and not perfect DT.

    Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
    ---
    I am fine dropping this file entirely as well, if judged useless.
    ---
    .../bindings/net/marvell,dfx-server.yaml | 60 +++++++++++++++++++
    .../bindings/net/marvell,prestera.txt | 18 ------
    2 files changed, 60 insertions(+), 18 deletions(-)
    create mode 100644 Documentation/devicetree/bindings/net/marvell,dfx-server.yaml

    diff --git a/Documentation/devicetree/bindings/net/marvell,dfx-server.yaml b/Documentation/devicetree/bindings/net/marvell,dfx-server.yaml
    new file mode 100644
    index 000000000000..72151a78396f
    --- /dev/null
    +++ b/Documentation/devicetree/bindings/net/marvell,dfx-server.yaml
    @@ -0,0 +1,60 @@
    +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
    +%YAML 1.2
    +---
    +$id: http://devicetree.org/schemas/net/marvell,dfx-server.yaml#
    +$schema: http://devicetree.org/meta-schemas/core.yaml#
    +
    +title: Marvell Prestera DFX server
    +
    +maintainers:
    + - Miquel Raynal <miquel.raynal@bootlin.com>
    +
    +select:
    + properties:
    + compatible:
    + contains:
    + const: marvell,dfx-server
    + required:
    + - compatible
    +
    +properties:
    + compatible:
    + items:
    + - const: marvell,dfx-server
    + - const: simple-bus
    +
    + reg: true
    +
    + ranges: true
    +
    + '#address-cells':
    + const: 1
    +
    + '#size-cells':
    + const: 1
    +
    +required:
    + - compatible
    + - reg
    + - ranges
    +
    +# The DFX server may expose clocks described as subnodes
    +additionalProperties: true
    +
    +examples:
    + - |
    +
    + #define MBUS_ID(target,attributes) (((target) << 24) | ((attributes) << 16))
    + bus@0 {
    + reg = <0 0>;
    + #address-cells = <2>;
    + #size-cells = <1>;
    +
    + dfx-bus@ac000000 {
    + compatible = "marvell,dfx-server", "simple-bus";
    + #address-cells = <1>;
    + #size-cells = <1>;
    + ranges = <0 MBUS_ID(0x08, 0x00) 0 0x100000>;
    + reg = <MBUS_ID(0x08, 0x00) 0 0x100000>;
    + };
    + };
    diff --git a/Documentation/devicetree/bindings/net/marvell,prestera.txt b/Documentation/devicetree/bindings/net/marvell,prestera.txt
    index 83370ebf5b89..8868d774da67 100644
    --- a/Documentation/devicetree/bindings/net/marvell,prestera.txt
    +++ b/Documentation/devicetree/bindings/net/marvell,prestera.txt
    @@ -27,21 +27,3 @@ switch {
    dfx = <&dfx>;
    };
    };
    -
    -DFX Server bindings
    --------------------
    -
    -Required properties:
    -- compatible: must be "marvell,dfx-server", "simple-bus"
    -- ranges: describes the address mapping of a memory-mapped bus.
    -- reg: address and length of the register set for the device.
    -
    -Example:
    -
    -dfx-server {
    - compatible = "marvell,dfx-server", "simple-bus";
    - #address-cells = <1>;
    - #size-cells = <1>;
    - ranges = <0 MBUS_ID(0x08, 0x00) 0 0x100000>;
    - reg = <MBUS_ID(0x08, 0x00) 0 0x100000>;
    -};
    --
    2.34.1
    \
     
     \ /
      Last update: 2022-11-17 22:56    [W:5.366 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site