From: Guenter Roeck <linux@roeck-us.net>
To: Rob Herring <robh@kernel.org>
Cc: Nishanth Menon <nm@ti.com>,
alsa-devel@alsa-project.org,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Yangtao Li <tiny.windzz@gmail.com>,
Viresh Kumar <viresh.kumar@linaro.org>,
Konrad Dybcio <konrad.dybcio@somainline.org>,
linux-kernel@vger.kernel.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Thierry Reding <thierry.reding@gmail.com>,
Krzysztof Kozlowski <krzk@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
linux-spi@vger.kernel.org,
Marek Vasut <marek.vasut+renesas@gmail.com>,
Ryder Lee <ryder.lee@mediatek.com>,
Samuel Holland <samuel@sholland.org>,
Ilia Lin <ilia.lin@kernel.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Jianjun Wang <jianjun.wang@mediatek.com>,
Sergio Paracuellos <sergio.paracuellos@gmail.com>,
Chen-Yu Tsai <wens@csie.org>, Andy Gross <agross@kernel.org>,
linux-pci@vger.kernel.org,
Javier Martinez Canillas <javier@dowhile0.org>,
linux-sunxi@lists.linux.dev, devicetree@vger.kernel.org,
Jean Delvare <jdelvare@suse.com>,
linux-tegra@vger.kernel.org, linux-pm@vger.kernel.org,
linux-arm-msm@vger.kernel.org, Maxime Ripard <mripard@kernel.org>,
Mark Brown <broonie@kernel.org>,
linux-mediatek@lists.infradead.org,
Bjorn Helgaas <bhelgaas@google.com>,
linux-arm-kernel@lists.infradead.org,
linux-hwmon@vger.kernel.org, Stephen Boyd <sboyd@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Daniel Mack <zonque@gmail.com>,
linux-renesas-soc@vger.kernel.org,
Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Subject: Re: [PATCH] dt-bindings: Add missing start and/or end of line regex anchors
Date: Sun, 20 Nov 2022 05:53:08 -0800 [thread overview]
Message-ID: <20221120135308.GA1787641@roeck-us.net> (raw)
In-Reply-To: <20221118223728.1721589-1-robh@kernel.org>
On Fri, Nov 18, 2022 at 04:37:27PM -0600, Rob Herring wrote:
> json-schema patterns by default will match anywhere in a string, so
> typically we want at least the start or end anchored. Fix the obvious
> cases where the anchors were forgotten.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> Acked-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> Acked-by: Mark Brown <broonie@kernel.org>
> ---
> .../devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml | 2 +-
> Documentation/devicetree/bindings/hwmon/adt7475.yaml | 4 ++--
For hwmon:
Acked-by: Guenter Roeck <linux@roeck-us.net>
> .../bindings/opp/allwinner,sun50i-h6-operating-points.yaml | 4 ++--
> .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 2 +-
> .../devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml | 2 +-
> Documentation/devicetree/bindings/regulator/max8660.yaml | 2 +-
> .../devicetree/bindings/regulator/maxim,max77802.yaml | 2 +-
> Documentation/devicetree/bindings/regulator/regulator.yaml | 2 +-
> .../devicetree/bindings/regulator/rohm,bd9576-regulator.yaml | 2 +-
> Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 2 +-
> .../devicetree/bindings/spi/nvidia,tegra210-quad.yaml | 2 +-
> 11 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml b/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> index a11e1b867379..3c00ad09eeaa 100644
> --- a/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> +++ b/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> @@ -38,7 +38,7 @@ properties:
> type: object
>
> patternProperties:
> - 'cpu@[0-9a-f]+':
> + '^cpu@[0-9a-f]+$':
> type: object
>
> properties:
> diff --git a/Documentation/devicetree/bindings/hwmon/adt7475.yaml b/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> index ea595102a86e..051c976ab711 100644
> --- a/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> @@ -61,7 +61,7 @@ patternProperties:
> $ref: /schemas/types.yaml#/definitions/uint32
> enum: [0, 1]
>
> - "adi,pin(5|10)-function":
> + "^adi,pin(5|10)-function$":
> description: |
> Configures the function for pin 5 on the adi,adt7473 and adi,adt7475. Or
> pin 10 on the adi,adt7476 and adi,adt7490.
> @@ -70,7 +70,7 @@ patternProperties:
> - pwm2
> - smbalert#
>
> - "adi,pin(9|14)-function":
> + "^adi,pin(9|14)-function$":
> description: |
> Configures the function for pin 9 on the adi,adt7473 and adi,adt7475. Or
> pin 14 on the adi,adt7476 and adi,adt7490
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index 385b0692261c..51f62c3ae194 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -41,7 +41,7 @@ required:
> - nvmem-cells
>
> patternProperties:
> - "opp-[0-9]+":
> + "^opp-[0-9]+$":
> type: object
>
> properties:
> @@ -49,7 +49,7 @@ patternProperties:
> clock-latency-ns: true
>
> patternProperties:
> - "opp-microvolt-.*": true
> + "^opp-microvolt-speed[0-9]$": true
>
> required:
> - opp-hz
> diff --git a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> index d60f43fd9c5a..e63e6458cea8 100644
> --- a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> +++ b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> @@ -31,7 +31,7 @@ properties:
> maxItems: 2
>
> patternProperties:
> - 'pcie@[0-2],0':
> + '^pcie@[0-2],0$':
> type: object
> $ref: /schemas/pci/pci-bus.yaml#
>
> diff --git a/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml b/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> index 0f18cceba3d5..5a0d64d3ae6b 100644
> --- a/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> +++ b/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> @@ -65,7 +65,7 @@ properties:
> maxItems: 1
>
> patternProperties:
> - 'usb@[0-1],0':
> + '^usb@[0-1],0$':
> type: object
>
> description:
> diff --git a/Documentation/devicetree/bindings/regulator/max8660.yaml b/Documentation/devicetree/bindings/regulator/max8660.yaml
> index 9c038698f880..4d550ca396eb 100644
> --- a/Documentation/devicetree/bindings/regulator/max8660.yaml
> +++ b/Documentation/devicetree/bindings/regulator/max8660.yaml
> @@ -24,7 +24,7 @@ properties:
> type: object
>
> patternProperties:
> - "regulator-.+":
> + "^regulator-.+$":
> $ref: "regulator.yaml#"
>
> additionalProperties: false
> diff --git a/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml b/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> index 71138c611b6c..b704f05ea454 100644
> --- a/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> +++ b/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> @@ -77,7 +77,7 @@ patternProperties:
> regulator-initial-mode: false
>
> patternProperties:
> - regulator-state-(standby|mem|disk):
> + "^regulator-state-(standby|mem|disk)$":
> type: object
> additionalProperties: true
> properties:
> diff --git a/Documentation/devicetree/bindings/regulator/regulator.yaml b/Documentation/devicetree/bindings/regulator/regulator.yaml
> index 6e8aa9eed3aa..53b81d8a2d41 100644
> --- a/Documentation/devicetree/bindings/regulator/regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/regulator.yaml
> @@ -231,7 +231,7 @@ patternProperties:
> ".*-supply$":
> description: Input supply phandle(s) for this node
>
> - regulator-state-(standby|mem|disk):
> + "^regulator-state-(standby|mem|disk)$":
> type: object
> description:
> sub-nodes for regulator state in Standby, Suspend-to-RAM, and
> diff --git a/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml b/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> index 7cb74cc8c5d9..54be194bb244 100644
> --- a/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> @@ -21,7 +21,7 @@ description: |
> regulator-voutl1, regulator-vouts1
>
> patternProperties:
> - "regulator-.+":
> + "^regulator-.+$":
> type: object
> description:
> Properties for single regulator.
> diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> index 679a246dd666..7df40c38e865 100644
> --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> @@ -115,7 +115,7 @@ properties:
> ports:
> $ref: /schemas/graph.yaml#/properties/ports
> patternProperties:
> - port(@[0-9a-f]+)?:
> + '^port(@[0-9a-f]+)?$':
> $ref: audio-graph-port.yaml#
> unevaluatedProperties: false
>
> diff --git a/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml b/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> index 6b733e5c1163..899100e783c9 100644
> --- a/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> +++ b/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> @@ -48,7 +48,7 @@ properties:
> - const: tx
>
> patternProperties:
> - "@[0-9a-f]+":
> + "@[0-9a-f]+$":
> type: object
>
> properties:
WARNING: multiple messages have this Message-ID (diff)
From: Guenter Roeck <linux@roeck-us.net>
To: Rob Herring <robh@kernel.org>
Cc: Ilia Lin <ilia.lin@kernel.org>, Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@somainline.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Jean Delvare <jdelvare@suse.com>,
Yangtao Li <tiny.windzz@gmail.com>, Nishanth Menon <nm@ti.com>,
Stephen Boyd <sboyd@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Samuel Holland <samuel@sholland.org>,
Sergio Paracuellos <sergio.paracuellos@gmail.com>,
Ryder Lee <ryder.lee@mediatek.com>,
Jianjun Wang <jianjun.wang@mediatek.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Marek Vasut <marek.vasut+renesas@gmail.com>,
Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
Javier Martinez Canillas <javier@dowhile0.org>,
Thierry Reding <thierry.reding@gmail.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Maxime Ripard <mripard@kernel.org>,
Daniel Mack <zonque@gmail.com>,
Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
Krzysztof Kozlowski <krzk@kernel.org>,
linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-hwmon@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-sunxi@lists.linux.dev, linux-pci@vger.kernel.org,
linux-mediatek@lists.infradead.org,
linux-renesas-soc@vger.kernel.org, alsa-devel@alsa-project.org,
linux-spi@vger.kernel.org, linux-tegra@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: Add missing start and/or end of line regex anchors
Date: Sun, 20 Nov 2022 05:53:08 -0800 [thread overview]
Message-ID: <20221120135308.GA1787641@roeck-us.net> (raw)
In-Reply-To: <20221118223728.1721589-1-robh@kernel.org>
On Fri, Nov 18, 2022 at 04:37:27PM -0600, Rob Herring wrote:
> json-schema patterns by default will match anywhere in a string, so
> typically we want at least the start or end anchored. Fix the obvious
> cases where the anchors were forgotten.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> Acked-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> Acked-by: Mark Brown <broonie@kernel.org>
> ---
> .../devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml | 2 +-
> Documentation/devicetree/bindings/hwmon/adt7475.yaml | 4 ++--
For hwmon:
Acked-by: Guenter Roeck <linux@roeck-us.net>
> .../bindings/opp/allwinner,sun50i-h6-operating-points.yaml | 4 ++--
> .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 2 +-
> .../devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml | 2 +-
> Documentation/devicetree/bindings/regulator/max8660.yaml | 2 +-
> .../devicetree/bindings/regulator/maxim,max77802.yaml | 2 +-
> Documentation/devicetree/bindings/regulator/regulator.yaml | 2 +-
> .../devicetree/bindings/regulator/rohm,bd9576-regulator.yaml | 2 +-
> Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 2 +-
> .../devicetree/bindings/spi/nvidia,tegra210-quad.yaml | 2 +-
> 11 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml b/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> index a11e1b867379..3c00ad09eeaa 100644
> --- a/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> +++ b/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> @@ -38,7 +38,7 @@ properties:
> type: object
>
> patternProperties:
> - 'cpu@[0-9a-f]+':
> + '^cpu@[0-9a-f]+$':
> type: object
>
> properties:
> diff --git a/Documentation/devicetree/bindings/hwmon/adt7475.yaml b/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> index ea595102a86e..051c976ab711 100644
> --- a/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> @@ -61,7 +61,7 @@ patternProperties:
> $ref: /schemas/types.yaml#/definitions/uint32
> enum: [0, 1]
>
> - "adi,pin(5|10)-function":
> + "^adi,pin(5|10)-function$":
> description: |
> Configures the function for pin 5 on the adi,adt7473 and adi,adt7475. Or
> pin 10 on the adi,adt7476 and adi,adt7490.
> @@ -70,7 +70,7 @@ patternProperties:
> - pwm2
> - smbalert#
>
> - "adi,pin(9|14)-function":
> + "^adi,pin(9|14)-function$":
> description: |
> Configures the function for pin 9 on the adi,adt7473 and adi,adt7475. Or
> pin 14 on the adi,adt7476 and adi,adt7490
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index 385b0692261c..51f62c3ae194 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -41,7 +41,7 @@ required:
> - nvmem-cells
>
> patternProperties:
> - "opp-[0-9]+":
> + "^opp-[0-9]+$":
> type: object
>
> properties:
> @@ -49,7 +49,7 @@ patternProperties:
> clock-latency-ns: true
>
> patternProperties:
> - "opp-microvolt-.*": true
> + "^opp-microvolt-speed[0-9]$": true
>
> required:
> - opp-hz
> diff --git a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> index d60f43fd9c5a..e63e6458cea8 100644
> --- a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> +++ b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> @@ -31,7 +31,7 @@ properties:
> maxItems: 2
>
> patternProperties:
> - 'pcie@[0-2],0':
> + '^pcie@[0-2],0$':
> type: object
> $ref: /schemas/pci/pci-bus.yaml#
>
> diff --git a/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml b/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> index 0f18cceba3d5..5a0d64d3ae6b 100644
> --- a/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> +++ b/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> @@ -65,7 +65,7 @@ properties:
> maxItems: 1
>
> patternProperties:
> - 'usb@[0-1],0':
> + '^usb@[0-1],0$':
> type: object
>
> description:
> diff --git a/Documentation/devicetree/bindings/regulator/max8660.yaml b/Documentation/devicetree/bindings/regulator/max8660.yaml
> index 9c038698f880..4d550ca396eb 100644
> --- a/Documentation/devicetree/bindings/regulator/max8660.yaml
> +++ b/Documentation/devicetree/bindings/regulator/max8660.yaml
> @@ -24,7 +24,7 @@ properties:
> type: object
>
> patternProperties:
> - "regulator-.+":
> + "^regulator-.+$":
> $ref: "regulator.yaml#"
>
> additionalProperties: false
> diff --git a/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml b/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> index 71138c611b6c..b704f05ea454 100644
> --- a/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> +++ b/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> @@ -77,7 +77,7 @@ patternProperties:
> regulator-initial-mode: false
>
> patternProperties:
> - regulator-state-(standby|mem|disk):
> + "^regulator-state-(standby|mem|disk)$":
> type: object
> additionalProperties: true
> properties:
> diff --git a/Documentation/devicetree/bindings/regulator/regulator.yaml b/Documentation/devicetree/bindings/regulator/regulator.yaml
> index 6e8aa9eed3aa..53b81d8a2d41 100644
> --- a/Documentation/devicetree/bindings/regulator/regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/regulator.yaml
> @@ -231,7 +231,7 @@ patternProperties:
> ".*-supply$":
> description: Input supply phandle(s) for this node
>
> - regulator-state-(standby|mem|disk):
> + "^regulator-state-(standby|mem|disk)$":
> type: object
> description:
> sub-nodes for regulator state in Standby, Suspend-to-RAM, and
> diff --git a/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml b/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> index 7cb74cc8c5d9..54be194bb244 100644
> --- a/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> @@ -21,7 +21,7 @@ description: |
> regulator-voutl1, regulator-vouts1
>
> patternProperties:
> - "regulator-.+":
> + "^regulator-.+$":
> type: object
> description:
> Properties for single regulator.
> diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> index 679a246dd666..7df40c38e865 100644
> --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> @@ -115,7 +115,7 @@ properties:
> ports:
> $ref: /schemas/graph.yaml#/properties/ports
> patternProperties:
> - port(@[0-9a-f]+)?:
> + '^port(@[0-9a-f]+)?$':
> $ref: audio-graph-port.yaml#
> unevaluatedProperties: false
>
> diff --git a/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml b/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> index 6b733e5c1163..899100e783c9 100644
> --- a/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> +++ b/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> @@ -48,7 +48,7 @@ properties:
> - const: tx
>
> patternProperties:
> - "@[0-9a-f]+":
> + "@[0-9a-f]+$":
> type: object
>
> properties:
WARNING: multiple messages have this Message-ID (diff)
From: Guenter Roeck <linux@roeck-us.net>
To: Rob Herring <robh@kernel.org>
Cc: Ilia Lin <ilia.lin@kernel.org>, Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@somainline.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Viresh Kumar <viresh.kumar@linaro.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Jean Delvare <jdelvare@suse.com>,
Yangtao Li <tiny.windzz@gmail.com>, Nishanth Menon <nm@ti.com>,
Stephen Boyd <sboyd@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Samuel Holland <samuel@sholland.org>,
Sergio Paracuellos <sergio.paracuellos@gmail.com>,
Ryder Lee <ryder.lee@mediatek.com>,
Jianjun Wang <jianjun.wang@mediatek.com>,
Bjorn Helgaas <bhelgaas@google.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Marek Vasut <marek.vasut+renesas@gmail.com>,
Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>,
Javier Martinez Canillas <javier@dowhile0.org>,
Thierry Reding <thierry.reding@gmail.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Maxime Ripard <mripard@kernel.org>,
Daniel Mack <zonque@gmail.com>,
Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
Krzysztof Kozlowski <krzk@kernel.org>,
linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-hwmon@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-sunxi@lists.linux.dev, linux-pci@vger.kernel.org,
linux-mediatek@lists.infradead.org,
linux-renesas-soc@vger.kernel.org, alsa-devel@alsa-project.org,
linux-spi@vger.kernel.org, linux-tegra@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: Add missing start and/or end of line regex anchors
Date: Sun, 20 Nov 2022 05:53:08 -0800 [thread overview]
Message-ID: <20221120135308.GA1787641@roeck-us.net> (raw)
In-Reply-To: <20221118223728.1721589-1-robh@kernel.org>
On Fri, Nov 18, 2022 at 04:37:27PM -0600, Rob Herring wrote:
> json-schema patterns by default will match anywhere in a string, so
> typically we want at least the start or end anchored. Fix the obvious
> cases where the anchors were forgotten.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> Acked-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> Acked-by: Mark Brown <broonie@kernel.org>
> ---
> .../devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml | 2 +-
> Documentation/devicetree/bindings/hwmon/adt7475.yaml | 4 ++--
For hwmon:
Acked-by: Guenter Roeck <linux@roeck-us.net>
> .../bindings/opp/allwinner,sun50i-h6-operating-points.yaml | 4 ++--
> .../devicetree/bindings/pci/mediatek,mt7621-pcie.yaml | 2 +-
> .../devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml | 2 +-
> Documentation/devicetree/bindings/regulator/max8660.yaml | 2 +-
> .../devicetree/bindings/regulator/maxim,max77802.yaml | 2 +-
> Documentation/devicetree/bindings/regulator/regulator.yaml | 2 +-
> .../devicetree/bindings/regulator/rohm,bd9576-regulator.yaml | 2 +-
> Documentation/devicetree/bindings/sound/renesas,rsnd.yaml | 2 +-
> .../devicetree/bindings/spi/nvidia,tegra210-quad.yaml | 2 +-
> 11 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml b/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> index a11e1b867379..3c00ad09eeaa 100644
> --- a/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> +++ b/Documentation/devicetree/bindings/cpufreq/qcom-cpufreq-nvmem.yaml
> @@ -38,7 +38,7 @@ properties:
> type: object
>
> patternProperties:
> - 'cpu@[0-9a-f]+':
> + '^cpu@[0-9a-f]+$':
> type: object
>
> properties:
> diff --git a/Documentation/devicetree/bindings/hwmon/adt7475.yaml b/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> index ea595102a86e..051c976ab711 100644
> --- a/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/adt7475.yaml
> @@ -61,7 +61,7 @@ patternProperties:
> $ref: /schemas/types.yaml#/definitions/uint32
> enum: [0, 1]
>
> - "adi,pin(5|10)-function":
> + "^adi,pin(5|10)-function$":
> description: |
> Configures the function for pin 5 on the adi,adt7473 and adi,adt7475. Or
> pin 10 on the adi,adt7476 and adi,adt7490.
> @@ -70,7 +70,7 @@ patternProperties:
> - pwm2
> - smbalert#
>
> - "adi,pin(9|14)-function":
> + "^adi,pin(9|14)-function$":
> description: |
> Configures the function for pin 9 on the adi,adt7473 and adi,adt7475. Or
> pin 14 on the adi,adt7476 and adi,adt7490
> diff --git a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> index 385b0692261c..51f62c3ae194 100644
> --- a/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> +++ b/Documentation/devicetree/bindings/opp/allwinner,sun50i-h6-operating-points.yaml
> @@ -41,7 +41,7 @@ required:
> - nvmem-cells
>
> patternProperties:
> - "opp-[0-9]+":
> + "^opp-[0-9]+$":
> type: object
>
> properties:
> @@ -49,7 +49,7 @@ patternProperties:
> clock-latency-ns: true
>
> patternProperties:
> - "opp-microvolt-.*": true
> + "^opp-microvolt-speed[0-9]$": true
>
> required:
> - opp-hz
> diff --git a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> index d60f43fd9c5a..e63e6458cea8 100644
> --- a/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> +++ b/Documentation/devicetree/bindings/pci/mediatek,mt7621-pcie.yaml
> @@ -31,7 +31,7 @@ properties:
> maxItems: 2
>
> patternProperties:
> - 'pcie@[0-2],0':
> + '^pcie@[0-2],0$':
> type: object
> $ref: /schemas/pci/pci-bus.yaml#
>
> diff --git a/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml b/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> index 0f18cceba3d5..5a0d64d3ae6b 100644
> --- a/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> +++ b/Documentation/devicetree/bindings/pci/renesas,pci-rcar-gen2.yaml
> @@ -65,7 +65,7 @@ properties:
> maxItems: 1
>
> patternProperties:
> - 'usb@[0-1],0':
> + '^usb@[0-1],0$':
> type: object
>
> description:
> diff --git a/Documentation/devicetree/bindings/regulator/max8660.yaml b/Documentation/devicetree/bindings/regulator/max8660.yaml
> index 9c038698f880..4d550ca396eb 100644
> --- a/Documentation/devicetree/bindings/regulator/max8660.yaml
> +++ b/Documentation/devicetree/bindings/regulator/max8660.yaml
> @@ -24,7 +24,7 @@ properties:
> type: object
>
> patternProperties:
> - "regulator-.+":
> + "^regulator-.+$":
> $ref: "regulator.yaml#"
>
> additionalProperties: false
> diff --git a/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml b/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> index 71138c611b6c..b704f05ea454 100644
> --- a/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> +++ b/Documentation/devicetree/bindings/regulator/maxim,max77802.yaml
> @@ -77,7 +77,7 @@ patternProperties:
> regulator-initial-mode: false
>
> patternProperties:
> - regulator-state-(standby|mem|disk):
> + "^regulator-state-(standby|mem|disk)$":
> type: object
> additionalProperties: true
> properties:
> diff --git a/Documentation/devicetree/bindings/regulator/regulator.yaml b/Documentation/devicetree/bindings/regulator/regulator.yaml
> index 6e8aa9eed3aa..53b81d8a2d41 100644
> --- a/Documentation/devicetree/bindings/regulator/regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/regulator.yaml
> @@ -231,7 +231,7 @@ patternProperties:
> ".*-supply$":
> description: Input supply phandle(s) for this node
>
> - regulator-state-(standby|mem|disk):
> + "^regulator-state-(standby|mem|disk)$":
> type: object
> description:
> sub-nodes for regulator state in Standby, Suspend-to-RAM, and
> diff --git a/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml b/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> index 7cb74cc8c5d9..54be194bb244 100644
> --- a/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> +++ b/Documentation/devicetree/bindings/regulator/rohm,bd9576-regulator.yaml
> @@ -21,7 +21,7 @@ description: |
> regulator-voutl1, regulator-vouts1
>
> patternProperties:
> - "regulator-.+":
> + "^regulator-.+$":
> type: object
> description:
> Properties for single regulator.
> diff --git a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> index 679a246dd666..7df40c38e865 100644
> --- a/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> +++ b/Documentation/devicetree/bindings/sound/renesas,rsnd.yaml
> @@ -115,7 +115,7 @@ properties:
> ports:
> $ref: /schemas/graph.yaml#/properties/ports
> patternProperties:
> - port(@[0-9a-f]+)?:
> + '^port(@[0-9a-f]+)?$':
> $ref: audio-graph-port.yaml#
> unevaluatedProperties: false
>
> diff --git a/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml b/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> index 6b733e5c1163..899100e783c9 100644
> --- a/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> +++ b/Documentation/devicetree/bindings/spi/nvidia,tegra210-quad.yaml
> @@ -48,7 +48,7 @@ properties:
> - const: tx
>
> patternProperties:
> - "@[0-9a-f]+":
> + "@[0-9a-f]+$":
> type: object
>
> properties:
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-11-22 9:29 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-18 22:37 [PATCH] dt-bindings: Add missing start and/or end of line regex anchors Rob Herring
2022-11-18 22:37 ` Rob Herring
2022-11-18 22:37 ` Rob Herring
2022-11-19 6:37 ` Sergio Paracuellos
2022-11-19 6:37 ` Sergio Paracuellos
2022-11-19 6:37 ` Sergio Paracuellos
2022-11-20 11:57 ` Mark Brown
2022-11-20 11:57 ` Mark Brown
2022-11-20 11:57 ` Mark Brown
2022-11-20 13:53 ` Guenter Roeck [this message]
2022-11-20 13:53 ` Guenter Roeck
2022-11-20 13:53 ` Guenter Roeck
2022-11-21 0:25 ` Yoshihiro Shimoda
2022-11-21 0:25 ` Yoshihiro Shimoda
2022-11-21 0:25 ` Yoshihiro Shimoda
2022-11-21 5:08 ` Viresh Kumar
2022-11-21 5:08 ` Viresh Kumar
2022-11-21 5:08 ` Viresh Kumar
2022-11-21 8:37 ` Geert Uytterhoeven
2022-11-21 8:37 ` Geert Uytterhoeven
2022-11-21 8:37 ` Geert Uytterhoeven
2022-11-21 10:16 ` Matti Vaittinen
2022-11-21 10:16 ` Matti Vaittinen
2022-11-21 10:16 ` Matti Vaittinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221120135308.GA1787641@roeck-us.net \
--to=linux@roeck-us.net \
--cc=agross@kernel.org \
--cc=alsa-devel@alsa-project.org \
--cc=andersson@kernel.org \
--cc=bhelgaas@google.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=ilia.lin@kernel.org \
--cc=javier@dowhile0.org \
--cc=jdelvare@suse.com \
--cc=jernej.skrabec@gmail.com \
--cc=jianjun.wang@mediatek.com \
--cc=jonathanh@nvidia.com \
--cc=konrad.dybcio@somainline.org \
--cc=krzk@kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pci@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-sunxi@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=marek.vasut+renesas@gmail.com \
--cc=matthias.bgg@gmail.com \
--cc=matti.vaittinen@fi.rohmeurope.com \
--cc=mripard@kernel.org \
--cc=nm@ti.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=ryder.lee@mediatek.com \
--cc=samuel@sholland.org \
--cc=sboyd@kernel.org \
--cc=sergio.paracuellos@gmail.com \
--cc=thierry.reding@gmail.com \
--cc=tiny.windzz@gmail.com \
--cc=viresh.kumar@linaro.org \
--cc=wens@csie.org \
--cc=yoshihiro.shimoda.uh@renesas.com \
--cc=zonque@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.