linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed
@ 2022-07-26 11:58 Krzysztof Kozlowski
  2022-07-26 11:58 ` [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema Krzysztof Kozlowski
                   ` (3 more replies)
  0 siblings, 4 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-26 11:58 UTC (permalink / raw)
  To: Arnd Bergmann, Olof Johansson, arm, soc, Rob Herring,
	Krzysztof Kozlowski, Florian Fainelli, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel
  Cc: Krzysztof Kozlowski

Id and schema fields do not need quotes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml          | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml
index 894786640938..a6e8018084c6 100644
--- a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml
+++ b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/bcm/brcm,bcm2835-pm.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/bcm/brcm,bcm2835-pm.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: BCM2835 PM (Power domains, watchdog)
 
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema
  2022-07-26 11:58 [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Krzysztof Kozlowski
@ 2022-07-26 11:58 ` Krzysztof Kozlowski
  2022-07-26 15:59   ` Florian Fainelli
  2022-07-26 11:58 ` [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed Krzysztof Kozlowski
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 13+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-26 11:58 UTC (permalink / raw)
  To: Arnd Bergmann, Olof Johansson, arm, soc, Rob Herring,
	Krzysztof Kozlowski, Florian Fainelli, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel
  Cc: Krzysztof Kozlowski

Absolute path to other DT schema is preferred over relative one.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml
index a6e8018084c6..e28ef198a801 100644
--- a/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml
+++ b/Documentation/devicetree/bindings/soc/bcm/brcm,bcm2835-pm.yaml
@@ -14,7 +14,7 @@ maintainers:
   - Nicolas Saenz Julienne <nsaenz@kernel.org>
 
 allOf:
-  - $ref: ../../watchdog/watchdog.yaml#
+  - $ref: /schemas/watchdog/watchdog.yaml#
 
 properties:
   compatible:
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed
  2022-07-26 11:58 [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Krzysztof Kozlowski
  2022-07-26 11:58 ` [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema Krzysztof Kozlowski
@ 2022-07-26 11:58 ` Krzysztof Kozlowski
  2022-07-26 13:52   ` Conor.Dooley
                     ` (2 more replies)
  2022-07-26 11:58 ` [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema Krzysztof Kozlowski
  2022-07-26 15:58 ` [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Florian Fainelli
  3 siblings, 3 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-26 11:58 UTC (permalink / raw)
  To: Arnd Bergmann, Olof Johansson, arm, soc, Rob Herring,
	Krzysztof Kozlowski, Florian Fainelli, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel
  Cc: Krzysztof Kozlowski

Id and schema fields do not need quotes.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml          | 4 ++--
 .../bindings/soc/microchip/microchip,mpfs-sys-controller.yaml | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
index 597d67fba92f..2f8cf6191812 100644
--- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
+++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Atmel Timer Counter Block
 
diff --git a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
index b0dae51e1d42..04ffee3a7c59 100644
--- a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
+++ b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
@@ -1,8 +1,8 @@
 # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
 %YAML 1.2
 ---
-$id: "http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
 
 title: Microchip PolarFire SoC (MPFS) MSS (microprocessor subsystem) system controller
 
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema
  2022-07-26 11:58 [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Krzysztof Kozlowski
  2022-07-26 11:58 ` [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema Krzysztof Kozlowski
  2022-07-26 11:58 ` [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed Krzysztof Kozlowski
@ 2022-07-26 11:58 ` Krzysztof Kozlowski
  2022-07-26 14:19   ` Claudiu.Beznea
  2022-07-27  7:54   ` Claudiu.Beznea
  2022-07-26 15:58 ` [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Florian Fainelli
  3 siblings, 2 replies; 13+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-26 11:58 UTC (permalink / raw)
  To: Arnd Bergmann, Olof Johansson, arm, soc, Rob Herring,
	Krzysztof Kozlowski, Florian Fainelli, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel
  Cc: Krzysztof Kozlowski

Absolute path to other DT schema is preferred over relative one.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
index 2f8cf6191812..33748a061898 100644
--- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
+++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
@@ -75,7 +75,7 @@ patternProperties:
 
   "^pwm@[0-2]$":
     description: The timer block channels that are used as PWMs.
-    $ref: ../../pwm/pwm.yaml#
+    $ref: /schemas/pwm/pwm.yaml#
     type: object
     properties:
       compatible:
-- 
2.34.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed
  2022-07-26 11:58 ` [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed Krzysztof Kozlowski
@ 2022-07-26 13:52   ` Conor.Dooley
  2022-07-26 14:13   ` Claudiu.Beznea
  2022-07-27  7:54   ` Claudiu.Beznea
  2 siblings, 0 replies; 13+ messages in thread
From: Conor.Dooley @ 2022-07-26 13:52 UTC (permalink / raw)
  To: krzysztof.kozlowski, arnd, olof, arm, soc, robh+dt,
	krzysztof.kozlowski+dt, f.fainelli, rjui, sbranden,
	bcm-kernel-feedback-list, Nicolas.Ferre, alexandre.belloni,
	Claudiu.Beznea, nsaenz, devicetree, linux-rpi-kernel,
	linux-arm-kernel, linux-kernel

Whole series LGTM.

Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

Thanks Krzysztof,
Conor.

On 26/07/2022 12:58, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Id and schema fields do not need quotes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml          | 4 ++--
>  .../bindings/soc/microchip/microchip,mpfs-sys-controller.yaml | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 597d67fba92f..2f8cf6191812 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> 
>  title: Atmel Timer Counter Block
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> index b0dae51e1d42..04ffee3a7c59 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> 
>  title: Microchip PolarFire SoC (MPFS) MSS (microprocessor subsystem) system controller
> 
> --
> 2.34.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed
  2022-07-26 11:58 ` [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed Krzysztof Kozlowski
  2022-07-26 13:52   ` Conor.Dooley
@ 2022-07-26 14:13   ` Claudiu.Beznea
  2022-07-27  7:54   ` Claudiu.Beznea
  2 siblings, 0 replies; 13+ messages in thread
From: Claudiu.Beznea @ 2022-07-26 14:13 UTC (permalink / raw)
  To: krzysztof.kozlowski, arnd, olof, arm, soc, robh+dt,
	krzysztof.kozlowski+dt, f.fainelli, rjui, sbranden,
	bcm-kernel-feedback-list, Nicolas.Ferre, alexandre.belloni,
	nsaenz, Conor.Dooley, devicetree, linux-rpi-kernel,
	linux-arm-kernel, linux-kernel

On 26.07.2022 14:58, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Id and schema fields do not need quotes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>


> ---
>  .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml          | 4 ++--
>  .../bindings/soc/microchip/microchip,mpfs-sys-controller.yaml | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 597d67fba92f..2f8cf6191812 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> 
>  title: Atmel Timer Counter Block
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> index b0dae51e1d42..04ffee3a7c59 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> 
>  title: Microchip PolarFire SoC (MPFS) MSS (microprocessor subsystem) system controller
> 
> --
> 2.34.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema
  2022-07-26 11:58 ` [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema Krzysztof Kozlowski
@ 2022-07-26 14:19   ` Claudiu.Beznea
  2022-07-27  7:54   ` Claudiu.Beznea
  1 sibling, 0 replies; 13+ messages in thread
From: Claudiu.Beznea @ 2022-07-26 14:19 UTC (permalink / raw)
  To: krzysztof.kozlowski, arnd, olof, arm, soc, robh+dt,
	krzysztof.kozlowski+dt, f.fainelli, rjui, sbranden,
	bcm-kernel-feedback-list, Nicolas.Ferre, alexandre.belloni,
	nsaenz, Conor.Dooley, devicetree, linux-rpi-kernel,
	linux-arm-kernel, linux-kernel

On 26.07.2022 14:58, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Absolute path to other DT schema is preferred over relative one.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com>


> ---
>  .../devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 2f8cf6191812..33748a061898 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -75,7 +75,7 @@ patternProperties:
> 
>    "^pwm@[0-2]$":
>      description: The timer block channels that are used as PWMs.
> -    $ref: ../../pwm/pwm.yaml#
> +    $ref: /schemas/pwm/pwm.yaml#
>      type: object
>      properties:
>        compatible:
> --
> 2.34.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed
  2022-07-26 11:58 [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Krzysztof Kozlowski
                   ` (2 preceding siblings ...)
  2022-07-26 11:58 ` [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema Krzysztof Kozlowski
@ 2022-07-26 15:58 ` Florian Fainelli
  3 siblings, 0 replies; 13+ messages in thread
From: Florian Fainelli @ 2022-07-26 15:58 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Arnd Bergmann, Olof Johansson, arm, soc,
	Rob Herring, Krzysztof Kozlowski, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel

On 7/26/22 04:58, Krzysztof Kozlowski wrote:
> Id and schema fields do not need quotes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema
  2022-07-26 11:58 ` [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema Krzysztof Kozlowski
@ 2022-07-26 15:59   ` Florian Fainelli
  2022-07-26 17:31     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 13+ messages in thread
From: Florian Fainelli @ 2022-07-26 15:59 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Arnd Bergmann, Olof Johansson, arm, soc,
	Rob Herring, Krzysztof Kozlowski, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel

On 7/26/22 04:58, Krzysztof Kozlowski wrote:
> Absolute path to other DT schema is preferred over relative one.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Acked-by: Florian Fainelli <f.fainelli@gmail.com>

Are you applying those patches or do you want the SoC maintainer to do that?
-- 
Florian

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema
  2022-07-26 15:59   ` Florian Fainelli
@ 2022-07-26 17:31     ` Krzysztof Kozlowski
  2022-08-04 18:11       ` Florian Fainelli
  0 siblings, 1 reply; 13+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-26 17:31 UTC (permalink / raw)
  To: Florian Fainelli, Arnd Bergmann, Olof Johansson, arm, soc,
	Rob Herring, Krzysztof Kozlowski, Ray Jui, Scott Branden,
	Broadcom internal kernel review list, Nicolas Ferre,
	Alexandre Belloni, Claudiu Beznea, Saenz Julienne, Conor Dooley,
	devicetree, linux-rpi-kernel, linux-arm-kernel, linux-kernel

On 26/07/2022 17:59, Florian Fainelli wrote:
> On 7/26/22 04:58, Krzysztof Kozlowski wrote:
>> Absolute path to other DT schema is preferred over relative one.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> 
> Are you applying those patches or do you want the SoC maintainer to do that?

The best if  you (SoC maintainer) would pick them up. If you cannot or
it is too late for this, maybe arm-soc could grab them (already Cced).

Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed
  2022-07-26 11:58 ` [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed Krzysztof Kozlowski
  2022-07-26 13:52   ` Conor.Dooley
  2022-07-26 14:13   ` Claudiu.Beznea
@ 2022-07-27  7:54   ` Claudiu.Beznea
  2 siblings, 0 replies; 13+ messages in thread
From: Claudiu.Beznea @ 2022-07-27  7:54 UTC (permalink / raw)
  To: krzysztof.kozlowski, arnd, olof, arm, soc, robh+dt,
	krzysztof.kozlowski+dt, f.fainelli, rjui, sbranden,
	bcm-kernel-feedback-list, Nicolas.Ferre, alexandre.belloni,
	nsaenz, Conor.Dooley, devicetree, linux-rpi-kernel,
	linux-arm-kernel, linux-kernel

On 26.07.2022 14:58, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Id and schema fields do not need quotes.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Applied to at91-dt, thanks!

> ---
>  .../bindings/soc/microchip/atmel,at91rm9200-tcb.yaml          | 4 ++--
>  .../bindings/soc/microchip/microchip,mpfs-sys-controller.yaml | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 597d67fba92f..2f8cf6191812 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/microchip/atmel,at91rm9200-tcb.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> 
>  title: Atmel Timer Counter Block
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> index b0dae51e1d42..04ffee3a7c59 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/microchip,mpfs-sys-controller.yaml
> @@ -1,8 +1,8 @@
>  # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>  %YAML 1.2
>  ---
> -$id: "http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/soc/microchip/microchip,mpfs-sys-controller.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> 
>  title: Microchip PolarFire SoC (MPFS) MSS (microprocessor subsystem) system controller
> 
> --
> 2.34.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema
  2022-07-26 11:58 ` [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema Krzysztof Kozlowski
  2022-07-26 14:19   ` Claudiu.Beznea
@ 2022-07-27  7:54   ` Claudiu.Beznea
  1 sibling, 0 replies; 13+ messages in thread
From: Claudiu.Beznea @ 2022-07-27  7:54 UTC (permalink / raw)
  To: krzysztof.kozlowski, arnd, olof, arm, soc, robh+dt,
	krzysztof.kozlowski+dt, f.fainelli, rjui, sbranden,
	bcm-kernel-feedback-list, Nicolas.Ferre, alexandre.belloni,
	nsaenz, Conor.Dooley, devicetree, linux-rpi-kernel,
	linux-arm-kernel, linux-kernel

On 26.07.2022 14:58, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Absolute path to other DT schema is preferred over relative one.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Applied to at91-dt, thanks!

> ---
>  .../devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> index 2f8cf6191812..33748a061898 100644
> --- a/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> +++ b/Documentation/devicetree/bindings/soc/microchip/atmel,at91rm9200-tcb.yaml
> @@ -75,7 +75,7 @@ patternProperties:
> 
>    "^pwm@[0-2]$":
>      description: The timer block channels that are used as PWMs.
> -    $ref: ../../pwm/pwm.yaml#
> +    $ref: /schemas/pwm/pwm.yaml#
>      type: object
>      properties:
>        compatible:
> --
> 2.34.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema
  2022-07-26 17:31     ` Krzysztof Kozlowski
@ 2022-08-04 18:11       ` Florian Fainelli
  0 siblings, 0 replies; 13+ messages in thread
From: Florian Fainelli @ 2022-08-04 18:11 UTC (permalink / raw)
  To: Krzysztof Kozlowski, Florian Fainelli, Arnd Bergmann,
	Olof Johansson, arm, soc, Rob Herring, Krzysztof Kozlowski,
	Ray Jui, Scott Branden, Broadcom internal kernel review list,
	Nicolas Ferre, Alexandre Belloni, Claudiu Beznea, Saenz Julienne,
	Conor Dooley, devicetree, linux-rpi-kernel, linux-arm-kernel,
	linux-kernel

On 7/26/22 10:31, Krzysztof Kozlowski wrote:
> On 26/07/2022 17:59, Florian Fainelli wrote:
>> On 7/26/22 04:58, Krzysztof Kozlowski wrote:
>>> Absolute path to other DT schema is preferred over relative one.
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>
>> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
>>
>> Are you applying those patches or do you want the SoC maintainer to do that?
> 
> The best if  you (SoC maintainer) would pick them up. If you cannot or
> it is too late for this, maybe arm-soc could grab them (already Cced).

Arnd ended up picking up patches 1 and 2, FWIW.
-- 
Florian

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2022-08-04 18:14 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-26 11:58 [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Krzysztof Kozlowski
2022-07-26 11:58 ` [PATCH 2/4] dt-bindings: soc: bcm: use absolute path to other schema Krzysztof Kozlowski
2022-07-26 15:59   ` Florian Fainelli
2022-07-26 17:31     ` Krzysztof Kozlowski
2022-08-04 18:11       ` Florian Fainelli
2022-07-26 11:58 ` [PATCH 3/4] dt-bindings: soc: microchip: drop quotes when not needed Krzysztof Kozlowski
2022-07-26 13:52   ` Conor.Dooley
2022-07-26 14:13   ` Claudiu.Beznea
2022-07-27  7:54   ` Claudiu.Beznea
2022-07-26 11:58 ` [PATCH 4/4] dt-bindings: soc: microchip: use absolute path to other schema Krzysztof Kozlowski
2022-07-26 14:19   ` Claudiu.Beznea
2022-07-27  7:54   ` Claudiu.Beznea
2022-07-26 15:58 ` [PATCH 1/4] dt-bindings: soc: bcm: drop quotes when not needed Florian Fainelli

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