| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Thu, 23 May 2019 08:10:22 -0500
From: Rob Herring <robh+dt@...nel.org>
To: Maxime Ripard <maxime.ripard@...tlin.com>
Cc: Mark Rutland <mark.rutland@....com>,
Frank Rowand <frowand.list@...il.com>,
"David S . Miller" <davem@...emloft.net>,
Chen-Yu Tsai <wens@...e.org>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Alexandre Torgue <alexandre.torgue@...com>,
netdev <netdev@...r.kernel.org>,
"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
<linux-arm-kernel@...ts.infradead.org>, devicetree@...r.kernel.org,
linux-stm32@...md-mailman.stormreply.com,
Maxime Chevallier <maxime.chevallier@...tlin.com>,
Antoine Ténart <antoine.tenart@...tlin.com>
Subject: Re: [PATCH 1/8] dt-bindings: net: Add YAML schemas for the generic
Ethernet options
On Thu, May 23, 2019 at 4:57 AM Maxime Ripard <maxime.ripard@...tlin.com> wrote:
>
> The Ethernet controllers have a good number of generic options that can be
> needed in a device tree. Add a YAML schemas for those.
>
> Signed-off-by: Maxime Ripard <maxime.ripard@...tlin.com>
> ---
> Documentation/devicetree/bindings/net/ethernet-controller.yaml | 197 +++++++-
> Documentation/devicetree/bindings/net/ethernet.txt | 68 +--
> Documentation/devicetree/bindings/net/fixed-link.txt | 55 +--
> 3 files changed, 199 insertions(+), 121 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/net/ethernet-controller.yaml
>
> diff --git a/Documentation/devicetree/bindings/net/ethernet-controller.yaml b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> new file mode 100644
> index 000000000000..1c6e9e755481
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/net/ethernet-controller.yaml
> @@ -0,0 +1,197 @@
> +# SPDX-License-Identifier: GPL-2.0
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/net/ethernet-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Ethernet Controller Generic Binding
> +
> +maintainers:
> + - David S. Miller <davem@...emloft.net>
> +
> +properties:
> + $nodename:
> + pattern: "^ethernet(@.*)?$"
> +
> + local-mac-address:
> + allOf:
> + - $ref: /schemas/types.yaml#definitions/uint8-array
> + - minItems: 6
> + maxItems: 6
> + description:
> + Specifies the MAC address that was assigned to the network device.
> +
> + mac-address:
> + allOf:
> + - $ref: /schemas/types.yaml#definitions/uint8-array
> + - minItems: 6
> + maxItems: 6
> + description:
> + Specifies the MAC address that was last used by the boot
> + program; should be used in cases where the MAC address assigned
> + to the device by the boot program is different from the
> + local-mac-address property.
> +
> + max-frame-size:
> + $ref: /schemas/types.yaml#definitions/uint32
> + description:
> + Maximum transfer unit (IEEE defined MTU), rather than the
> + maximum frame size (there\'s contradiction in the Devicetree
> + Specification).
> +
> + max-speed:
> + $ref: /schemas/types.yaml#definitions/uint32
> + description:
> + Specifies maximum speed in Mbit/s supported by the device.
> +
> + nvmem-cells:
> + maxItems: 1
> + description:
> + Reference to an nvmem node for the MAC address
> +
> + nvmem-cells-names:
> + const: mac-address
> +
> + phy-connection-type:
> + description:
> + Operation mode of the PHY interface
> + oneOf:
While yes, this lets us have descriptions, oneOf makes for poor error
messages. I'd just make the descriptions comments instead.
> + - const: internal
> + description:
> + there is not a standard bus between the MAC and the PHY,
> + something proprietary is being used to embed the PHY in the
> + MAC.
> + - const: mii
> + - const: gmii
> + - const: sgmii
> + - const: qsgmii
> + - const: tbi
> + - const: rev-mii
> + - const: rmii
> + - const: rgmii
> + description:
> + RX and TX delays are added by the MAC when required
> + - const: rgmii-id
> + description:
> + RGMII with internal RX and TX delays provided by the PHY,
> + the MAC should not add the RX or TX delays in this case
> + - const: rgmii-rxid
> + description:
> + RGMII with internal RX delay provided by the PHY, the MAC
> + should not add an RX delay in this case
> + - const: rgmii-txid
> + description:
> + RGMII with internal TX delay provided by the PHY, the MAC
> + should not add an TX delay in this case
> + - const: rtbi
> + - const: smii
> + - const: xgmii
> + - const: trgmii
> + - const: 1000base-x
> + - const: 2500base-x
> + - const: rxaui
> + - const: xaui
> + - const: 10gbase-kr
> + description:
> + 10GBASE-KR, XFI, SFI
> +
> + phy-mode:
> + $ref: "#/properties/phy-connection-type"
> + description:
> + Deprecated in favor of phy-connection-type
> +
> + phy-handle:
> + $ref: /schemas/types.yaml#definitions/phandle
> + description:
> + Specifies a reference to a node representing a PHY device.
> +
> + phy:
> + $ref: "#/properties/phy-handle"
> + description:
> + Deprecated in favor of phy-handle
> +
> + phy-device:
> + $ref: "#/properties/phy-handle"
> + description:
> + Deprecated in favor of phy-handle
> +
> + rx-fifo-depth:
> + $ref: /schemas/types.yaml#definitions/uint32
> + description:
> + The size of the controller\'s receive fifo in bytes. This is used
> + for components that can have configurable receive fifo sizes,
> + and is useful for determining certain configuration settings
> + such as flow control thresholds.
> +
> + tx-fifo-depth:
> + $ref: /schemas/types.yaml#definitions/uint32
> + description:
> + The size of the controller\'s transmit fifo in bytes. This
> + is used for components that can have configurable fifo sizes.
> +
> + managed:
> + allOf:
> + - $ref: /schemas/types.yaml#definitions/string
> + - default: auto
> + enum:
> + - auto
> + - in-band-status
> + description:
> + Specifies the PHY management type. If auto is set and fixed-link
> + is not specified, it uses MDIO for management.
> +
> + fixed-link:
> + allOf:
> + - if:
> + type: array
> + then:
> + minItems: 1
> + maxItems: 1
> + items:
> + type: array
> + minItems: 5
> + maxItems: 5
> + description:
> + An array of 5 cells, with the following accepted values
> + - At index 0, the emulated PHY ID, choose any but but
> + unique to the all specified fixed-links, from 0 to 31
> + - at index 1, duplex configuration with 0 for half duplex
> + or 1 for full duplex
> + - at index 2, link speed in Mbits/sec, accepted values are
> + 10, 100 and 1000
> + - at index 3, pause configuration with 0 for no pause, 1
> + for pause
> + - at index 4, asymmetric pause configuration with 0 for no
> + asymmetric pause, 1 for asymmetric pause
Looks like constraints to me:
items:
- minimum: 0
maximum: 31
- enum: [ 0, 1 ]
- enum: [ 10, 100, 1000 ]
...
> +
> +
> + - if:
Couldn't this be an 'else' and avoid the allOf?
> + type: object
> + then:
> + properties:
> + speed:
> + allOf:
> + - $ref: /schemas/types.yaml#definitions/uint32
> + - enum: [10, 100, 1000]
> + description:
> + Link speed.
> +
> + full-duplex:
> + $ref: /schemas/types.yaml#definitions/flag
> + description:
> + Indicates that full-duplex is used. When absent, half
> + duplex is assumed.
> +
> + asym-pause:
> + $ref: /schemas/types.yaml#definitions/flag
> + description:
> + Indicates that asym_pause should be enabled.
> +
> + link-gpios:
> + description:
> + GPIO to determine if the link is up
Only 1?
> +
> + required:
> + - speed
> +
> +...
Powered by blists - more mailing lists