* [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address
@ 2017-12-13 19:37 Corentin Labbe
2017-12-13 19:37 ` [PATCH 2/2] arm: dts: sun8i: a83t: Add an unit address to the memory node Corentin Labbe
2017-12-14 9:53 ` [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address Maxime Ripard
0 siblings, 2 replies; 4+ messages in thread
From: Corentin Labbe @ 2017-12-13 19:37 UTC (permalink / raw)
To: linux-arm-kernel
This will fix the following warning:
Warning (simple_bus_reg): Node /soc/cpucfg at 01700000 simple-bus unit address format error, expected "1700000"
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-a83t.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 7b372acf4bf6..715719f9ea2e 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -377,7 +377,7 @@
#reset-cells = <1>;
};
- cpucfg at 01700000 {
+ cpucfg at 1700000 {
compatible = "allwinner,sun9i-a80-cpucfg";
reg = <0x01700000 0x100>;
};
--
2.13.6
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/2] arm: dts: sun8i: a83t: Add an unit address to the memory node
2017-12-13 19:37 [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address Corentin Labbe
@ 2017-12-13 19:37 ` Corentin Labbe
2017-12-14 9:54 ` Maxime Ripard
2017-12-14 9:53 ` [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address Maxime Ripard
1 sibling, 1 reply; 4+ messages in thread
From: Corentin Labbe @ 2017-12-13 19:37 UTC (permalink / raw)
To: linux-arm-kernel
This will fix the following warning:
Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-a83t.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 715719f9ea2e..bddde0141f24 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -160,7 +160,7 @@
};
};
- memory {
+ memory at 40000000 {
reg = <0x40000000 0x80000000>;
device_type = "memory";
};
--
2.13.6
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/2] arm: dts: sun8i: a83t: Add an unit address to the memory node
2017-12-13 19:37 ` [PATCH 2/2] arm: dts: sun8i: a83t: Add an unit address to the memory node Corentin Labbe
@ 2017-12-14 9:54 ` Maxime Ripard
0 siblings, 0 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-12-14 9:54 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Wed, Dec 13, 2017 at 08:37:15PM +0100, Corentin Labbe wrote:
> This will fix the following warning:
> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
>
> Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> ---
> arch/arm/boot/dts/sun8i-a83t.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
> index 715719f9ea2e..bddde0141f24 100644
> --- a/arch/arm/boot/dts/sun8i-a83t.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
> @@ -160,7 +160,7 @@
> };
> };
>
> - memory {
> + memory at 40000000 {
> reg = <0x40000000 0x80000000>;
> device_type = "memory";
> };
Can't we just get rid of the memory node entirely? U-Boot will add it
for us anyway.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20171214/c81f0f97/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address
2017-12-13 19:37 [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address Corentin Labbe
2017-12-13 19:37 ` [PATCH 2/2] arm: dts: sun8i: a83t: Add an unit address to the memory node Corentin Labbe
@ 2017-12-14 9:53 ` Maxime Ripard
1 sibling, 0 replies; 4+ messages in thread
From: Maxime Ripard @ 2017-12-14 9:53 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Wed, Dec 13, 2017 at 08:37:14PM +0100, Corentin Labbe wrote:
> This will fix the following warning:
> Warning (simple_bus_reg): Node /soc/cpucfg at 01700000 simple-bus unit address format error, expected "1700000"
>
> Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> ---
> arch/arm/boot/dts/sun8i-a83t.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
> index 7b372acf4bf6..715719f9ea2e 100644
> --- a/arch/arm/boot/dts/sun8i-a83t.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
> @@ -377,7 +377,7 @@
> #reset-cells = <1>;
> };
>
> - cpucfg at 01700000 {
> + cpucfg at 1700000 {
> compatible = "allwinner,sun9i-a80-cpucfg";
> reg = <0x01700000 0x100>;
That node isn't upstream.
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20171214/b94b4a36/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-12-14 9:54 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-12-13 19:37 [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address Corentin Labbe
2017-12-13 19:37 ` [PATCH 2/2] arm: dts: sun8i: a83t: Add an unit address to the memory node Corentin Labbe
2017-12-14 9:54 ` Maxime Ripard
2017-12-14 9:53 ` [PATCH 1/2] arm: dts: sun8i: a83t: remove leading zero from cpucfg node address Maxime Ripard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox