Re: [PATCH] dt-bindings: Fix JSON pointers
From: Lee Jones
Date: Fri Dec 18 2020 - 08:14:28 EST
On Thu, 17 Dec 2020, Rob Herring wrote:
> The correct syntax for JSON pointers begins with a '/' after the '#'.
> Without a '/', the string should be interpretted as a subschema
> identifier. The jsonschema module currently doesn't handle subschema
> identifiers and incorrectly allows JSON pointers to begin without a '/'.
> Let's fix this before it becomes a problem when jsonschema module is
> fixed.
>
> Converted with:
> perl -p -i -e 's/yaml#definitions/yaml#\/definitions/g' `find Documentation/devicetree/bindings/ -name "*.yaml"`
>
> Cc: Maxime Ripard <mripard@xxxxxxxxxx>
> Cc: Vinod Koul <vkoul@xxxxxxxxxx>
> Cc: Guenter Roeck <linux@xxxxxxxxxxxx>
> Cc: Jonathan Cameron <jic23@xxxxxxxxxx>
> Cc: Lars-Peter Clausen <lars@xxxxxxxxxx>
> Cc: Lee Jones <lee.jones@xxxxxxxxxx>
> Cc: Daniel Thompson <daniel.thompson@xxxxxxxxxx>
> Cc: Jingoo Han <jingoohan1@xxxxxxxxx>
> Cc: Pavel Machek <pavel@xxxxxx>
> Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
> Cc: Andrew Lunn <andrew@xxxxxxx>
> Cc: Florian Fainelli <f.fainelli@xxxxxxxxx>
> Cc: Sebastian Reichel <sre@xxxxxxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Cc: netdev@xxxxxxxxxxxxxxx
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> .../devicetree/bindings/arm/idle-states.yaml | 2 +-
> .../bus/allwinner,sun50i-a64-de2.yaml | 2 +-
> .../bindings/bus/baikal,bt1-axi.yaml | 2 +-
> .../bindings/connector/usb-connector.yaml | 10 ++---
> .../devicetree/bindings/dma/dma-common.yaml | 4 +-
> .../devicetree/bindings/dma/dma-router.yaml | 2 +-
> .../devicetree/bindings/dma/ingenic,dma.yaml | 2 +-
> .../bindings/dma/snps,dma-spear1340.yaml | 10 ++---
> .../devicetree/bindings/eeprom/at24.yaml | 4 +-
> .../devicetree/bindings/eeprom/at25.yaml | 4 +-
> .../bindings/hwmon/moortec,mr75203.yaml | 2 +-
> .../bindings/hwmon/sensirion,shtc1.yaml | 4 +-
> .../devicetree/bindings/hwmon/ti,tmp513.yaml | 2 +-
> .../bindings/iio/light/upisemi,us5182.yaml | 2 +-
> .../iio/proximity/semtech,sx9310.yaml | 6 +--
> .../devicetree/bindings/input/gpio-keys.yaml | 12 +++---
> .../interrupt-controller/mti,gic.yaml | 4 +-
> .../interrupt-controller/ti,pruss-intc.yaml | 2 +-
> .../interrupt-controller/ti,sci-inta.yaml | 2 +-
> .../bindings/leds/backlight/common.yaml | 4 +-
Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>
> .../devicetree/bindings/leds/common.yaml | 16 ++++----
> .../devicetree/bindings/leds/leds-lp55xx.yaml | 10 ++---
> .../net/allwinner,sun8i-a83t-emac.yaml | 6 +--
> .../bindings/net/amlogic,meson-dwmac.yaml | 2 +-
> .../devicetree/bindings/net/dsa/dsa.yaml | 6 +--
> .../bindings/net/ethernet-controller.yaml | 24 ++++++------
> .../devicetree/bindings/net/ethernet-phy.yaml | 20 +++++-----
> .../bindings/net/fsl,qoriq-mc-dpmac.yaml | 2 +-
> .../devicetree/bindings/net/mdio.yaml | 2 +-
> .../bindings/net/mediatek,star-emac.yaml | 2 +-
> .../devicetree/bindings/net/qcom,ipa.yaml | 2 +-
> .../devicetree/bindings/net/snps,dwmac.yaml | 38 +++++++++----------
> .../bindings/net/socionext,uniphier-ave4.yaml | 2 +-
> .../bindings/net/ti,cpsw-switch.yaml | 2 +-
> .../devicetree/bindings/net/ti,dp83867.yaml | 12 +++---
> .../devicetree/bindings/net/ti,dp83869.yaml | 8 ++--
> .../bindings/net/ti,k3-am654-cpsw-nuss.yaml | 4 +-
> .../bindings/net/wireless/qcom,ath11k.yaml | 2 +-
> .../devicetree/bindings/phy/ti,omap-usb2.yaml | 4 +-
> .../power/mediatek,power-controller.yaml | 12 +++---
> .../bindings/power/supply/cw2015_battery.yaml | 2 +-
> .../devicetree/bindings/powerpc/sleep.yaml | 2 +-
> .../devicetree/bindings/serial/8250.yaml | 6 +--
> .../bindings/soc/ti/k3-ringacc.yaml | 2 +-
> .../sound/allwinner,sun4i-a10-codec.yaml | 2 +-
> .../bindings/sound/st,stm32-sai.yaml | 4 +-
> 46 files changed, 138 insertions(+), 138 deletions(-)
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog