* [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
* [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
* [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
* [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 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
* 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
* 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
* 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
* 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
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