* [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern
@ 2023-01-20 2:04 Rob Herring
2023-01-20 17:38 ` Pratyush Yadav
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Rob Herring @ 2023-01-20 2:04 UTC (permalink / raw)
To: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra,
Krzysztof Kozlowski
Cc: linux-mtd, devicetree, linux-kernel
The 'partition' node name pattern is missing start and end anchors, so
anything is allowed before or after the regex pattern. There's no in tree
users needing that, so add anchors to the pattern.
Signed-off-by: Rob Herring <robh@kernel.org>
---
.../devicetree/bindings/mtd/partitions/partitions.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml
index 9aca4e6c6047..2edc65e0e361 100644
--- a/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml
+++ b/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml
@@ -32,7 +32,7 @@ properties:
enum: [1, 2]
patternProperties:
- "partition(-.+|@[0-9a-f]+)":
+ "^partition(-.+|@[0-9a-f]+)$":
$ref: partition.yaml
required:
--
2.39.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern
2023-01-20 2:04 [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern Rob Herring
@ 2023-01-20 17:38 ` Pratyush Yadav
2023-01-21 5:09 ` Dhruva Gole
2023-01-30 16:07 ` Miquel Raynal
2 siblings, 0 replies; 4+ messages in thread
From: Pratyush Yadav @ 2023-01-20 17:38 UTC (permalink / raw)
To: Rob Herring
Cc: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra,
Krzysztof Kozlowski, linux-mtd, devicetree, linux-kernel
On Thu, Jan 19 2023, Rob Herring wrote:
> The 'partition' node name pattern is missing start and end anchors, so
> anything is allowed before or after the regex pattern. There's no in tree
> users needing that, so add anchors to the pattern.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
Reviewed-by: Pratyush Yadav <ptyadav@amazon.de>
--
Regards,
Pratyush Yadav
Amazon Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss
Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B
Sitz: Berlin
Ust-ID: DE 289 237 879
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern
2023-01-20 2:04 [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern Rob Herring
2023-01-20 17:38 ` Pratyush Yadav
@ 2023-01-21 5:09 ` Dhruva Gole
2023-01-30 16:07 ` Miquel Raynal
2 siblings, 0 replies; 4+ messages in thread
From: Dhruva Gole @ 2023-01-21 5:09 UTC (permalink / raw)
To: Rob Herring, Miquel Raynal, Richard Weinberger,
Vignesh Raghavendra, Krzysztof Kozlowski
Cc: linux-mtd, devicetree, linux-kernel
On 20/01/23 07:34, Rob Herring wrote:
> The 'partition' node name pattern is missing start and end anchors, so
> anything is allowed before or after the regex pattern. There's no in tree
> users needing that, so add anchors to the pattern.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
Reviewed-by: Dhruva Gole <d-gole@ti.com>
> .../devicetree/bindings/mtd/partitions/partitions.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml b/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml
> index 9aca4e6c6047..2edc65e0e361 100644
> --- a/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml
> +++ b/Documentation/devicetree/bindings/mtd/partitions/partitions.yaml
> @@ -32,7 +32,7 @@ properties:
> enum: [1, 2]
>
> patternProperties:
> - "partition(-.+|@[0-9a-f]+)":
> + "^partition(-.+|@[0-9a-f]+)$":
> $ref: partition.yaml
>
> required:
--
Best regards,
Dhruva Gole
Texas Instruments Incorporated
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern
2023-01-20 2:04 [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern Rob Herring
2023-01-20 17:38 ` Pratyush Yadav
2023-01-21 5:09 ` Dhruva Gole
@ 2023-01-30 16:07 ` Miquel Raynal
2 siblings, 0 replies; 4+ messages in thread
From: Miquel Raynal @ 2023-01-30 16:07 UTC (permalink / raw)
To: Rob Herring
Cc: Richard Weinberger, Vignesh Raghavendra, Krzysztof Kozlowski,
linux-mtd, devicetree, linux-kernel
Hi Rob,
robh@kernel.org wrote on Thu, 19 Jan 2023 20:04:53 -0600:
> The 'partition' node name pattern is missing start and end anchors, so
> anything is allowed before or after the regex pattern. There's no in tree
> users needing that, so add anchors to the pattern.
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
Looks like my scripts did not properly sent the "applied" e-mail, so
here it is, I applied this patch to mtd/next.
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-01-30 16:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-20 2:04 [PATCH] dt-bindings: mtd: partitions: Fix partition node name pattern Rob Herring
2023-01-20 17:38 ` Pratyush Yadav
2023-01-21 5:09 ` Dhruva Gole
2023-01-30 16:07 ` Miquel Raynal
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).