* [PATCH v2 0/2] arm64: dts: add new compatibles for J7 platforms
@ 2022-10-20 3:28 Matt Ranostay
2022-10-20 3:28 ` [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms Matt Ranostay
2022-10-20 3:28 ` [PATCH v2 2/2] arm64: dts: k3-j72*: correct compatible for syscon entries Matt Ranostay
0 siblings, 2 replies; 6+ messages in thread
From: Matt Ranostay @ 2022-10-20 3:28 UTC (permalink / raw)
To: nm, vigneshr, krzysztof.kozlowski+dt, robh+dt
Cc: linux-arm-kernel, devicetree, Matt Ranostay
Changes from v1:
* add j7200 and j721s2 syscon compatible strings to dt-binding documentation
* changed compatible strings to use newly defined ones per platform
Matt Ranostay (2):
dt-bindings: mfd: ti,j721e-system-controller: add compatible strings
for other platforms
arm64: dts: k3-j72*: correct compatible for syscon entries
.../devicetree/bindings/mfd/ti,j721e-system-controller.yaml | 2 ++
arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 2 +-
4 files changed, 5 insertions(+), 3 deletions(-)
--
2.38.GIT
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms 2022-10-20 3:28 [PATCH v2 0/2] arm64: dts: add new compatibles for J7 platforms Matt Ranostay @ 2022-10-20 3:28 ` Matt Ranostay 2022-10-20 13:49 ` Krzysztof Kozlowski 2022-10-20 14:24 ` Rob Herring 2022-10-20 3:28 ` [PATCH v2 2/2] arm64: dts: k3-j72*: correct compatible for syscon entries Matt Ranostay 1 sibling, 2 replies; 6+ messages in thread From: Matt Ranostay @ 2022-10-20 3:28 UTC (permalink / raw) To: nm, vigneshr, krzysztof.kozlowski+dt, robh+dt Cc: linux-arm-kernel, devicetree, Matt Ranostay There are multiple J7 based platforms, and the j721e-system-controller shouldn't be define in non-j721e devices device trees. This is mainly for clarity; but also useful in case there are future erratas that need to be fixed for a specific platform. Signed-off-by: Matt Ranostay <mranostay@ti.com> --- .../devicetree/bindings/mfd/ti,j721e-system-controller.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml index 873ee0c0973f..76ef4352e13c 100644 --- a/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml +++ b/Documentation/devicetree/bindings/mfd/ti,j721e-system-controller.yaml @@ -26,7 +26,9 @@ properties: compatible: items: - enum: + - ti,j7200-system-controller - ti,j721e-system-controller + - ti,j721s2-system-controller - const: syscon - const: simple-mfd -- 2.38.GIT ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms 2022-10-20 3:28 ` [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms Matt Ranostay @ 2022-10-20 13:49 ` Krzysztof Kozlowski 2022-10-20 14:24 ` Rob Herring 1 sibling, 0 replies; 6+ messages in thread From: Krzysztof Kozlowski @ 2022-10-20 13:49 UTC (permalink / raw) To: Matt Ranostay, nm, vigneshr, krzysztof.kozlowski+dt, robh+dt Cc: linux-arm-kernel, devicetree On 19/10/2022 23:28, Matt Ranostay wrote: > There are multiple J7 based platforms, and the j721e-system-controller > shouldn't be define in non-j721e devices device trees. > > This is mainly for clarity; but also useful in case there are future > erratas that need to be fixed for a specific platform. > > Signed-off-by: Matt Ranostay <mranostay@ti.com> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms 2022-10-20 3:28 ` [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms Matt Ranostay 2022-10-20 13:49 ` Krzysztof Kozlowski @ 2022-10-20 14:24 ` Rob Herring 1 sibling, 0 replies; 6+ messages in thread From: Rob Herring @ 2022-10-20 14:24 UTC (permalink / raw) To: Matt Ranostay Cc: nm, vigneshr, krzysztof.kozlowski+dt, linux-arm-kernel, devicetree On Wed, Oct 19, 2022 at 08:28:01PM -0700, Matt Ranostay wrote: > There are multiple J7 based platforms, and the j721e-system-controller > shouldn't be define in non-j721e devices device trees. > > This is mainly for clarity; but also useful in case there are future > erratas that need to be fixed for a specific platform. > > Signed-off-by: Matt Ranostay <mranostay@ti.com> > --- > .../devicetree/bindings/mfd/ti,j721e-system-controller.yaml | 2 ++ > 1 file changed, 2 insertions(+) Use get_maintainers.pl if you want this applied (by Lee). ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 2/2] arm64: dts: k3-j72*: correct compatible for syscon entries 2022-10-20 3:28 [PATCH v2 0/2] arm64: dts: add new compatibles for J7 platforms Matt Ranostay 2022-10-20 3:28 ` [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms Matt Ranostay @ 2022-10-20 3:28 ` Matt Ranostay 2022-10-20 13:49 ` Krzysztof Kozlowski 1 sibling, 1 reply; 6+ messages in thread From: Matt Ranostay @ 2022-10-20 3:28 UTC (permalink / raw) To: nm, vigneshr, krzysztof.kozlowski+dt, robh+dt Cc: linux-arm-kernel, devicetree, Matt Ranostay Add missing ti,j7*-system-controller compatible to bus defines in mcu/wakeup domains to avoid the following similar warnings from dt-schema checks: arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: syscon@40f00000: compatible: ['syscon', 'simple-mfd'] is too short' Signed-off-by: Matt Ranostay <mranostay@ti.com> --- arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 2 +- arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 2 +- arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi index e5be78a58682..2025aab4b66f 100644 --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi @@ -35,7 +35,7 @@ k3_reset: reset-controller { }; mcu_conf: syscon@40f00000 { - compatible = "syscon", "simple-mfd"; + compatible = "ti,j7200-system-controller", "syscon", "simple-mfd"; reg = <0x00 0x40f00000 0x00 0x20000>; #address-cells = <1>; #size-cells = <1>; diff --git a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi index df08724bbf1c..5d9ec221fa34 100644 --- a/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j721e-mcu-wakeup.dtsi @@ -35,7 +35,7 @@ k3_reset: reset-controller { }; mcu_conf: syscon@40f00000 { - compatible = "syscon", "simple-mfd"; + compatible = "ti,j721e-system-controller", "syscon", "simple-mfd"; reg = <0x0 0x40f00000 0x0 0x20000>; #address-cells = <1>; #size-cells = <1>; diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi index 4d1bfabd1313..dfd40369b931 100644 --- a/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi @@ -69,7 +69,7 @@ wkup_gpio_intr: interrupt-controller@42200000 { }; mcu_conf: syscon@40f00000 { - compatible = "syscon", "simple-mfd"; + compatible = "ti,j721s2-system-controller", "syscon", "simple-mfd"; reg = <0x0 0x40f00000 0x0 0x20000>; #address-cells = <1>; #size-cells = <1>; -- 2.38.GIT ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: k3-j72*: correct compatible for syscon entries 2022-10-20 3:28 ` [PATCH v2 2/2] arm64: dts: k3-j72*: correct compatible for syscon entries Matt Ranostay @ 2022-10-20 13:49 ` Krzysztof Kozlowski 0 siblings, 0 replies; 6+ messages in thread From: Krzysztof Kozlowski @ 2022-10-20 13:49 UTC (permalink / raw) To: Matt Ranostay, nm, vigneshr, krzysztof.kozlowski+dt, robh+dt Cc: linux-arm-kernel, devicetree On 19/10/2022 23:28, Matt Ranostay wrote: > Add missing ti,j7*-system-controller compatible to bus defines in mcu/wakeup > domains to avoid the following similar warnings from dt-schema checks: > > arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dtb: syscon@40f00000: compatible: ['syscon', 'simple-mfd'] is too short' > > Signed-off-by: Matt Ranostay <mranostay@ti.com> > --- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-10-20 14:24 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-10-20 3:28 [PATCH v2 0/2] arm64: dts: add new compatibles for J7 platforms Matt Ranostay 2022-10-20 3:28 ` [PATCH v2 1/2] dt-bindings: mfd: ti,j721e-system-controller: add compatible strings for other platforms Matt Ranostay 2022-10-20 13:49 ` Krzysztof Kozlowski 2022-10-20 14:24 ` Rob Herring 2022-10-20 3:28 ` [PATCH v2 2/2] arm64: dts: k3-j72*: correct compatible for syscon entries Matt Ranostay 2022-10-20 13:49 ` Krzysztof Kozlowski
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).