* [PATCH] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
@ 2022-01-26 13:21 ` Rafał Miłecki
0 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-01-26 13:21 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen
Cc: linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
From: Rafał Miłecki <rafal@milecki.pl>
This hardware block is used on almost all BCM63xx family chipsets and
BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
strings and also include a generic one.
The only SoC with a different block I found is BCM6838 (thus not included
in this change).
It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
include "SoftRst" register but that can be handled by drivers based on
precise compatible string.
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
.../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
1 file changed, 17 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
index a926809352b8..b663f360f571 100644
--- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
@@ -16,9 +16,22 @@ maintainers:
properties:
compatible:
- enum:
- - brcm,bcm6345-wdt
- - brcm,bcm7038-wdt
+ items:
+ - enum:
+ - brcm,bcm4908-wdt
+ - brcm,bcm60333-wdt
+ - brcm,bcm63138-wdt
+ - brcm,bcm63148-wdt
+ - brcm,bcm63268-wdt
+ - brcm,bcm63381-wdt
+ - brcm,bcm6848-wdt
+ - brcm,bcm6858-wdt
+ - brcm,bcm68360-wdt
+ - brcm,bcm6338-wdt
+ - brcm,bcm6345-wdt
+ - brcm,bcm6348-wdt
+ - brcm,bcm7038-wdt
+ - const: brcm,bcm63xx-wdt
reg:
maxItems: 1
@@ -37,7 +50,7 @@ required:
examples:
- |
watchdog@f040a7e8 {
- compatible = "brcm,bcm7038-wdt";
+ compatible = "brcm,bcm7038-wdt", "brcm,bcm63xx-wdt";
reg = <0xf040a7e8 0x16>;
clocks = <&upg_fixed>;
};
--
2.31.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] dt-bindings: watchdog: brcm, bcm7038: add more compatible strings
@ 2022-01-26 13:21 ` Rafał Miłecki
0 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-01-26 13:21 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen
Cc: linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
From: Rafał Miłecki <rafal@milecki.pl>
This hardware block is used on almost all BCM63xx family chipsets and
BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
strings and also include a generic one.
The only SoC with a different block I found is BCM6838 (thus not included
in this change).
It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
include "SoftRst" register but that can be handled by drivers based on
precise compatible string.
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
.../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
1 file changed, 17 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
index a926809352b8..b663f360f571 100644
--- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
@@ -16,9 +16,22 @@ maintainers:
properties:
compatible:
- enum:
- - brcm,bcm6345-wdt
- - brcm,bcm7038-wdt
+ items:
+ - enum:
+ - brcm,bcm4908-wdt
+ - brcm,bcm60333-wdt
+ - brcm,bcm63138-wdt
+ - brcm,bcm63148-wdt
+ - brcm,bcm63268-wdt
+ - brcm,bcm63381-wdt
+ - brcm,bcm6848-wdt
+ - brcm,bcm6858-wdt
+ - brcm,bcm68360-wdt
+ - brcm,bcm6338-wdt
+ - brcm,bcm6345-wdt
+ - brcm,bcm6348-wdt
+ - brcm,bcm7038-wdt
+ - const: brcm,bcm63xx-wdt
reg:
maxItems: 1
@@ -37,7 +50,7 @@ required:
examples:
- |
watchdog@f040a7e8 {
- compatible = "brcm,bcm7038-wdt";
+ compatible = "brcm,bcm7038-wdt", "brcm,bcm63xx-wdt";
reg = <0xf040a7e8 0x16>;
clocks = <&upg_fixed>;
};
--
2.31.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] 14+ messages in thread
* Re: [PATCH] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
2022-01-26 13:21 ` [PATCH] dt-bindings: watchdog: brcm, bcm7038: " Rafał Miłecki
@ 2022-01-26 22:10 ` Rob Herring
-1 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2022-01-26 22:10 UTC (permalink / raw)
To: Rafał Miłecki
Cc: linux-arm-kernel, devicetree, Florian Fainelli, Rob Herring,
bcm-kernel-feedback-list, Rafał Miłecki,
Wim Van Sebroeck, Guenter Roeck, Justin Chen, linux-watchdog
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1904 bytes --]
On Wed, 26 Jan 2022 14:21:16 +0100, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> This hardware block is used on almost all BCM63xx family chipsets and
> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
> strings and also include a generic one.
>
> The only SoC with a different block I found is BCM6838 (thus not included
> in this change).
>
> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
> include "SoftRst" register but that can be handled by drivers based on
> precise compatible string.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
> 1 file changed, 17 insertions(+), 4 deletions(-)
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mfd/brcm,twd.example.dt.yaml: timer-mfd@ff800400: watchdog@28:compatible: ['brcm,bcm7038-wdt'] is too short
From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mfd/brcm,twd.example.dt.yaml: watchdog@28: compatible: ['brcm,bcm7038-wdt'] is too short
From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/patch/1584464
This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] dt-bindings: watchdog: brcm, bcm7038: add more compatible strings
@ 2022-01-26 22:10 ` Rob Herring
0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2022-01-26 22:10 UTC (permalink / raw)
To: Rafał Miłecki
Cc: linux-arm-kernel, devicetree, Florian Fainelli, Rob Herring,
bcm-kernel-feedback-list, Rafał Miłecki,
Wim Van Sebroeck, Guenter Roeck, Justin Chen, linux-watchdog
[-- Attachment #1: Type: text/plain, Size: 1905 bytes --]
On Wed, 26 Jan 2022 14:21:16 +0100, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> This hardware block is used on almost all BCM63xx family chipsets and
> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
> strings and also include a generic one.
>
> The only SoC with a different block I found is BCM6838 (thus not included
> in this change).
>
> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
> include "SoftRst" register but that can be handled by drivers based on
> precise compatible string.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
> 1 file changed, 17 insertions(+), 4 deletions(-)
>
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mfd/brcm,twd.example.dt.yaml: timer-mfd@ff800400: watchdog@28:compatible: ['brcm,bcm7038-wdt'] is too short
From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/mfd/brcm,twd.example.dt.yaml: watchdog@28: compatible: ['brcm,bcm7038-wdt'] is too short
From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
doc reference errors (make refcheckdocs):
See https://patchwork.ozlabs.org/patch/1584464
This check can fail if there are any dependencies. The base for a patch
series is generally the most recent rc1.
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure 'yamllint' is installed and dt-schema is up to
date:
pip3 install dtschema --upgrade
Please check and re-submit.
[-- Attachment #2: Type: text/plain, Size: 176 bytes --]
_______________________________________________
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] 14+ messages in thread
* [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
2022-01-26 13:21 ` [PATCH] dt-bindings: watchdog: brcm, bcm7038: " Rafał Miłecki
@ 2022-01-26 22:20 ` Rafał Miłecki
-1 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-01-26 22:20 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen
Cc: linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
From: Rafał Miłecki <rafal@milecki.pl>
This hardware block is used on almost all BCM63xx family chipsets and
BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
strings and also include a generic one.
The only SoC with a different block I found is BCM6838 (thus not included
in this change).
It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
include "SoftRst" register but that can be handled by drivers based on
precise compatible string.
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
V2: Sort enum entries & update brcm,twd.yaml
---
.../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
.../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
2 files changed, 18 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
index 634526f790b8..3f5db1990aba 100644
--- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
+++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
@@ -55,7 +55,7 @@ examples:
#size-cells = <1>;
watchdog@28 {
- compatible = "brcm,bcm7038-wdt";
+ compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
reg = <0x28 0x8>;
};
};
diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
index a926809352b8..4d848442913c 100644
--- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
@@ -16,9 +16,22 @@ maintainers:
properties:
compatible:
- enum:
- - brcm,bcm6345-wdt
- - brcm,bcm7038-wdt
+ items:
+ - enum:
+ - brcm,bcm4908-wdt
+ - brcm,bcm6338-wdt
+ - brcm,bcm6345-wdt
+ - brcm,bcm6348-wdt
+ - brcm,bcm6848-wdt
+ - brcm,bcm6858-wdt
+ - brcm,bcm7038-wdt
+ - brcm,bcm60333-wdt
+ - brcm,bcm63138-wdt
+ - brcm,bcm63148-wdt
+ - brcm,bcm63268-wdt
+ - brcm,bcm63381-wdt
+ - brcm,bcm68360-wdt
+ - const: brcm,bcm63xx-wdt
reg:
maxItems: 1
@@ -37,7 +50,7 @@ required:
examples:
- |
watchdog@f040a7e8 {
- compatible = "brcm,bcm7038-wdt";
+ compatible = "brcm,bcm7038-wdt", "brcm,bcm63xx-wdt";
reg = <0xf040a7e8 0x16>;
clocks = <&upg_fixed>;
};
--
2.31.1
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH V2] dt-bindings: watchdog: brcm, bcm7038: add more compatible strings
@ 2022-01-26 22:20 ` Rafał Miłecki
0 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-01-26 22:20 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Florian Fainelli,
Justin Chen
Cc: linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
From: Rafał Miłecki <rafal@milecki.pl>
This hardware block is used on almost all BCM63xx family chipsets and
BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
strings and also include a generic one.
The only SoC with a different block I found is BCM6838 (thus not included
in this change).
It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
include "SoftRst" register but that can be handled by drivers based on
precise compatible string.
Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
V2: Sort enum entries & update brcm,twd.yaml
---
.../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
.../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
2 files changed, 18 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
index 634526f790b8..3f5db1990aba 100644
--- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
+++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
@@ -55,7 +55,7 @@ examples:
#size-cells = <1>;
watchdog@28 {
- compatible = "brcm,bcm7038-wdt";
+ compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
reg = <0x28 0x8>;
};
};
diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
index a926809352b8..4d848442913c 100644
--- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
@@ -16,9 +16,22 @@ maintainers:
properties:
compatible:
- enum:
- - brcm,bcm6345-wdt
- - brcm,bcm7038-wdt
+ items:
+ - enum:
+ - brcm,bcm4908-wdt
+ - brcm,bcm6338-wdt
+ - brcm,bcm6345-wdt
+ - brcm,bcm6348-wdt
+ - brcm,bcm6848-wdt
+ - brcm,bcm6858-wdt
+ - brcm,bcm7038-wdt
+ - brcm,bcm60333-wdt
+ - brcm,bcm63138-wdt
+ - brcm,bcm63148-wdt
+ - brcm,bcm63268-wdt
+ - brcm,bcm63381-wdt
+ - brcm,bcm68360-wdt
+ - const: brcm,bcm63xx-wdt
reg:
maxItems: 1
@@ -37,7 +50,7 @@ required:
examples:
- |
watchdog@f040a7e8 {
- compatible = "brcm,bcm7038-wdt";
+ compatible = "brcm,bcm7038-wdt", "brcm,bcm63xx-wdt";
reg = <0xf040a7e8 0x16>;
clocks = <&upg_fixed>;
};
--
2.31.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] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
2022-01-26 22:20 ` [PATCH V2] dt-bindings: watchdog: brcm, bcm7038: " Rafał Miłecki
@ 2022-02-09 19:09 ` Rob Herring
-1 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2022-02-09 19:09 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> This hardware block is used on almost all BCM63xx family chipsets and
> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
> strings and also include a generic one.
>
> The only SoC with a different block I found is BCM6838 (thus not included
> in this change).
>
> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
> include "SoftRst" register but that can be handled by drivers based on
> precise compatible string.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> V2: Sort enum entries & update brcm,twd.yaml
> ---
> .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
> 2 files changed, 18 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> index 634526f790b8..3f5db1990aba 100644
> --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> @@ -55,7 +55,7 @@ examples:
> #size-cells = <1>;
>
> watchdog@28 {
> - compatible = "brcm,bcm7038-wdt";
> + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
> reg = <0x28 0x8>;
> };
> };
> diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> index a926809352b8..4d848442913c 100644
> --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> @@ -16,9 +16,22 @@ maintainers:
>
> properties:
> compatible:
> - enum:
> - - brcm,bcm6345-wdt
> - - brcm,bcm7038-wdt
> + items:
> + - enum:
> + - brcm,bcm4908-wdt
> + - brcm,bcm6338-wdt
> + - brcm,bcm6345-wdt
> + - brcm,bcm6348-wdt
> + - brcm,bcm6848-wdt
> + - brcm,bcm6858-wdt
> + - brcm,bcm7038-wdt
> + - brcm,bcm60333-wdt
> + - brcm,bcm63138-wdt
> + - brcm,bcm63148-wdt
> + - brcm,bcm63268-wdt
> + - brcm,bcm63381-wdt
> + - brcm,bcm68360-wdt
> + - const: brcm,bcm63xx-wdt
Is it really worthwhile to update all these DTs?:
arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
I don't think so.
Rob
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
@ 2022-02-09 19:09 ` Rob Herring
0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2022-02-09 19:09 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> This hardware block is used on almost all BCM63xx family chipsets and
> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
> strings and also include a generic one.
>
> The only SoC with a different block I found is BCM6838 (thus not included
> in this change).
>
> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
> include "SoftRst" register but that can be handled by drivers based on
> precise compatible string.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> V2: Sort enum entries & update brcm,twd.yaml
> ---
> .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
> 2 files changed, 18 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> index 634526f790b8..3f5db1990aba 100644
> --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> @@ -55,7 +55,7 @@ examples:
> #size-cells = <1>;
>
> watchdog@28 {
> - compatible = "brcm,bcm7038-wdt";
> + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
> reg = <0x28 0x8>;
> };
> };
> diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> index a926809352b8..4d848442913c 100644
> --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> @@ -16,9 +16,22 @@ maintainers:
>
> properties:
> compatible:
> - enum:
> - - brcm,bcm6345-wdt
> - - brcm,bcm7038-wdt
> + items:
> + - enum:
> + - brcm,bcm4908-wdt
> + - brcm,bcm6338-wdt
> + - brcm,bcm6345-wdt
> + - brcm,bcm6348-wdt
> + - brcm,bcm6848-wdt
> + - brcm,bcm6858-wdt
> + - brcm,bcm7038-wdt
> + - brcm,bcm60333-wdt
> + - brcm,bcm63138-wdt
> + - brcm,bcm63148-wdt
> + - brcm,bcm63268-wdt
> + - brcm,bcm63381-wdt
> + - brcm,bcm68360-wdt
> + - const: brcm,bcm63xx-wdt
Is it really worthwhile to update all these DTs?:
arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
I don't think so.
Rob
_______________________________________________
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] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
2022-02-09 19:09 ` Rob Herring
@ 2022-02-09 19:26 ` Rafał Miłecki
-1 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-02-09 19:26 UTC (permalink / raw)
To: Rob Herring
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On 9.02.2022 20:09, Rob Herring wrote:
> On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
>> From: Rafał Miłecki <rafal@milecki.pl>
>>
>> This hardware block is used on almost all BCM63xx family chipsets and
>> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
>> strings and also include a generic one.
>>
>> The only SoC with a different block I found is BCM6838 (thus not included
>> in this change).
>>
>> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
>> include "SoftRst" register but that can be handled by drivers based on
>> precise compatible string.
>>
>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>> ---
>> V2: Sort enum entries & update brcm,twd.yaml
>> ---
>> .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
>> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
>> 2 files changed, 18 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>> index 634526f790b8..3f5db1990aba 100644
>> --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>> +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>> @@ -55,7 +55,7 @@ examples:
>> #size-cells = <1>;
>>
>> watchdog@28 {
>> - compatible = "brcm,bcm7038-wdt";
>> + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
>> reg = <0x28 0x8>;
>> };
>> };
>> diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>> index a926809352b8..4d848442913c 100644
>> --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>> +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>> @@ -16,9 +16,22 @@ maintainers:
>>
>> properties:
>> compatible:
>> - enum:
>> - - brcm,bcm6345-wdt
>> - - brcm,bcm7038-wdt
>> + items:
>> + - enum:
>> + - brcm,bcm4908-wdt
>> + - brcm,bcm6338-wdt
>> + - brcm,bcm6345-wdt
>> + - brcm,bcm6348-wdt
>> + - brcm,bcm6848-wdt
>> + - brcm,bcm6858-wdt
>> + - brcm,bcm7038-wdt
>> + - brcm,bcm60333-wdt
>> + - brcm,bcm63138-wdt
>> + - brcm,bcm63148-wdt
>> + - brcm,bcm63268-wdt
>> + - brcm,bcm63381-wdt
>> + - brcm,bcm68360-wdt
>> + - const: brcm,bcm63xx-wdt
>
> Is it really worthwhile to update all these DTs?:
>
> arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
I don't have problem handling that.
> I don't think so.
So what's the policy for such bindings then? How to select SoCs that
should have their own bindings? How can I tell which should /borrow/ a
binding instead?
I assumed we want binding per SoC based on the
17fffe91ba36 ("dt-bindings: watchdog: Add BCM6345 compatible to BCM7038 binding")
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=17fffe91ba36d11c7b17be154ecc7c1ed31527eb
_______________________________________________
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] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
@ 2022-02-09 19:26 ` Rafał Miłecki
0 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-02-09 19:26 UTC (permalink / raw)
To: Rob Herring
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On 9.02.2022 20:09, Rob Herring wrote:
> On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
>> From: Rafał Miłecki <rafal@milecki.pl>
>>
>> This hardware block is used on almost all BCM63xx family chipsets and
>> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
>> strings and also include a generic one.
>>
>> The only SoC with a different block I found is BCM6838 (thus not included
>> in this change).
>>
>> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
>> include "SoftRst" register but that can be handled by drivers based on
>> precise compatible string.
>>
>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>> ---
>> V2: Sort enum entries & update brcm,twd.yaml
>> ---
>> .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
>> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
>> 2 files changed, 18 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>> index 634526f790b8..3f5db1990aba 100644
>> --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>> +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>> @@ -55,7 +55,7 @@ examples:
>> #size-cells = <1>;
>>
>> watchdog@28 {
>> - compatible = "brcm,bcm7038-wdt";
>> + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
>> reg = <0x28 0x8>;
>> };
>> };
>> diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>> index a926809352b8..4d848442913c 100644
>> --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>> +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>> @@ -16,9 +16,22 @@ maintainers:
>>
>> properties:
>> compatible:
>> - enum:
>> - - brcm,bcm6345-wdt
>> - - brcm,bcm7038-wdt
>> + items:
>> + - enum:
>> + - brcm,bcm4908-wdt
>> + - brcm,bcm6338-wdt
>> + - brcm,bcm6345-wdt
>> + - brcm,bcm6348-wdt
>> + - brcm,bcm6848-wdt
>> + - brcm,bcm6858-wdt
>> + - brcm,bcm7038-wdt
>> + - brcm,bcm60333-wdt
>> + - brcm,bcm63138-wdt
>> + - brcm,bcm63148-wdt
>> + - brcm,bcm63268-wdt
>> + - brcm,bcm63381-wdt
>> + - brcm,bcm68360-wdt
>> + - const: brcm,bcm63xx-wdt
>
> Is it really worthwhile to update all these DTs?:
>
> arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
> arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
I don't have problem handling that.
> I don't think so.
So what's the policy for such bindings then? How to select SoCs that
should have their own bindings? How can I tell which should /borrow/ a
binding instead?
I assumed we want binding per SoC based on the
17fffe91ba36 ("dt-bindings: watchdog: Add BCM6345 compatible to BCM7038 binding")
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=17fffe91ba36d11c7b17be154ecc7c1ed31527eb
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
2022-02-09 19:26 ` Rafał Miłecki
@ 2022-02-09 20:02 ` Rob Herring
-1 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2022-02-09 20:02 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On Wed, Feb 09, 2022 at 08:26:00PM +0100, Rafał Miłecki wrote:
> On 9.02.2022 20:09, Rob Herring wrote:
> > On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
> > > From: Rafał Miłecki <rafal@milecki.pl>
> > >
> > > This hardware block is used on almost all BCM63xx family chipsets and
> > > BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
> > > strings and also include a generic one.
> > >
> > > The only SoC with a different block I found is BCM6838 (thus not included
> > > in this change).
> > >
> > > It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
> > > include "SoftRst" register but that can be handled by drivers based on
> > > precise compatible string.
> > >
> > > Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> > > ---
> > > V2: Sort enum entries & update brcm,twd.yaml
> > > ---
> > > .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
> > > .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
> > > 2 files changed, 18 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> > > index 634526f790b8..3f5db1990aba 100644
> > > --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> > > +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> > > @@ -55,7 +55,7 @@ examples:
> > > #size-cells = <1>;
> > > watchdog@28 {
> > > - compatible = "brcm,bcm7038-wdt";
> > > + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
> > > reg = <0x28 0x8>;
> > > };
> > > };
> > > diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> > > index a926809352b8..4d848442913c 100644
> > > --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> > > +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> > > @@ -16,9 +16,22 @@ maintainers:
> > > properties:
> > > compatible:
> > > - enum:
> > > - - brcm,bcm6345-wdt
> > > - - brcm,bcm7038-wdt
> > > + items:
> > > + - enum:
> > > + - brcm,bcm4908-wdt
> > > + - brcm,bcm6338-wdt
> > > + - brcm,bcm6345-wdt
> > > + - brcm,bcm6348-wdt
> > > + - brcm,bcm6848-wdt
> > > + - brcm,bcm6858-wdt
> > > + - brcm,bcm7038-wdt
> > > + - brcm,bcm60333-wdt
> > > + - brcm,bcm63138-wdt
> > > + - brcm,bcm63148-wdt
> > > + - brcm,bcm63268-wdt
> > > + - brcm,bcm63381-wdt
> > > + - brcm,bcm68360-wdt
> > > + - const: brcm,bcm63xx-wdt
> >
> > Is it really worthwhile to update all these DTs?:
> >
> > arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
>
> I don't have problem handling that.
Even if the dts files above are updated, the driver still has to support
the above case.
It's pointless to change this. We've already got 1000s of warnings to
fix and 2000 bindings still to convert if you need something to do.
> > I don't think so.
> So what's the policy for such bindings then? How to select SoCs that
> should have their own bindings? How can I tell which should /borrow/ a
> binding instead?
The way it is supposed to work is the first implementation gets
'soc1-block'. When the 2nd implementation comes about, it gets
'soc2-block'. If soc2 implementation is 'the same' or a superset, then
it should have a fallback to 'soc1-block'. Another way to test that is,
"I want my existing s/w to work as-is with this new h/w". The process
repeats on the next SoC, but you could be backwards compatible with
soc1, soc2, both or none. Is that what you are asking?
If you are doing all this after the fact at once, then it can be a
bit different in how you do compatibles.
In this case, I would make "brcm,bcm7038-wdt" the fallback to the rest
(granted, I know nothing about these chips or the relationship between
them), but you have to keep supporting "brcm,bcm7038-wdt". 6345 is up
to you I guess as there aren't any upstream dts files using it. Florian
might care.
Rob
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
@ 2022-02-09 20:02 ` Rob Herring
0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2022-02-09 20:02 UTC (permalink / raw)
To: Rafał Miłecki
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On Wed, Feb 09, 2022 at 08:26:00PM +0100, Rafał Miłecki wrote:
> On 9.02.2022 20:09, Rob Herring wrote:
> > On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
> > > From: Rafał Miłecki <rafal@milecki.pl>
> > >
> > > This hardware block is used on almost all BCM63xx family chipsets and
> > > BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
> > > strings and also include a generic one.
> > >
> > > The only SoC with a different block I found is BCM6838 (thus not included
> > > in this change).
> > >
> > > It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
> > > include "SoftRst" register but that can be handled by drivers based on
> > > precise compatible string.
> > >
> > > Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> > > ---
> > > V2: Sort enum entries & update brcm,twd.yaml
> > > ---
> > > .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
> > > .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
> > > 2 files changed, 18 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> > > index 634526f790b8..3f5db1990aba 100644
> > > --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> > > +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
> > > @@ -55,7 +55,7 @@ examples:
> > > #size-cells = <1>;
> > > watchdog@28 {
> > > - compatible = "brcm,bcm7038-wdt";
> > > + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
> > > reg = <0x28 0x8>;
> > > };
> > > };
> > > diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> > > index a926809352b8..4d848442913c 100644
> > > --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> > > +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
> > > @@ -16,9 +16,22 @@ maintainers:
> > > properties:
> > > compatible:
> > > - enum:
> > > - - brcm,bcm6345-wdt
> > > - - brcm,bcm7038-wdt
> > > + items:
> > > + - enum:
> > > + - brcm,bcm4908-wdt
> > > + - brcm,bcm6338-wdt
> > > + - brcm,bcm6345-wdt
> > > + - brcm,bcm6348-wdt
> > > + - brcm,bcm6848-wdt
> > > + - brcm,bcm6858-wdt
> > > + - brcm,bcm7038-wdt
> > > + - brcm,bcm60333-wdt
> > > + - brcm,bcm63138-wdt
> > > + - brcm,bcm63148-wdt
> > > + - brcm,bcm63268-wdt
> > > + - brcm,bcm63381-wdt
> > > + - brcm,bcm68360-wdt
> > > + - const: brcm,bcm63xx-wdt
> >
> > Is it really worthwhile to update all these DTs?:
> >
> > arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
> > arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
>
> I don't have problem handling that.
Even if the dts files above are updated, the driver still has to support
the above case.
It's pointless to change this. We've already got 1000s of warnings to
fix and 2000 bindings still to convert if you need something to do.
> > I don't think so.
> So what's the policy for such bindings then? How to select SoCs that
> should have their own bindings? How can I tell which should /borrow/ a
> binding instead?
The way it is supposed to work is the first implementation gets
'soc1-block'. When the 2nd implementation comes about, it gets
'soc2-block'. If soc2 implementation is 'the same' or a superset, then
it should have a fallback to 'soc1-block'. Another way to test that is,
"I want my existing s/w to work as-is with this new h/w". The process
repeats on the next SoC, but you could be backwards compatible with
soc1, soc2, both or none. Is that what you are asking?
If you are doing all this after the fact at once, then it can be a
bit different in how you do compatibles.
In this case, I would make "brcm,bcm7038-wdt" the fallback to the rest
(granted, I know nothing about these chips or the relationship between
them), but you have to keep supporting "brcm,bcm7038-wdt". 6345 is up
to you I guess as there aren't any upstream dts files using it. Florian
might care.
Rob
_______________________________________________
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] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
2022-02-09 20:02 ` Rob Herring
@ 2022-02-09 20:05 ` Rafał Miłecki
-1 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-02-09 20:05 UTC (permalink / raw)
To: Rob Herring
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On 9.02.2022 21:02, Rob Herring wrote:
> On Wed, Feb 09, 2022 at 08:26:00PM +0100, Rafał Miłecki wrote:
>> On 9.02.2022 20:09, Rob Herring wrote:
>>> On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>
>>>> This hardware block is used on almost all BCM63xx family chipsets and
>>>> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
>>>> strings and also include a generic one.
>>>>
>>>> The only SoC with a different block I found is BCM6838 (thus not included
>>>> in this change).
>>>>
>>>> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
>>>> include "SoftRst" register but that can be handled by drivers based on
>>>> precise compatible string.
>>>>
>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>> ---
>>>> V2: Sort enum entries & update brcm,twd.yaml
>>>> ---
>>>> .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
>>>> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
>>>> 2 files changed, 18 insertions(+), 5 deletions(-)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>>>> index 634526f790b8..3f5db1990aba 100644
>>>> --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>>>> +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>>>> @@ -55,7 +55,7 @@ examples:
>>>> #size-cells = <1>;
>>>> watchdog@28 {
>>>> - compatible = "brcm,bcm7038-wdt";
>>>> + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
>>>> reg = <0x28 0x8>;
>>>> };
>>>> };
>>>> diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>>>> index a926809352b8..4d848442913c 100644
>>>> --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>>>> +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>>>> @@ -16,9 +16,22 @@ maintainers:
>>>> properties:
>>>> compatible:
>>>> - enum:
>>>> - - brcm,bcm6345-wdt
>>>> - - brcm,bcm7038-wdt
>>>> + items:
>>>> + - enum:
>>>> + - brcm,bcm4908-wdt
>>>> + - brcm,bcm6338-wdt
>>>> + - brcm,bcm6345-wdt
>>>> + - brcm,bcm6348-wdt
>>>> + - brcm,bcm6848-wdt
>>>> + - brcm,bcm6858-wdt
>>>> + - brcm,bcm7038-wdt
>>>> + - brcm,bcm60333-wdt
>>>> + - brcm,bcm63138-wdt
>>>> + - brcm,bcm63148-wdt
>>>> + - brcm,bcm63268-wdt
>>>> + - brcm,bcm63381-wdt
>>>> + - brcm,bcm68360-wdt
>>>> + - const: brcm,bcm63xx-wdt
>>>
>>> Is it really worthwhile to update all these DTs?:
>>>
>>> arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
>>
>> I don't have problem handling that.
>
> Even if the dts files above are updated, the driver still has to support
> the above case.
>
> It's pointless to change this. We've already got 1000s of warnings to
> fix and 2000 bindings still to convert if you need something to do.
>
>>> I don't think so.
>> So what's the policy for such bindings then? How to select SoCs that
>> should have their own bindings? How can I tell which should /borrow/ a
>> binding instead?
>
> The way it is supposed to work is the first implementation gets
> 'soc1-block'. When the 2nd implementation comes about, it gets
> 'soc2-block'. If soc2 implementation is 'the same' or a superset, then
> it should have a fallback to 'soc1-block'. Another way to test that is,
> "I want my existing s/w to work as-is with this new h/w". The process
> repeats on the next SoC, but you could be backwards compatible with
> soc1, soc2, both or none. Is that what you are asking?
>
> If you are doing all this after the fact at once, then it can be a
> bit different in how you do compatibles.
>
> In this case, I would make "brcm,bcm7038-wdt" the fallback to the rest
> (granted, I know nothing about these chips or the relationship between
> them), but you have to keep supporting "brcm,bcm7038-wdt". 6345 is up
> to you I guess as there aren't any upstream dts files using it. Florian
> might care.
I think I got it now, thanks for explaining!
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings
@ 2022-02-09 20:05 ` Rafał Miłecki
0 siblings, 0 replies; 14+ messages in thread
From: Rafał Miłecki @ 2022-02-09 20:05 UTC (permalink / raw)
To: Rob Herring
Cc: Wim Van Sebroeck, Guenter Roeck, Florian Fainelli, Justin Chen,
linux-watchdog, devicetree, linux-arm-kernel,
bcm-kernel-feedback-list, Rafał Miłecki
On 9.02.2022 21:02, Rob Herring wrote:
> On Wed, Feb 09, 2022 at 08:26:00PM +0100, Rafał Miłecki wrote:
>> On 9.02.2022 20:09, Rob Herring wrote:
>>> On Wed, Jan 26, 2022 at 11:20:34PM +0100, Rafał Miłecki wrote:
>>>> From: Rafał Miłecki <rafal@milecki.pl>
>>>>
>>>> This hardware block is used on almost all BCM63xx family chipsets and
>>>> BCM4908 which reuses a lot of BCM63xx parts. Add relevant compatible
>>>> strings and also include a generic one.
>>>>
>>>> The only SoC with a different block I found is BCM6838 (thus not included
>>>> in this change).
>>>>
>>>> It may be worth noting that BCM6338, BCM6345, BCM6348 and BCM63268 don't
>>>> include "SoftRst" register but that can be handled by drivers based on
>>>> precise compatible string.
>>>>
>>>> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>>>> ---
>>>> V2: Sort enum entries & update brcm,twd.yaml
>>>> ---
>>>> .../devicetree/bindings/mfd/brcm,twd.yaml | 2 +-
>>>> .../bindings/watchdog/brcm,bcm7038-wdt.yaml | 21 +++++++++++++++----
>>>> 2 files changed, 18 insertions(+), 5 deletions(-)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>>>> index 634526f790b8..3f5db1990aba 100644
>>>> --- a/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>>>> +++ b/Documentation/devicetree/bindings/mfd/brcm,twd.yaml
>>>> @@ -55,7 +55,7 @@ examples:
>>>> #size-cells = <1>;
>>>> watchdog@28 {
>>>> - compatible = "brcm,bcm7038-wdt";
>>>> + compatible = "brcm,bcm4908-wdt", "brcm,bcm63xx-wdt";
>>>> reg = <0x28 0x8>;
>>>> };
>>>> };
>>>> diff --git a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>>>> index a926809352b8..4d848442913c 100644
>>>> --- a/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>>>> +++ b/Documentation/devicetree/bindings/watchdog/brcm,bcm7038-wdt.yaml
>>>> @@ -16,9 +16,22 @@ maintainers:
>>>> properties:
>>>> compatible:
>>>> - enum:
>>>> - - brcm,bcm6345-wdt
>>>> - - brcm,bcm7038-wdt
>>>> + items:
>>>> + - enum:
>>>> + - brcm,bcm4908-wdt
>>>> + - brcm,bcm6338-wdt
>>>> + - brcm,bcm6345-wdt
>>>> + - brcm,bcm6348-wdt
>>>> + - brcm,bcm6848-wdt
>>>> + - brcm,bcm6858-wdt
>>>> + - brcm,bcm7038-wdt
>>>> + - brcm,bcm60333-wdt
>>>> + - brcm,bcm63138-wdt
>>>> + - brcm,bcm63148-wdt
>>>> + - brcm,bcm63268-wdt
>>>> + - brcm,bcm63381-wdt
>>>> + - brcm,bcm68360-wdt
>>>> + - const: brcm,bcm63xx-wdt
>>>
>>> Is it really worthwhile to update all these DTs?:
>>>
>>> arch/mips/boot/dts/brcm/bcm63268.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6328.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6358.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6362.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm6368.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7125.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7346.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7358.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7360.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7362.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7420.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7425.dtsi: compatible = "brcm,bcm7038-wdt";
>>> arch/mips/boot/dts/brcm/bcm7435.dtsi: compatible = "brcm,bcm7038-wdt";
>>
>> I don't have problem handling that.
>
> Even if the dts files above are updated, the driver still has to support
> the above case.
>
> It's pointless to change this. We've already got 1000s of warnings to
> fix and 2000 bindings still to convert if you need something to do.
>
>>> I don't think so.
>> So what's the policy for such bindings then? How to select SoCs that
>> should have their own bindings? How can I tell which should /borrow/ a
>> binding instead?
>
> The way it is supposed to work is the first implementation gets
> 'soc1-block'. When the 2nd implementation comes about, it gets
> 'soc2-block'. If soc2 implementation is 'the same' or a superset, then
> it should have a fallback to 'soc1-block'. Another way to test that is,
> "I want my existing s/w to work as-is with this new h/w". The process
> repeats on the next SoC, but you could be backwards compatible with
> soc1, soc2, both or none. Is that what you are asking?
>
> If you are doing all this after the fact at once, then it can be a
> bit different in how you do compatibles.
>
> In this case, I would make "brcm,bcm7038-wdt" the fallback to the rest
> (granted, I know nothing about these chips or the relationship between
> them), but you have to keep supporting "brcm,bcm7038-wdt". 6345 is up
> to you I guess as there aren't any upstream dts files using it. Florian
> might care.
I think I got it now, thanks for explaining!
_______________________________________________
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] 14+ messages in thread
end of thread, other threads:[~2022-02-09 20:07 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-26 13:21 [PATCH] dt-bindings: watchdog: brcm,bcm7038: add more compatible strings Rafał Miłecki
2022-01-26 13:21 ` [PATCH] dt-bindings: watchdog: brcm, bcm7038: " Rafał Miłecki
2022-01-26 22:10 ` [PATCH] dt-bindings: watchdog: brcm,bcm7038: " Rob Herring
2022-01-26 22:10 ` [PATCH] dt-bindings: watchdog: brcm, bcm7038: " Rob Herring
2022-01-26 22:20 ` [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: " Rafał Miłecki
2022-01-26 22:20 ` [PATCH V2] dt-bindings: watchdog: brcm, bcm7038: " Rafał Miłecki
2022-02-09 19:09 ` [PATCH V2] dt-bindings: watchdog: brcm,bcm7038: " Rob Herring
2022-02-09 19:09 ` Rob Herring
2022-02-09 19:26 ` Rafał Miłecki
2022-02-09 19:26 ` Rafał Miłecki
2022-02-09 20:02 ` Rob Herring
2022-02-09 20:02 ` Rob Herring
2022-02-09 20:05 ` Rafał Miłecki
2022-02-09 20:05 ` Rafał Miłecki
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.