mbox series

[net-next,0/8] dt-bindings: net: Child node schema cleanups

Message ID 20231016-dt-net-cleanups-v1-0-a525a090b444@kernel.org
Headers show
Series dt-bindings: net: Child node schema cleanups | expand

Message

Rob Herring (Arm) Oct. 16, 2023, 9:44 p.m. UTC 
This is a series of clean-ups related to ensuring that child node 
schemas are constrained to not allow undefined properties. Typically, 
that means just adding additionalProperties or unevaluatedProperties as 
appropriate. The DSA/switch schemas turned out to be a bit more 
involved, so there's some more fixes and a bit of restructuring in them.

Signed-off-by: Rob Herring <robh@kernel.org>
---
Rob Herring (8):
      dt-bindings: net: Add missing (unevaluated|additional)Properties on child node schemas
      dt-bindings: net: renesas: Drop ethernet-phy node schema
      dt-bindings: net: dsa/switch: Make 'ethernet-port' node addresses hex
      dt-bindings: net: ethernet-switch: Add missing 'ethernet-ports' level
      dt-bindings: net: ethernet-switch: Rename $defs "base" to 'ethernet-ports'
      dt-bindings: net: mscc,vsc7514-switch: Clean-up example indentation
      dt-bindings: net: mscc,vsc7514-switch: Simplify DSA and switch references
      dt-bindings: net: dsa: Drop 'ethernet-ports' node properties

 .../bindings/net/allwinner,sun8i-a83t-emac.yaml    |  2 +
 .../devicetree/bindings/net/brcm,asp-v2.0.yaml     |  2 +-
 .../devicetree/bindings/net/dsa/brcm,sf2.yaml      |  1 +
 Documentation/devicetree/bindings/net/dsa/dsa.yaml | 11 +-----
 .../bindings/net/dsa/mediatek,mt7530.yaml          |  8 ++--
 .../bindings/net/dsa/microchip,lan937x.yaml        |  3 +-
 .../devicetree/bindings/net/dsa/nxp,sja1105.yaml   |  4 +-
 .../devicetree/bindings/net/dsa/qca8k.yaml         |  1 +
 .../devicetree/bindings/net/dsa/realtek.yaml       |  2 +
 .../bindings/net/dsa/renesas,rzn1-a5psw.yaml       | 10 +----
 .../devicetree/bindings/net/engleder,tsnep.yaml    |  1 +
 .../devicetree/bindings/net/ethernet-switch.yaml   | 14 ++++---
 .../bindings/net/mscc,vsc7514-switch.yaml          | 46 +++++++++-------------
 .../devicetree/bindings/net/nxp,tja11xx.yaml       |  1 +
 .../devicetree/bindings/net/renesas,ether.yaml     |  3 +-
 .../devicetree/bindings/net/renesas,etheravb.yaml  |  3 +-
 .../devicetree/bindings/net/ti,cpsw-switch.yaml    |  2 +-
 17 files changed, 51 insertions(+), 63 deletions(-)
---
base-commit: 4d0515b235dec789578d135a5db586b25c5870cb
change-id: 20231016-dt-net-cleanups-ba3238b4ca98

Best regards,

Comments

Linus Walleij Oct. 18, 2023, 8:53 a.m. UTC | #1 
On Mon, Oct 16, 2023 at 11:44 PM Rob Herring <robh@kernel.org> wrote:

> This is a series of clean-ups related to ensuring that child node
> schemas are constrained to not allow undefined properties. Typically,
> that means just adding additionalProperties or unevaluatedProperties as
> appropriate. The DSA/switch schemas turned out to be a bit more
> involved, so there's some more fixes and a bit of restructuring in them.
>
> Signed-off-by: Rob Herring <robh@kernel.org>

Oh this drives a truck through my Marvell binding work. Luckily it
also solves my most annoying problems so I will just rebase on this
and continue, all good!
Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij
patchwork-bot+netdevbpf@kernel.org Oct. 19, 2023, 4:10 p.m. UTC | #2 
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon, 16 Oct 2023 16:44:19 -0500 you wrote:
> This is a series of clean-ups related to ensuring that child node
> schemas are constrained to not allow undefined properties. Typically,
> that means just adding additionalProperties or unevaluatedProperties as
> appropriate. The DSA/switch schemas turned out to be a bit more
> involved, so there's some more fixes and a bit of restructuring in them.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>
> 
> [...]

Here is the summary with links:
  - [net-next,1/8] dt-bindings: net: Add missing (unevaluated|additional)Properties on child node schemas
    https://git.kernel.org/netdev/net-next/c/659fd097b098
  - [net-next,2/8] dt-bindings: net: renesas: Drop ethernet-phy node schema
    https://git.kernel.org/netdev/net-next/c/ac8fe40c3628
  - [net-next,3/8] dt-bindings: net: dsa/switch: Make 'ethernet-port' node addresses hex
    https://git.kernel.org/netdev/net-next/c/51ff5150258a
  - [net-next,4/8] dt-bindings: net: ethernet-switch: Add missing 'ethernet-ports' level
    https://git.kernel.org/netdev/net-next/c/f0fdec925fe7
  - [net-next,5/8] dt-bindings: net: ethernet-switch: Rename $defs "base" to 'ethernet-ports'
    https://git.kernel.org/netdev/net-next/c/b9823df7bbad
  - [net-next,6/8] dt-bindings: net: mscc,vsc7514-switch: Clean-up example indentation
    https://git.kernel.org/netdev/net-next/c/491ec40d67a5
  - [net-next,7/8] dt-bindings: net: mscc,vsc7514-switch: Simplify DSA and switch references
    https://git.kernel.org/netdev/net-next/c/7c93392d754e
  - [net-next,8/8] dt-bindings: net: dsa: Drop 'ethernet-ports' node properties
    https://git.kernel.org/netdev/net-next/c/31f47f303c6b

You are awesome, thank you!