* [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
@ 2022-02-28 21:38 Rob Herring
2022-02-28 21:45 ` Marek Behún
` (8 more replies)
0 siblings, 9 replies; 10+ messages in thread
From: Rob Herring @ 2022-02-28 21:38 UTC (permalink / raw)
To: devicetree
Cc: linux-kernel, Krzysztof Kozlowski, Thierry Reding, Sam Ravnborg,
Dmitry Torokhov, Pavel Machek, Lee Jones, Guenter Roeck,
Miquel Raynal, Richard Weinberger, Vignesh Raghavendra,
David S. Miller, Jakub Kicinski, Kishon Vijay Abraham I,
Vinod Koul, Sebastian Reichel, Bjorn Andersson, Mathieu Poirier,
Mark Brown, Greg Kroah-Hartman, Laurent Pinchart, dri-devel,
linux-arm-kernel, linux-input, linux-leds, linux-mtd, netdev,
linux-phy, linux-pm, linux-remoteproc, alsa-devel, linux-spi,
linux-usb
Another pass at removing unnecessary use of 'allOf' with a '$ref'.
json-schema versions draft7 and earlier have a weird behavior in that
any keywords combined with a '$ref' are ignored (silently). The correct
form was to put a '$ref' under an 'allOf'. This behavior is now changed
in the 2019-09 json-schema spec and '$ref' can be mixed with other
keywords.
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Guenter Roeck <groeck@chromium.org>
Cc: Miquel Raynal <miquel.raynal@bootlin.com>
Cc: Richard Weinberger <richard@nod.at>
Cc: Vignesh Raghavendra <vigneshr@ti.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Kishon Vijay Abraham I <kishon@ti.com>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: Sebastian Reichel <sre@kernel.org>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-input@vger.kernel.org
Cc: linux-leds@vger.kernel.org
Cc: linux-mtd@lists.infradead.org
Cc: netdev@vger.kernel.org
Cc: linux-phy@lists.infradead.org
Cc: linux-pm@vger.kernel.org
Cc: linux-remoteproc@vger.kernel.org
Cc: alsa-devel@alsa-project.org
Cc: linux-spi@vger.kernel.org
Cc: linux-usb@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
.../bindings/connector/usb-connector.yaml | 3 +--
.../bindings/display/brcm,bcm2711-hdmi.yaml | 3 +--
.../bindings/display/bridge/adi,adv7511.yaml | 5 ++---
.../bindings/display/bridge/synopsys,dw-hdmi.yaml | 5 ++---
.../bindings/display/panel/display-timings.yaml | 3 +--
.../devicetree/bindings/display/ste,mcde.yaml | 4 ++--
.../devicetree/bindings/input/adc-joystick.yaml | 9 ++++-----
.../bindings/leds/cznic,turris-omnia-leds.yaml | 3 +--
.../devicetree/bindings/leds/leds-lp50xx.yaml | 3 +--
.../devicetree/bindings/mfd/google,cros-ec.yaml | 12 ++++--------
.../devicetree/bindings/mtd/nand-controller.yaml | 8 +++-----
.../bindings/mtd/rockchip,nand-controller.yaml | 3 +--
.../devicetree/bindings/net/ti,cpsw-switch.yaml | 3 +--
.../bindings/phy/phy-stm32-usbphyc.yaml | 3 +--
.../bindings/power/supply/sbs,sbs-manager.yaml | 4 +---
.../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 3 +--
.../devicetree/bindings/soc/ti/ti,pruss.yaml | 15 +++------------
.../devicetree/bindings/sound/st,stm32-sai.yaml | 3 +--
.../devicetree/bindings/sound/tlv320adcx140.yaml | 13 ++++++-------
.../devicetree/bindings/spi/spi-controller.yaml | 4 +---
.../devicetree/bindings/usb/st,stusb160x.yaml | 4 +---
21 files changed, 39 insertions(+), 74 deletions(-)
diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
index 7eb8659fa610..0420fa563532 100644
--- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
+++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
@@ -104,8 +104,7 @@ properties:
- "1.5A" and "3.0A", 5V 1.5A and 5V 3.0A respectively, as defined in USB
Type-C Cable and Connector specification, when Power Delivery is not
supported.
- allOf:
- - $ref: /schemas/types.yaml#/definitions/string
+ $ref: /schemas/types.yaml#/definitions/string
enum:
- default
- 1.5A
diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml
index a1d5a32660e0..a9d34dd7bbc5 100644
--- a/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml
+++ b/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml
@@ -72,8 +72,7 @@ properties:
- const: hpd-removed
ddc:
- allOf:
- - $ref: /schemas/types.yaml#/definitions/phandle
+ $ref: /schemas/types.yaml#/definitions/phandle
description: >
Phandle of the I2C controller used for DDC EDID probing
diff --git a/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml b/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml
index d3dd7a79b909..f08a01dfedf3 100644
--- a/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/adi,adv7511.yaml
@@ -76,9 +76,8 @@ properties:
adi,input-depth:
description: Number of bits per color component at the input.
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32
- - enum: [ 8, 10, 12 ]
+ $ref: /schemas/types.yaml#/definitions/uint32
+ enum: [ 8, 10, 12 ]
adi,input-colorspace:
description: Input color space.
diff --git a/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml b/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml
index 9be44a682e67..b00246faea57 100644
--- a/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/synopsys,dw-hdmi.yaml
@@ -26,9 +26,8 @@ properties:
reg-io-width:
description:
Width (in bytes) of the registers specified by the reg property.
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32
- - enum: [1, 4]
+ $ref: /schemas/types.yaml#/definitions/uint32
+ enum: [1, 4]
default: 1
clocks:
diff --git a/Documentation/devicetree/bindings/display/panel/display-timings.yaml b/Documentation/devicetree/bindings/display/panel/display-timings.yaml
index 56903ded005e..6d30575819d3 100644
--- a/Documentation/devicetree/bindings/display/panel/display-timings.yaml
+++ b/Documentation/devicetree/bindings/display/panel/display-timings.yaml
@@ -31,8 +31,7 @@ properties:
patternProperties:
"^timing":
type: object
- allOf:
- - $ref: panel-timing.yaml#
+ $ref: panel-timing.yaml#
additionalProperties: false
diff --git a/Documentation/devicetree/bindings/display/ste,mcde.yaml b/Documentation/devicetree/bindings/display/ste,mcde.yaml
index de0c678b3c29..564ea845c82e 100644
--- a/Documentation/devicetree/bindings/display/ste,mcde.yaml
+++ b/Documentation/devicetree/bindings/display/ste,mcde.yaml
@@ -58,8 +58,8 @@ patternProperties:
"^dsi@[0-9a-f]+$":
description: subnodes for the three DSI host adapters
type: object
- allOf:
- - $ref: dsi-controller.yaml#
+ $ref: dsi-controller.yaml#
+
properties:
compatible:
const: ste,mcde-dsi
diff --git a/Documentation/devicetree/bindings/input/adc-joystick.yaml b/Documentation/devicetree/bindings/input/adc-joystick.yaml
index 721878d5b7af..2ee04e03bc22 100644
--- a/Documentation/devicetree/bindings/input/adc-joystick.yaml
+++ b/Documentation/devicetree/bindings/input/adc-joystick.yaml
@@ -61,11 +61,10 @@ patternProperties:
description: EV_ABS specific event code generated by the axis.
abs-range:
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32-array
- - items:
- - description: minimum value
- - description: maximum value
+ $ref: /schemas/types.yaml#/definitions/uint32-array
+ items:
+ - description: minimum value
+ - description: maximum value
description: >
Minimum and maximum values produced by the axis.
For an ABS_X axis this will be the left-most and right-most
diff --git a/Documentation/devicetree/bindings/leds/cznic,turris-omnia-leds.yaml b/Documentation/devicetree/bindings/leds/cznic,turris-omnia-leds.yaml
index c7ed2871da06..9362b1ef9e88 100644
--- a/Documentation/devicetree/bindings/leds/cznic,turris-omnia-leds.yaml
+++ b/Documentation/devicetree/bindings/leds/cznic,turris-omnia-leds.yaml
@@ -32,8 +32,7 @@ properties:
patternProperties:
"^multi-led@[0-9a-b]$":
type: object
- allOf:
- - $ref: leds-class-multicolor.yaml#
+ $ref: leds-class-multicolor.yaml#
description:
This node represents one of the RGB LED devices on Turris Omnia.
No subnodes need to be added for subchannels since this controller only
diff --git a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
index c192b5feadc7..f12fe5b53f30 100644
--- a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
+++ b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
@@ -55,8 +55,7 @@ properties:
patternProperties:
'^multi-led@[0-9a-f]$':
type: object
- allOf:
- - $ref: leds-class-multicolor.yaml#
+ $ref: leds-class-multicolor.yaml#
properties:
reg:
minItems: 1
diff --git a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
index d1f53bd449f7..a52f3a82283c 100644
--- a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
+++ b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
@@ -38,18 +38,14 @@ properties:
description:
This property specifies the delay in usecs between the
assertion of the CS and the first clock pulse.
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32
- - default: 0
- - minimum: 0
+ $ref: /schemas/types.yaml#/definitions/uint32
+ default: 0
google,cros-ec-spi-msg-delay:
description:
This property specifies the delay in usecs between messages.
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32
- - default: 0
- - minimum: 0
+ $ref: /schemas/types.yaml#/definitions/uint32
+ default: 0
google,has-vbc-nvram:
description:
diff --git a/Documentation/devicetree/bindings/mtd/nand-controller.yaml b/Documentation/devicetree/bindings/mtd/nand-controller.yaml
index bd217e6f5018..0714f614550d 100644
--- a/Documentation/devicetree/bindings/mtd/nand-controller.yaml
+++ b/Documentation/devicetree/bindings/mtd/nand-controller.yaml
@@ -58,8 +58,7 @@ patternProperties:
Contains the native Ready/Busy IDs.
nand-ecc-engine:
- allOf:
- - $ref: /schemas/types.yaml#/definitions/phandle
+ $ref: /schemas/types.yaml#/definitions/phandle
description: |
A phandle on the hardware ECC engine if any. There are
basically three possibilities:
@@ -79,9 +78,8 @@ patternProperties:
description: Do not use any ECC correction.
nand-ecc-placement:
- allOf:
- - $ref: /schemas/types.yaml#/definitions/string
- - enum: [ oob, interleaved ]
+ $ref: /schemas/types.yaml#/definitions/string
+ enum: [ oob, interleaved ]
description:
Location of the ECC bytes. This location is unknown by default
but can be explicitly set to "oob", if all ECC bytes are
diff --git a/Documentation/devicetree/bindings/mtd/rockchip,nand-controller.yaml b/Documentation/devicetree/bindings/mtd/rockchip,nand-controller.yaml
index 0922536b1811..d681a4676f06 100644
--- a/Documentation/devicetree/bindings/mtd/rockchip,nand-controller.yaml
+++ b/Documentation/devicetree/bindings/mtd/rockchip,nand-controller.yaml
@@ -96,8 +96,7 @@ patternProperties:
rockchip,boot-ecc-strength:
enum: [16, 24, 40, 60, 70]
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32
+ $ref: /schemas/types.yaml#/definitions/uint32
description: |
If specified it indicates that a different BCH/ECC setting is
supported by the boot ROM.
diff --git a/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml b/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml
index 07a00f53adbf..31bf825c6598 100644
--- a/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml
+++ b/Documentation/devicetree/bindings/net/ti,cpsw-switch.yaml
@@ -88,8 +88,7 @@ properties:
type: object
description: CPSW external ports
- allOf:
- - $ref: ethernet-controller.yaml#
+ $ref: ethernet-controller.yaml#
properties:
reg:
diff --git a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml
index 267b695215b6..dc287d428e49 100644
--- a/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml
+++ b/Documentation/devicetree/bindings/phy/phy-stm32-usbphyc.yaml
@@ -76,8 +76,7 @@ patternProperties:
connector:
type: object
- allOf:
- - $ref: ../connector/usb-connector.yaml
+ $ref: /schemas/connector/usb-connector.yaml
properties:
vbus-supply: true
diff --git a/Documentation/devicetree/bindings/power/supply/sbs,sbs-manager.yaml b/Documentation/devicetree/bindings/power/supply/sbs,sbs-manager.yaml
index 72e8f274c791..99f506d6b0a0 100644
--- a/Documentation/devicetree/bindings/power/supply/sbs,sbs-manager.yaml
+++ b/Documentation/devicetree/bindings/power/supply/sbs,sbs-manager.yaml
@@ -46,9 +46,7 @@ additionalProperties: false
patternProperties:
"^i2c@[1-4]$":
type: object
-
- allOf:
- - $ref: /schemas/i2c/i2c-controller.yaml#
+ $ref: /schemas/i2c/i2c-controller.yaml#
examples:
- |
diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml
index 0868154a0364..fb9605f0655b 100644
--- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml
@@ -95,8 +95,7 @@ patternProperties:
addresses. Cache and memory access settings are provided through a
Memory Protection Unit (MPU), programmable only from the R5Fs.
- allOf:
- - $ref: /schemas/arm/keystone/ti,k3-sci-common.yaml#
+ $ref: /schemas/arm/keystone/ti,k3-sci-common.yaml#
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
index 9d128b9e7deb..64461d432004 100644
--- a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
+++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
@@ -281,10 +281,7 @@ patternProperties:
PRUSS INTC Node. Each PRUSS has a single interrupt controller instance
that is common to all the PRU cores. This should be represented as an
interrupt-controller node.
-
- allOf:
- - $ref: /schemas/interrupt-controller/ti,pruss-intc.yaml#
-
+ $ref: /schemas/interrupt-controller/ti,pruss-intc.yaml#
type: object
mdio@[a-f0-9]+$:
@@ -292,10 +289,7 @@ patternProperties:
MDIO Node. Each PRUSS has an MDIO module that can be used to control
external PHYs. The MDIO module used within the PRU-ICSS is an instance of
the MDIO Controller used in TI Davinci SoCs.
-
- allOf:
- - $ref: /schemas/net/ti,davinci-mdio.yaml#
-
+ $ref: /schemas/net/ti,davinci-mdio.yaml#
type: object
"^(pru|rtu|txpru)@[0-9a-f]+$":
@@ -305,10 +299,7 @@ patternProperties:
inactive by using the standard DT string property, "status". The ICSSG IP
present on K3 SoCs have additional auxiliary PRU cores with slightly
different IP integration.
-
- allOf:
- - $ref: /schemas/remoteproc/ti,pru-rproc.yaml#
-
+ $ref: /schemas/remoteproc/ti,pru-rproc.yaml#
type: object
required:
diff --git a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml
index d4fc8fdcb72f..b3dbcba33e41 100644
--- a/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml
+++ b/Documentation/devicetree/bindings/sound/st,stm32-sai.yaml
@@ -115,8 +115,7 @@ patternProperties:
By default, custom protocol is assumed, meaning that protocol is
configured according to protocol defined in related DAI link node,
such as i2s, left justified, right justified, dsp and pdm protocols.
- allOf:
- - $ref: /schemas/types.yaml#/definitions/flag
+ $ref: /schemas/types.yaml#/definitions/flag
"#clock-cells":
description: Configure the SAI device as master clock provider.
diff --git a/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml b/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml
index d77c8283526d..2ad17b361db0 100644
--- a/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml
+++ b/Documentation/devicetree/bindings/sound/tlv320adcx140.yaml
@@ -176,13 +176,12 @@ patternProperties:
4 - Drive weak low and active high
5 - Drive Hi-Z and active high
- allOf:
- - $ref: /schemas/types.yaml#/definitions/uint32-array
- - minItems: 2
- maxItems: 2
- items:
- maximum: 15
- default: [2, 2]
+ $ref: /schemas/types.yaml#/definitions/uint32-array
+ minItems: 2
+ maxItems: 2
+ items:
+ maximum: 15
+ default: [2, 2]
required:
- compatible
diff --git a/Documentation/devicetree/bindings/spi/spi-controller.yaml b/Documentation/devicetree/bindings/spi/spi-controller.yaml
index 36b72518f565..6d0511a8727f 100644
--- a/Documentation/devicetree/bindings/spi/spi-controller.yaml
+++ b/Documentation/devicetree/bindings/spi/spi-controller.yaml
@@ -93,9 +93,7 @@ properties:
patternProperties:
"^.*@[0-9a-f]+$":
type: object
-
- allOf:
- - $ref: spi-peripheral-props.yaml
+ $ref: spi-peripheral-props.yaml
required:
- compatible
diff --git a/Documentation/devicetree/bindings/usb/st,stusb160x.yaml b/Documentation/devicetree/bindings/usb/st,stusb160x.yaml
index ead1571e0e43..b5a8c9814dd3 100644
--- a/Documentation/devicetree/bindings/usb/st,stusb160x.yaml
+++ b/Documentation/devicetree/bindings/usb/st,stusb160x.yaml
@@ -32,9 +32,7 @@ properties:
connector:
type: object
-
- allOf:
- - $ref: ../connector/usb-connector.yaml
+ $ref: /schemas/connector/usb-connector.yaml#
properties:
compatible:
--
2.32.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
@ 2022-02-28 21:45 ` Marek Behún
2022-02-28 22:09 ` Mark Brown
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Marek Behún @ 2022-02-28 21:45 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Jakub Kicinski,
Kishon Vijay Abraham I, Vinod Koul, Sebastian Reichel,
Bjorn Andersson, Mathieu Poirier, Mark Brown, Greg Kroah-Hartman,
Laurent Pinchart, dri-devel, linux-arm-kernel, linux-input,
linux-leds, linux-mtd, netdev, linux-phy, linux-pm,
linux-remoteproc, alsa-devel, linux-spi, linux-usb
On Mon, 28 Feb 2022 15:38:02 -0600
Rob Herring <robh@kernel.org> wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
For Documentation/devicetree/bindings/leds/cznic,turris-omnia-leds.yaml
Acked-by: Marek Behún <kabel@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
2022-02-28 21:45 ` Marek Behún
@ 2022-02-28 22:09 ` Mark Brown
2022-03-01 7:40 ` Miquel Raynal
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Mark Brown @ 2022-02-28 22:09 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Jakub Kicinski,
Kishon Vijay Abraham I, Vinod Koul, Sebastian Reichel,
Bjorn Andersson, Mathieu Poirier, Greg Kroah-Hartman,
Laurent Pinchart, dri-devel, linux-arm-kernel, linux-input,
linux-leds, linux-mtd, netdev, linux-phy, linux-pm,
linux-remoteproc, alsa-devel, linux-spi, linux-usb
[-- Attachment #1.1: Type: text/plain, Size: 487 bytes --]
On Mon, Feb 28, 2022 at 03:38:02PM -0600, Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
Acked-by: Mark Brown <broonie@kernel.org>
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
2022-02-28 21:45 ` Marek Behún
2022-02-28 22:09 ` Mark Brown
@ 2022-03-01 7:40 ` Miquel Raynal
2022-03-01 8:18 ` Laurent Pinchart
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Miquel Raynal @ 2022-03-01 7:40 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Richard Weinberger, Vignesh Raghavendra,
David S. Miller, Jakub Kicinski, Kishon Vijay Abraham I,
Vinod Koul, Sebastian Reichel, Bjorn Andersson, Mathieu Poirier,
Mark Brown, Greg Kroah-Hartman, Laurent Pinchart, dri-devel,
linux-arm-kernel, linux-input, linux-leds, linux-mtd, netdev,
linux-phy, linux-pm, linux-remoteproc, alsa-devel, linux-spi,
linux-usb
Hi Rob,
robh@kernel.org wrote on Mon, 28 Feb 2022 15:38:02 -0600:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Guenter Roeck <groeck@chromium.org>
> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> Cc: Richard Weinberger <richard@nod.at>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-leds@vger.kernel.org
> Cc: linux-mtd@lists.infradead.org
> Cc: netdev@vger.kernel.org
> Cc: linux-phy@lists.infradead.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-remoteproc@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/connector/usb-connector.yaml | 3 +--
> .../bindings/display/brcm,bcm2711-hdmi.yaml | 3 +--
> .../bindings/display/bridge/adi,adv7511.yaml | 5 ++---
> .../bindings/display/bridge/synopsys,dw-hdmi.yaml | 5 ++---
> .../bindings/display/panel/display-timings.yaml | 3 +--
> .../devicetree/bindings/display/ste,mcde.yaml | 4 ++--
> .../devicetree/bindings/input/adc-joystick.yaml | 9 ++++-----
> .../bindings/leds/cznic,turris-omnia-leds.yaml | 3 +--
> .../devicetree/bindings/leds/leds-lp50xx.yaml | 3 +--
> .../devicetree/bindings/mfd/google,cros-ec.yaml | 12 ++++--------
> .../devicetree/bindings/mtd/nand-controller.yaml | 8 +++-----
> .../bindings/mtd/rockchip,nand-controller.yaml | 3 +--
For MTD bindings:
Acked-by: Miquel Raynal <miquel.raynal@bootlin.com>
Thanks,
Miquèl
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
` (2 preceding siblings ...)
2022-03-01 7:40 ` Miquel Raynal
@ 2022-03-01 8:18 ` Laurent Pinchart
2022-03-01 9:14 ` Greg Kroah-Hartman
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Laurent Pinchart @ 2022-03-01 8:18 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Jakub Kicinski,
Kishon Vijay Abraham I, Vinod Koul, Sebastian Reichel,
Bjorn Andersson, Mathieu Poirier, Mark Brown, Greg Kroah-Hartman,
dri-devel, linux-arm-kernel, linux-input, linux-leds, linux-mtd,
netdev, linux-phy, linux-pm, linux-remoteproc, alsa-devel,
linux-spi, linux-usb
Hi Rob,
Thank you for the patch.
On Mon, Feb 28, 2022 at 03:38:02PM -0600, Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Guenter Roeck <groeck@chromium.org>
> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> Cc: Richard Weinberger <richard@nod.at>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-leds@vger.kernel.org
> Cc: linux-mtd@lists.infradead.org
> Cc: netdev@vger.kernel.org
> Cc: linux-phy@lists.infradead.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-remoteproc@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> .../bindings/connector/usb-connector.yaml | 3 +--
> .../bindings/display/brcm,bcm2711-hdmi.yaml | 3 +--
> .../bindings/display/bridge/adi,adv7511.yaml | 5 ++---
> .../bindings/display/bridge/synopsys,dw-hdmi.yaml | 5 ++---
> .../bindings/display/panel/display-timings.yaml | 3 +--
> .../devicetree/bindings/display/ste,mcde.yaml | 4 ++--
> .../devicetree/bindings/input/adc-joystick.yaml | 9 ++++-----
> .../bindings/leds/cznic,turris-omnia-leds.yaml | 3 +--
> .../devicetree/bindings/leds/leds-lp50xx.yaml | 3 +--
> .../devicetree/bindings/mfd/google,cros-ec.yaml | 12 ++++--------
> .../devicetree/bindings/mtd/nand-controller.yaml | 8 +++-----
> .../bindings/mtd/rockchip,nand-controller.yaml | 3 +--
> .../devicetree/bindings/net/ti,cpsw-switch.yaml | 3 +--
> .../bindings/phy/phy-stm32-usbphyc.yaml | 3 +--
> .../bindings/power/supply/sbs,sbs-manager.yaml | 4 +---
> .../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 3 +--
> .../devicetree/bindings/soc/ti/ti,pruss.yaml | 15 +++------------
> .../devicetree/bindings/sound/st,stm32-sai.yaml | 3 +--
> .../devicetree/bindings/sound/tlv320adcx140.yaml | 13 ++++++-------
> .../devicetree/bindings/spi/spi-controller.yaml | 4 +---
> .../devicetree/bindings/usb/st,stusb160x.yaml | 4 +---
> 21 files changed, 39 insertions(+), 74 deletions(-)
--
Regards,
Laurent Pinchart
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
` (3 preceding siblings ...)
2022-03-01 8:18 ` Laurent Pinchart
@ 2022-03-01 9:14 ` Greg Kroah-Hartman
2022-03-01 9:58 ` Lee Jones
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Greg Kroah-Hartman @ 2022-03-01 9:14 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Jakub Kicinski,
Kishon Vijay Abraham I, Vinod Koul, Sebastian Reichel,
Bjorn Andersson, Mathieu Poirier, Mark Brown, Laurent Pinchart,
dri-devel, linux-arm-kernel, linux-input, linux-leds, linux-mtd,
netdev, linux-phy, linux-pm, linux-remoteproc, alsa-devel,
linux-spi, linux-usb
On Mon, Feb 28, 2022 at 03:38:02PM -0600, Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Guenter Roeck <groeck@chromium.org>
> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> Cc: Richard Weinberger <richard@nod.at>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-leds@vger.kernel.org
> Cc: linux-mtd@lists.infradead.org
> Cc: netdev@vger.kernel.org
> Cc: linux-phy@lists.infradead.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-remoteproc@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
` (4 preceding siblings ...)
2022-03-01 9:14 ` Greg Kroah-Hartman
@ 2022-03-01 9:58 ` Lee Jones
2022-03-01 10:06 ` Vinod Koul
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: Lee Jones @ 2022-03-01 9:58 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Guenter Roeck,
Miquel Raynal, Richard Weinberger, Vignesh Raghavendra,
David S. Miller, Jakub Kicinski, Kishon Vijay Abraham I,
Vinod Koul, Sebastian Reichel, Bjorn Andersson, Mathieu Poirier,
Mark Brown, Greg Kroah-Hartman, Laurent Pinchart, dri-devel,
linux-arm-kernel, linux-input, linux-leds, linux-mtd, netdev,
linux-phy, linux-pm, linux-remoteproc, alsa-devel, linux-spi,
linux-usb
On Mon, 28 Feb 2022, Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Guenter Roeck <groeck@chromium.org>
> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> Cc: Richard Weinberger <richard@nod.at>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-leds@vger.kernel.org
> Cc: linux-mtd@lists.infradead.org
> Cc: netdev@vger.kernel.org
> Cc: linux-phy@lists.infradead.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-remoteproc@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/connector/usb-connector.yaml | 3 +--
> .../bindings/display/brcm,bcm2711-hdmi.yaml | 3 +--
> .../bindings/display/bridge/adi,adv7511.yaml | 5 ++---
> .../bindings/display/bridge/synopsys,dw-hdmi.yaml | 5 ++---
> .../bindings/display/panel/display-timings.yaml | 3 +--
> .../devicetree/bindings/display/ste,mcde.yaml | 4 ++--
> .../devicetree/bindings/input/adc-joystick.yaml | 9 ++++-----
> .../bindings/leds/cznic,turris-omnia-leds.yaml | 3 +--
> .../devicetree/bindings/leds/leds-lp50xx.yaml | 3 +--
> .../devicetree/bindings/mfd/google,cros-ec.yaml | 12 ++++--------
Go for it.
Acked-by: Lee Jones <lee.jones@linaro.org>
> .../devicetree/bindings/mtd/nand-controller.yaml | 8 +++-----
> .../bindings/mtd/rockchip,nand-controller.yaml | 3 +--
> .../devicetree/bindings/net/ti,cpsw-switch.yaml | 3 +--
> .../bindings/phy/phy-stm32-usbphyc.yaml | 3 +--
> .../bindings/power/supply/sbs,sbs-manager.yaml | 4 +---
> .../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 3 +--
> .../devicetree/bindings/soc/ti/ti,pruss.yaml | 15 +++------------
> .../devicetree/bindings/sound/st,stm32-sai.yaml | 3 +--
> .../devicetree/bindings/sound/tlv320adcx140.yaml | 13 ++++++-------
> .../devicetree/bindings/spi/spi-controller.yaml | 4 +---
> .../devicetree/bindings/usb/st,stusb160x.yaml | 4 +---
> 21 files changed, 39 insertions(+), 74 deletions(-)
--
Lee Jones [李琼斯]
Principal Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
` (5 preceding siblings ...)
2022-03-01 9:58 ` Lee Jones
@ 2022-03-01 10:06 ` Vinod Koul
2022-03-02 2:06 ` Jakub Kicinski
2022-03-02 2:47 ` Bjorn Andersson
8 siblings, 0 replies; 10+ messages in thread
From: Vinod Koul @ 2022-03-01 10:06 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Jakub Kicinski,
Kishon Vijay Abraham I, Sebastian Reichel, Bjorn Andersson,
Mathieu Poirier, Mark Brown, Greg Kroah-Hartman,
Laurent Pinchart, dri-devel, linux-arm-kernel, linux-input,
linux-leds, linux-mtd, netdev, linux-phy, linux-pm,
linux-remoteproc, alsa-devel, linux-spi, linux-usb
On 28-02-22, 15:38, Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
...
> .../bindings/connector/usb-connector.yaml | 3 +--
> .../bindings/display/brcm,bcm2711-hdmi.yaml | 3 +--
> .../bindings/display/bridge/adi,adv7511.yaml | 5 ++---
> .../bindings/display/bridge/synopsys,dw-hdmi.yaml | 5 ++---
> .../bindings/display/panel/display-timings.yaml | 3 +--
> .../devicetree/bindings/display/ste,mcde.yaml | 4 ++--
> .../devicetree/bindings/input/adc-joystick.yaml | 9 ++++-----
> .../bindings/leds/cznic,turris-omnia-leds.yaml | 3 +--
> .../devicetree/bindings/leds/leds-lp50xx.yaml | 3 +--
> .../devicetree/bindings/mfd/google,cros-ec.yaml | 12 ++++--------
> .../devicetree/bindings/mtd/nand-controller.yaml | 8 +++-----
> .../bindings/mtd/rockchip,nand-controller.yaml | 3 +--
> .../devicetree/bindings/net/ti,cpsw-switch.yaml | 3 +--
> .../bindings/phy/phy-stm32-usbphyc.yaml | 3 +--
Acked-By: Vinod Koul <vkoul@kernel.org>
--
~Vinod
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
` (6 preceding siblings ...)
2022-03-01 10:06 ` Vinod Koul
@ 2022-03-02 2:06 ` Jakub Kicinski
2022-03-02 2:47 ` Bjorn Andersson
8 siblings, 0 replies; 10+ messages in thread
From: Jakub Kicinski @ 2022-03-02 2:06 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Kishon Vijay Abraham I,
Vinod Koul, Sebastian Reichel, Bjorn Andersson, Mathieu Poirier,
Mark Brown, Greg Kroah-Hartman, Laurent Pinchart, dri-devel,
linux-arm-kernel, linux-input, linux-leds, linux-mtd, netdev,
linux-phy, linux-pm, linux-remoteproc, alsa-devel, linux-spi,
linux-usb
On Mon, 28 Feb 2022 15:38:02 -0600 Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
Acked-by: Jakub Kicinski <kuba@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
` (7 preceding siblings ...)
2022-03-02 2:06 ` Jakub Kicinski
@ 2022-03-02 2:47 ` Bjorn Andersson
8 siblings, 0 replies; 10+ messages in thread
From: Bjorn Andersson @ 2022-03-02 2:47 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Krzysztof Kozlowski, Thierry Reding,
Sam Ravnborg, Dmitry Torokhov, Pavel Machek, Lee Jones,
Guenter Roeck, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, David S. Miller, Jakub Kicinski,
Kishon Vijay Abraham I, Vinod Koul, Sebastian Reichel,
Mathieu Poirier, Mark Brown, Greg Kroah-Hartman,
Laurent Pinchart, dri-devel, linux-arm-kernel, linux-input,
linux-leds, linux-mtd, netdev, linux-phy, linux-pm,
linux-remoteproc, alsa-devel, linux-spi, linux-usb
On Mon 28 Feb 13:38 PST 2022, Rob Herring wrote:
> Another pass at removing unnecessary use of 'allOf' with a '$ref'.
>
> json-schema versions draft7 and earlier have a weird behavior in that
> any keywords combined with a '$ref' are ignored (silently). The correct
> form was to put a '$ref' under an 'allOf'. This behavior is now changed
> in the 2019-09 json-schema spec and '$ref' can be mixed with other
> keywords.
>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Guenter Roeck <groeck@chromium.org>
> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> Cc: Richard Weinberger <richard@nod.at>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Kishon Vijay Abraham I <kishon@ti.com>
> Cc: Vinod Koul <vkoul@kernel.org>
> Cc: Sebastian Reichel <sre@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Mathieu Poirier <mathieu.poirier@linaro.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-input@vger.kernel.org
> Cc: linux-leds@vger.kernel.org
> Cc: linux-mtd@lists.infradead.org
> Cc: netdev@vger.kernel.org
> Cc: linux-phy@lists.infradead.org
> Cc: linux-pm@vger.kernel.org
> Cc: linux-remoteproc@vger.kernel.org
> Cc: alsa-devel@alsa-project.org
> Cc: linux-spi@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> .../bindings/connector/usb-connector.yaml | 3 +--
> .../bindings/display/brcm,bcm2711-hdmi.yaml | 3 +--
> .../bindings/display/bridge/adi,adv7511.yaml | 5 ++---
> .../bindings/display/bridge/synopsys,dw-hdmi.yaml | 5 ++---
> .../bindings/display/panel/display-timings.yaml | 3 +--
> .../devicetree/bindings/display/ste,mcde.yaml | 4 ++--
> .../devicetree/bindings/input/adc-joystick.yaml | 9 ++++-----
> .../bindings/leds/cznic,turris-omnia-leds.yaml | 3 +--
> .../devicetree/bindings/leds/leds-lp50xx.yaml | 3 +--
> .../devicetree/bindings/mfd/google,cros-ec.yaml | 12 ++++--------
> .../devicetree/bindings/mtd/nand-controller.yaml | 8 +++-----
> .../bindings/mtd/rockchip,nand-controller.yaml | 3 +--
> .../devicetree/bindings/net/ti,cpsw-switch.yaml | 3 +--
> .../bindings/phy/phy-stm32-usbphyc.yaml | 3 +--
> .../bindings/power/supply/sbs,sbs-manager.yaml | 4 +---
> .../bindings/remoteproc/ti,k3-r5f-rproc.yaml | 3 +--
For the remoteproc binding:
Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Thanks,
Bjorn
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-03-02 2:47 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-28 21:38 [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref' Rob Herring
2022-02-28 21:45 ` Marek Behún
2022-02-28 22:09 ` Mark Brown
2022-03-01 7:40 ` Miquel Raynal
2022-03-01 8:18 ` Laurent Pinchart
2022-03-01 9:14 ` Greg Kroah-Hartman
2022-03-01 9:58 ` Lee Jones
2022-03-01 10:06 ` Vinod Koul
2022-03-02 2:06 ` Jakub Kicinski
2022-03-02 2:47 ` Bjorn Andersson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).