* [PATCH 0/4] Synquacer DT schema fixes
@ 2022-12-06 16:16 Rob Herring
2022-12-06 16:16 ` [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles Rob Herring
` (4 more replies)
0 siblings, 5 replies; 11+ messages in thread
From: Rob Herring @ 2022-12-06 16:16 UTC (permalink / raw)
To: Masahisa Kojima, Tom Rini, Ilias Apalodimas; +Cc: u-boot
This is a series of DT fixes for the Synquacer. These issues were found
running the dtschema tools.
I don't have a board, but Ilias has tested the changes for me. Thanks!
Signed-off-by: Rob Herring <robh@kernel.org>
---
Rob Herring (4):
dts: synquacer: Drop CPU 'arm,armv8' compatibles
dts: synquacer: Use generic node names
dts: synquacer: Fix "arm,armv7-timer-mem" node address sizes
dts: synquacer: Fix idle-states 'entry-method' value
.../dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 +-
arch/arm/dts/synquacer-sc2a11-developerbox.dts | 2 +-
arch/arm/dts/synquacer-sc2a11.dtsi | 70 +++++++++++-----------
3 files changed, 37 insertions(+), 37 deletions(-)
---
base-commit: bebb393b340295edb9ba50a996fc0510cd1b6ac0
change-id: 20221206-synquacer-dts-521500f88a1d
Best regards,
--
Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 11+ messages in thread* [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles 2022-12-06 16:16 [PATCH 0/4] Synquacer DT schema fixes Rob Herring @ 2022-12-06 16:16 ` Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-06 16:16 ` [PATCH 2/4] dts: synquacer: Use generic node names Rob Herring ` (3 subsequent siblings) 4 siblings, 1 reply; 11+ messages in thread From: Rob Herring @ 2022-12-06 16:16 UTC (permalink / raw) To: Masahisa Kojima, Tom Rini, Ilias Apalodimas; +Cc: u-boot 'arm,armv8' compatible is for software models only. so drop it from cpu nodes. Signed-off-by: Rob Herring <robh@kernel.org> --- arch/arm/dts/synquacer-sc2a11.dtsi | 48 +++++++++++++++++++------------------- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 1fe7d214b9c9..1f8cd9d25780 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -41,168 +41,168 @@ CPU0: cpu@0 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x0>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU1: cpu@1 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x1>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU2: cpu@100 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x100>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU3: cpu@101 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x101>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU4: cpu@200 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x200>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU5: cpu@201 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x201>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU6: cpu@300 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x300>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU7: cpu@301 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x301>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU8: cpu@400 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x400>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU9: cpu@401 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x401>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU10: cpu@500 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x500>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU11: cpu@501 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x501>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU12: cpu@600 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x600>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU13: cpu@601 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x601>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU14: cpu@700 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x700>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU15: cpu@701 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x701>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU16: cpu@800 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x800>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU17: cpu@801 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x801>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU18: cpu@900 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x900>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU19: cpu@901 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0x901>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU20: cpu@a00 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0xa00>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU21: cpu@a01 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0xa01>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU22: cpu@b00 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0xb00>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; }; CPU23: cpu@b01 { device_type = "cpu"; - compatible = "arm,cortex-a53","arm,armv8"; + compatible = "arm,cortex-a53"; reg = <0xb01>; enable-method = "psci"; cpu-idle-states = <&CPU_SLEEP_0 &CLUSTER_SLEEP_0>; -- b4 0.11.0-dev ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles 2022-12-06 16:16 ` [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles Rob Herring @ 2023-01-03 15:41 ` Tom Rini 0 siblings, 0 replies; 11+ messages in thread From: Tom Rini @ 2023-01-03 15:41 UTC (permalink / raw) To: Rob Herring; +Cc: Masahisa Kojima, Ilias Apalodimas, u-boot [-- Attachment #1: Type: text/plain, Size: 248 bytes --] On Tue, Dec 06, 2022 at 10:16:29AM -0600, Rob Herring wrote: > 'arm,armv8' compatible is for software models only. so drop it from cpu > nodes. > > Signed-off-by: Rob Herring <robh@kernel.org> Applied to u-boot/next, thanks! -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 2/4] dts: synquacer: Use generic node names 2022-12-06 16:16 [PATCH 0/4] Synquacer DT schema fixes Rob Herring 2022-12-06 16:16 ` [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles Rob Herring @ 2022-12-06 16:16 ` Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-06 16:16 ` [PATCH 3/4] dts: synquacer: Fix "arm, armv7-timer-mem" node address sizes Rob Herring ` (2 subsequent siblings) 4 siblings, 1 reply; 11+ messages in thread From: Rob Herring @ 2022-12-06 16:16 UTC (permalink / raw) To: Masahisa Kojima, Tom Rini, Ilias Apalodimas; +Cc: u-boot DT node names should follow generic names defined in the DT spec. These are also now checked by dtschema tools. Signed-off-by: Rob Herring <robh@kernel.org> --- arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 +- arch/arm/dts/synquacer-sc2a11-developerbox.dts | 2 +- arch/arm/dts/synquacer-sc2a11.dtsi | 10 +++++----- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi index 7a56116d6f12..6b85c05458d4 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi +++ b/arch/arm/dts/synquacer-sc2a11-developerbox-u-boot.dtsi @@ -23,7 +23,7 @@ active_clk_edges; chipselect_num = <1>; - spi-flash@0 { + flash@0 { #address-cells = <1>; #size-cells = <1>; compatible = "jedec,spi-nor"; diff --git a/arch/arm/dts/synquacer-sc2a11-developerbox.dts b/arch/arm/dts/synquacer-sc2a11-developerbox.dts index 42b6cbbb8251..c8087b99a781 100644 --- a/arch/arm/dts/synquacer-sc2a11-developerbox.dts +++ b/arch/arm/dts/synquacer-sc2a11-developerbox.dts @@ -18,7 +18,7 @@ compatible = "gpio-keys"; interrupt-parent = <&exiu>; - power { + power-button { label = "Power Button"; linux,code = <KEY_POWER>; interrupts = <GIC_SPI 120 IRQ_TYPE_EDGE_FALLING>; diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 1f8cd9d25780..0e1bc164549f 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -344,7 +344,7 @@ interrupt-controller; interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_LOW>; - its: gic-its@30020000 { + its: msi-controller@30020000 { compatible = "arm,gic-v3-its"; reg = <0x0 0x30020000 0x0 0x20000>; #msi-cells = <1>; @@ -361,7 +361,7 @@ <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>; // HYP }; - mmio-timer@2a810000 { + timer@2a810000 { compatible = "arm,armv7-timer-mem"; reg = <0x0 0x2a810000 0x0 0x10000>; #address-cells = <2>; @@ -398,7 +398,7 @@ clock-output-names = "apb_pclk"; }; - soc_uart0: uart@2a400000 { + soc_uart0: serial@2a400000 { compatible = "arm,pl011", "arm,primecell"; reg = <0x0 0x2a400000 0x0 0x1000>; interrupts = <GIC_SPI 63 IRQ_TYPE_LEVEL_HIGH>; @@ -406,7 +406,7 @@ clock-names = "uartclk", "apb_pclk"; }; - fuart: uart@51040000 { + fuart: serial@51040000 { compatible = "snps,dw-apb-uart"; reg = <0x0 0x51040000 0x0 0x1000>; interrupts = <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>; @@ -523,7 +523,7 @@ clock-output-names = "sd_sd4clk"; }; - sdhci: sdhci@52300000 { + sdhci: mmc@52300000 { compatible = "socionext,synquacer-sdhci", "fujitsu,mb86s70-sdhci-3.0"; reg = <0 0x52300000 0x0 0x1000>; interrupts = <GIC_SPI 152 IRQ_TYPE_LEVEL_HIGH>, -- b4 0.11.0-dev ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/4] dts: synquacer: Use generic node names 2022-12-06 16:16 ` [PATCH 2/4] dts: synquacer: Use generic node names Rob Herring @ 2023-01-03 15:41 ` Tom Rini 0 siblings, 0 replies; 11+ messages in thread From: Tom Rini @ 2023-01-03 15:41 UTC (permalink / raw) To: Rob Herring; +Cc: Masahisa Kojima, Ilias Apalodimas, u-boot [-- Attachment #1: Type: text/plain, Size: 281 bytes --] On Tue, Dec 06, 2022 at 10:16:30AM -0600, Rob Herring wrote: > DT node names should follow generic names defined in the DT spec. These > are also now checked by dtschema tools. > > Signed-off-by: Rob Herring <robh@kernel.org> Applied to u-boot/next, thanks! -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/4] dts: synquacer: Fix "arm, armv7-timer-mem" node address sizes 2022-12-06 16:16 [PATCH 0/4] Synquacer DT schema fixes Rob Herring 2022-12-06 16:16 ` [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles Rob Herring 2022-12-06 16:16 ` [PATCH 2/4] dts: synquacer: Use generic node names Rob Herring @ 2022-12-06 16:16 ` Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-06 16:16 ` [PATCH 4/4] dts: synquacer: Fix idle-states 'entry-method' value Rob Herring 2022-12-07 6:57 ` [PATCH 0/4] Synquacer DT schema fixes Ilias Apalodimas 4 siblings, 1 reply; 11+ messages in thread From: Rob Herring @ 2022-12-06 16:16 UTC (permalink / raw) To: Masahisa Kojima, Tom Rini, Ilias Apalodimas; +Cc: u-boot The "arm,armv7-timer-mem" schema defines the address sizes for child nodes to be 32-bit as there's no need for 64-bit offsets and sizes of the child 'frame' nodes. Signed-off-by: Rob Herring <robh@kernel.org> --- arch/arm/dts/synquacer-sc2a11.dtsi | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 0e1bc164549f..049afcb0af8a 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -364,13 +364,13 @@ timer@2a810000 { compatible = "arm,armv7-timer-mem"; reg = <0x0 0x2a810000 0x0 0x10000>; - #address-cells = <2>; - #size-cells = <2>; - ranges; - frame@2a830000 { + #address-cells = <1>; + #size-cells = <1>; + ranges = <0x0 0x0 0x2a810000 0x30000>; + frame@20000 { frame-number = <0>; interrupts = <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>; - reg = <0x0 0x2a830000 0x0 0x10000>; + reg = <0x20000 0x10000>; }; }; -- b4 0.11.0-dev ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 3/4] dts: synquacer: Fix "arm, armv7-timer-mem" node address sizes 2022-12-06 16:16 ` [PATCH 3/4] dts: synquacer: Fix "arm, armv7-timer-mem" node address sizes Rob Herring @ 2023-01-03 15:41 ` Tom Rini 0 siblings, 0 replies; 11+ messages in thread From: Tom Rini @ 2023-01-03 15:41 UTC (permalink / raw) To: Rob Herring; +Cc: Masahisa Kojima, Ilias Apalodimas, u-boot [-- Attachment #1: Type: text/plain, Size: 336 bytes --] On Tue, Dec 06, 2022 at 10:16:31AM -0600, Rob Herring wrote: > The "arm,armv7-timer-mem" schema defines the address sizes for child > nodes to be 32-bit as there's no need for 64-bit offsets and sizes of > the child 'frame' nodes. > > Signed-off-by: Rob Herring <robh@kernel.org> Applied to u-boot/next, thanks! -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 4/4] dts: synquacer: Fix idle-states 'entry-method' value 2022-12-06 16:16 [PATCH 0/4] Synquacer DT schema fixes Rob Herring ` (2 preceding siblings ...) 2022-12-06 16:16 ` [PATCH 3/4] dts: synquacer: Fix "arm, armv7-timer-mem" node address sizes Rob Herring @ 2022-12-06 16:16 ` Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-07 6:57 ` [PATCH 0/4] Synquacer DT schema fixes Ilias Apalodimas 4 siblings, 1 reply; 11+ messages in thread From: Rob Herring @ 2022-12-06 16:16 UTC (permalink / raw) To: Masahisa Kojima, Tom Rini, Ilias Apalodimas; +Cc: u-boot The correct value for 'entry-method' in the idle-states binding is 'psci', not 'arm,psci'. It hasn't mattered because it isn't used by the OS. Signed-off-by: Rob Herring <robh@kernel.org> --- arch/arm/dts/synquacer-sc2a11.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/dts/synquacer-sc2a11.dtsi b/arch/arm/dts/synquacer-sc2a11.dtsi index 049afcb0af8a..0dd2969b5e3c 100644 --- a/arch/arm/dts/synquacer-sc2a11.dtsi +++ b/arch/arm/dts/synquacer-sc2a11.dtsi @@ -309,7 +309,7 @@ }; idle-states { - entry-method = "arm,psci"; + entry-method = "psci"; CPU_SLEEP_0: cpu-sleep-0 { compatible = "arm,idle-state"; -- b4 0.11.0-dev ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 4/4] dts: synquacer: Fix idle-states 'entry-method' value 2022-12-06 16:16 ` [PATCH 4/4] dts: synquacer: Fix idle-states 'entry-method' value Rob Herring @ 2023-01-03 15:41 ` Tom Rini 0 siblings, 0 replies; 11+ messages in thread From: Tom Rini @ 2023-01-03 15:41 UTC (permalink / raw) To: Rob Herring; +Cc: Masahisa Kojima, Ilias Apalodimas, u-boot [-- Attachment #1: Type: text/plain, Size: 312 bytes --] On Tue, Dec 06, 2022 at 10:16:32AM -0600, Rob Herring wrote: > The correct value for 'entry-method' in the idle-states binding is 'psci', > not 'arm,psci'. It hasn't mattered because it isn't used by the OS. > > Signed-off-by: Rob Herring <robh@kernel.org> Applied to u-boot/next, thanks! -- Tom [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 659 bytes --] ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/4] Synquacer DT schema fixes 2022-12-06 16:16 [PATCH 0/4] Synquacer DT schema fixes Rob Herring ` (3 preceding siblings ...) 2022-12-06 16:16 ` [PATCH 4/4] dts: synquacer: Fix idle-states 'entry-method' value Rob Herring @ 2022-12-07 6:57 ` Ilias Apalodimas 2022-12-07 12:51 ` Masahisa Kojima 4 siblings, 1 reply; 11+ messages in thread From: Ilias Apalodimas @ 2022-12-07 6:57 UTC (permalink / raw) To: Rob Herring; +Cc: Masahisa Kojima, Tom Rini, u-boot Hi Rob On Tue, 6 Dec 2022 at 18:16, Rob Herring <robh@kernel.org> wrote: > > This is a series of DT fixes for the Synquacer. These issues were found > running the dtschema tools. > > I don't have a board, but Ilias has tested the changes for me. Thanks! I did and fwiw for the series Tested-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> but I'd prefer having someone from Socionext config that. Kojima-san does it look ok to you as well? Regards /Ilias > > Signed-off-by: Rob Herring <robh@kernel.org> > > --- > Rob Herring (4): > dts: synquacer: Drop CPU 'arm,armv8' compatibles > dts: synquacer: Use generic node names > dts: synquacer: Fix "arm,armv7-timer-mem" node address sizes > dts: synquacer: Fix idle-states 'entry-method' value > > .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 +- > arch/arm/dts/synquacer-sc2a11-developerbox.dts | 2 +- > arch/arm/dts/synquacer-sc2a11.dtsi | 70 +++++++++++----------- > 3 files changed, 37 insertions(+), 37 deletions(-) > --- > base-commit: bebb393b340295edb9ba50a996fc0510cd1b6ac0 > change-id: 20221206-synquacer-dts-521500f88a1d > > Best regards, > -- > Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 0/4] Synquacer DT schema fixes 2022-12-07 6:57 ` [PATCH 0/4] Synquacer DT schema fixes Ilias Apalodimas @ 2022-12-07 12:51 ` Masahisa Kojima 0 siblings, 0 replies; 11+ messages in thread From: Masahisa Kojima @ 2022-12-07 12:51 UTC (permalink / raw) To: Ilias Apalodimas, Rob Herring; +Cc: Tom Rini, u-boot Hi Rob, Ilias, On Wed, 7 Dec 2022 at 15:58, Ilias Apalodimas <ilias.apalodimas@linaro.org> wrote: > > Hi Rob > > On Tue, 6 Dec 2022 at 18:16, Rob Herring <robh@kernel.org> wrote: > > > > This is a series of DT fixes for the Synquacer. These issues were found > > running the dtschema tools. > > > > I don't have a board, but Ilias has tested the changes for me. Thanks! > > I did and fwiw for the series > Tested-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org> > > but I'd prefer having someone from Socionext config that. Kojima-san > does it look ok to you as well? I have checked and tested this series and everything is fine. Thank you for the modification. Regards, Masahisa Kojima > > Regards > /Ilias > > > > Signed-off-by: Rob Herring <robh@kernel.org> > > > > --- > > Rob Herring (4): > > dts: synquacer: Drop CPU 'arm,armv8' compatibles > > dts: synquacer: Use generic node names > > dts: synquacer: Fix "arm,armv7-timer-mem" node address sizes > > dts: synquacer: Fix idle-states 'entry-method' value > > > > .../dts/synquacer-sc2a11-developerbox-u-boot.dtsi | 2 +- > > arch/arm/dts/synquacer-sc2a11-developerbox.dts | 2 +- > > arch/arm/dts/synquacer-sc2a11.dtsi | 70 +++++++++++----------- > > 3 files changed, 37 insertions(+), 37 deletions(-) > > --- > > base-commit: bebb393b340295edb9ba50a996fc0510cd1b6ac0 > > change-id: 20221206-synquacer-dts-521500f88a1d > > > > Best regards, > > -- > > Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2023-01-03 15:42 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-12-06 16:16 [PATCH 0/4] Synquacer DT schema fixes Rob Herring 2022-12-06 16:16 ` [PATCH 1/4] dts: synquacer: Drop CPU 'arm,armv8' compatibles Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-06 16:16 ` [PATCH 2/4] dts: synquacer: Use generic node names Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-06 16:16 ` [PATCH 3/4] dts: synquacer: Fix "arm, armv7-timer-mem" node address sizes Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-06 16:16 ` [PATCH 4/4] dts: synquacer: Fix idle-states 'entry-method' value Rob Herring 2023-01-03 15:41 ` Tom Rini 2022-12-07 6:57 ` [PATCH 0/4] Synquacer DT schema fixes Ilias Apalodimas 2022-12-07 12:51 ` Masahisa Kojima
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox