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:
next prev parent reply other threads:[~2022-11-20 13:53 UTC|newest]
Thread overview: 8+ 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-19 6:37 ` Sergio Paracuellos
2022-11-20 11:57 ` Mark Brown
2022-11-20 13:53 ` Guenter Roeck [this message]
2022-11-21 0:25 ` Yoshihiro Shimoda
2022-11-21 5:08 ` Viresh Kumar
2022-11-21 8:37 ` Geert Uytterhoeven
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 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).