linux-kernel - Re: [PATCH] dt-bindings: Fix JSON pointers

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
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20201218131342.GQ207743@dell>
Date:   Fri, 18 Dec 2020 13:13:42 +0000
From:   Lee Jones <lee.jones@...aro.org>
To:     Rob Herring <robh@...nel.org>
Cc:     devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        Maxime Ripard <mripard@...nel.org>,
        Vinod Koul <vkoul@...nel.org>,
        Guenter Roeck <linux@...ck-us.net>,
        Jonathan Cameron <jic23@...nel.org>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Daniel Thompson <daniel.thompson@...aro.org>,
        Jingoo Han <jingoohan1@...il.com>, Pavel Machek <pavel@....cz>,
        "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>, Andrew Lunn <andrew@...n.ch>,
        Florian Fainelli <f.fainelli@...il.com>,
        Sebastian Reichel <sre@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Mark Brown <broonie@...nel.org>, netdev@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: Fix JSON pointers

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@...nel.org>
> Cc: Vinod Koul <vkoul@...nel.org>
> Cc: Guenter Roeck <linux@...ck-us.net>
> Cc: Jonathan Cameron <jic23@...nel.org>
> Cc: Lars-Peter Clausen <lars@...afoo.de>
> Cc: Lee Jones <lee.jones@...aro.org>
> Cc: Daniel Thompson <daniel.thompson@...aro.org>
> Cc: Jingoo Han <jingoohan1@...il.com>
> Cc: Pavel Machek <pavel@....cz>
> Cc: "David S. Miller" <davem@...emloft.net>
> Cc: Jakub Kicinski <kuba@...nel.org>
> Cc: Andrew Lunn <andrew@...n.ch>
> Cc: Florian Fainelli <f.fainelli@...il.com>
> Cc: Sebastian Reichel <sre@...nel.org>
> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Cc: Mark Brown <broonie@...nel.org>
> Cc: netdev@...r.kernel.org
> Signed-off-by: Rob Herring <robh@...nel.org>
> ---
>  .../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@...aro.org>

>  .../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