linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).