* [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address
@ 2017-03-29 9:38 Antoine Tenart
2017-03-29 9:38 ` [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units Antoine Tenart
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Antoine Tenart @ 2017-03-29 9:38 UTC (permalink / raw)
To: linux-arm-kernel
Remove 0x's from the uart1 node unit address to have consistent nodes.
Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com>
---
arch/arm/boot/dts/alpine.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi
index f09f9c8e178a..341a500c8d8a 100644
--- a/arch/arm/boot/dts/alpine.dtsi
+++ b/arch/arm/boot/dts/alpine.dtsi
@@ -129,7 +129,7 @@
reg-io-width = <4>;
};
- uart1:uart at 0xfd884000 {
+ uart1:uart at fd884000 {
compatible = "ns16550a";
reg = <0x0 0xfd884000 0x0 0x1000>;
clock-frequency = <0>; /* Filled by loader */
--
2.11.0
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units. 2017-03-29 9:38 [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart @ 2017-03-29 9:38 ` Antoine Tenart 2017-04-03 7:08 ` Antoine Tenart 2017-03-29 9:38 ` [PATCH 3/3] ARM: dts: alpine: add valid clock-frequency values Antoine Tenart 2017-04-03 7:07 ` [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart 2 siblings, 1 reply; 6+ messages in thread From: Antoine Tenart @ 2017-03-29 9:38 UTC (permalink / raw) To: linux-arm-kernel Cosmetic cleanup to have consistent node definitions. Add a space before the node units which do not have one. Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> --- arch/arm/boot/dts/alpine.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi index 341a500c8d8a..d84bd7955b54 100644 --- a/arch/arm/boot/dts/alpine.dtsi +++ b/arch/arm/boot/dts/alpine.dtsi @@ -120,7 +120,7 @@ <GIC_SPI 71 IRQ_TYPE_LEVEL_HIGH>; }; - uart0:uart at fd883000 { + uart0: uart at fd883000 { compatible = "ns16550a"; reg = <0x0 0xfd883000 0x0 0x1000>; clock-frequency = <0>; /* Filled by loader */ @@ -129,7 +129,7 @@ reg-io-width = <4>; }; - uart1:uart at fd884000 { + uart1: uart at fd884000 { compatible = "ns16550a"; reg = <0x0 0xfd884000 0x0 0x1000>; clock-frequency = <0>; /* Filled by loader */ -- 2.11.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units. 2017-03-29 9:38 ` [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units Antoine Tenart @ 2017-04-03 7:08 ` Antoine Tenart 0 siblings, 0 replies; 6+ messages in thread From: Antoine Tenart @ 2017-04-03 7:08 UTC (permalink / raw) To: linux-arm-kernel On Wed, Mar 29, 2017 at 11:38:26AM +0200, Antoine Tenart wrote: > Cosmetic cleanup to have consistent node definitions. Add a space before > the node units which do not have one. > > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> Applied on alpine/dt. Thanks! > --- > arch/arm/boot/dts/alpine.dtsi | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi > index 341a500c8d8a..d84bd7955b54 100644 > --- a/arch/arm/boot/dts/alpine.dtsi > +++ b/arch/arm/boot/dts/alpine.dtsi > @@ -120,7 +120,7 @@ > <GIC_SPI 71 IRQ_TYPE_LEVEL_HIGH>; > }; > > - uart0:uart at fd883000 { > + uart0: uart at fd883000 { > compatible = "ns16550a"; > reg = <0x0 0xfd883000 0x0 0x1000>; > clock-frequency = <0>; /* Filled by loader */ > @@ -129,7 +129,7 @@ > reg-io-width = <4>; > }; > > - uart1:uart at fd884000 { > + uart1: uart at fd884000 { > compatible = "ns16550a"; > reg = <0x0 0xfd884000 0x0 0x1000>; > clock-frequency = <0>; /* Filled by loader */ > -- > 2.11.0 > -- Antoine T?nart, 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: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170403/ec809886/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/3] ARM: dts: alpine: add valid clock-frequency values 2017-03-29 9:38 [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart 2017-03-29 9:38 ` [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units Antoine Tenart @ 2017-03-29 9:38 ` Antoine Tenart 2017-04-03 7:08 ` Antoine Tenart 2017-04-03 7:07 ` [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart 2 siblings, 1 reply; 6+ messages in thread From: Antoine Tenart @ 2017-03-29 9:38 UTC (permalink / raw) To: linux-arm-kernel Update the Alpine clock-frequency values with valid default values. The bootloader can still update these values if needed, but at least we can boot if it does not. Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> --- arch/arm/boot/dts/alpine.dtsi | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi index d84bd7955b54..731df7a8c4e6 100644 --- a/arch/arm/boot/dts/alpine.dtsi +++ b/arch/arm/boot/dts/alpine.dtsi @@ -41,28 +41,28 @@ compatible = "arm,cortex-a15"; device_type = "cpu"; reg = <0>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <1700000000>; }; cpu at 1 { compatible = "arm,cortex-a15"; device_type = "cpu"; reg = <1>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <1700000000>; }; cpu at 2 { compatible = "arm,cortex-a15"; device_type = "cpu"; reg = <2>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <1700000000>; }; cpu at 3 { compatible = "arm,cortex-a15"; device_type = "cpu"; reg = <3>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <1700000000>; }; }; @@ -81,7 +81,7 @@ <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <50000000>; }; /* Interrupt Controller */ @@ -123,7 +123,7 @@ uart0: uart at fd883000 { compatible = "ns16550a"; reg = <0x0 0xfd883000 0x0 0x1000>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <375000000>; interrupts = <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>; reg-shift = <2>; reg-io-width = <4>; @@ -132,7 +132,7 @@ uart1: uart at fd884000 { compatible = "ns16550a"; reg = <0x0 0xfd884000 0x0 0x1000>; - clock-frequency = <0>; /* Filled by loader */ + clock-frequency = <375000000>; interrupts = <GIC_SPI 18 IRQ_TYPE_LEVEL_HIGH>; reg-shift = <2>; reg-io-width = <4>; -- 2.11.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/3] ARM: dts: alpine: add valid clock-frequency values 2017-03-29 9:38 ` [PATCH 3/3] ARM: dts: alpine: add valid clock-frequency values Antoine Tenart @ 2017-04-03 7:08 ` Antoine Tenart 0 siblings, 0 replies; 6+ messages in thread From: Antoine Tenart @ 2017-04-03 7:08 UTC (permalink / raw) To: linux-arm-kernel On Wed, Mar 29, 2017 at 11:38:27AM +0200, Antoine Tenart wrote: > Update the Alpine clock-frequency values with valid default values. The > bootloader can still update these values if needed, but at least we can > boot if it does not. > > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> Applied on alpine/dt. Thanks! > --- > arch/arm/boot/dts/alpine.dtsi | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi > index d84bd7955b54..731df7a8c4e6 100644 > --- a/arch/arm/boot/dts/alpine.dtsi > +++ b/arch/arm/boot/dts/alpine.dtsi > @@ -41,28 +41,28 @@ > compatible = "arm,cortex-a15"; > device_type = "cpu"; > reg = <0>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <1700000000>; > }; > > cpu at 1 { > compatible = "arm,cortex-a15"; > device_type = "cpu"; > reg = <1>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <1700000000>; > }; > > cpu at 2 { > compatible = "arm,cortex-a15"; > device_type = "cpu"; > reg = <2>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <1700000000>; > }; > > cpu at 3 { > compatible = "arm,cortex-a15"; > device_type = "cpu"; > reg = <3>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <1700000000>; > }; > }; > > @@ -81,7 +81,7 @@ > <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, > <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <50000000>; > }; > > /* Interrupt Controller */ > @@ -123,7 +123,7 @@ > uart0: uart at fd883000 { > compatible = "ns16550a"; > reg = <0x0 0xfd883000 0x0 0x1000>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <375000000>; > interrupts = <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>; > reg-shift = <2>; > reg-io-width = <4>; > @@ -132,7 +132,7 @@ > uart1: uart at fd884000 { > compatible = "ns16550a"; > reg = <0x0 0xfd884000 0x0 0x1000>; > - clock-frequency = <0>; /* Filled by loader */ > + clock-frequency = <375000000>; > interrupts = <GIC_SPI 18 IRQ_TYPE_LEVEL_HIGH>; > reg-shift = <2>; > reg-io-width = <4>; > -- > 2.11.0 > -- Antoine T?nart, 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: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170403/c7a19edc/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address 2017-03-29 9:38 [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart 2017-03-29 9:38 ` [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units Antoine Tenart 2017-03-29 9:38 ` [PATCH 3/3] ARM: dts: alpine: add valid clock-frequency values Antoine Tenart @ 2017-04-03 7:07 ` Antoine Tenart 2 siblings, 0 replies; 6+ messages in thread From: Antoine Tenart @ 2017-04-03 7:07 UTC (permalink / raw) To: linux-arm-kernel On Wed, Mar 29, 2017 at 11:38:25AM +0200, Antoine Tenart wrote: > Remove 0x's from the uart1 node unit address to have consistent nodes. > > Signed-off-by: Antoine Tenart <antoine.tenart@free-electrons.com> Applied on alpine/dt. Thanks! > --- > arch/arm/boot/dts/alpine.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/alpine.dtsi b/arch/arm/boot/dts/alpine.dtsi > index f09f9c8e178a..341a500c8d8a 100644 > --- a/arch/arm/boot/dts/alpine.dtsi > +++ b/arch/arm/boot/dts/alpine.dtsi > @@ -129,7 +129,7 @@ > reg-io-width = <4>; > }; > > - uart1:uart at 0xfd884000 { > + uart1:uart at fd884000 { > compatible = "ns16550a"; > reg = <0x0 0xfd884000 0x0 0x1000>; > clock-frequency = <0>; /* Filled by loader */ > -- > 2.11.0 > -- Antoine T?nart, 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: 801 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170403/5846ce51/attachment.sig> ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-04-03 7:08 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-03-29 9:38 [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart 2017-03-29 9:38 ` [PATCH 2/3] ARM: dts: alpine: add spaces before the uart node units Antoine Tenart 2017-04-03 7:08 ` Antoine Tenart 2017-03-29 9:38 ` [PATCH 3/3] ARM: dts: alpine: add valid clock-frequency values Antoine Tenart 2017-04-03 7:08 ` Antoine Tenart 2017-04-03 7:07 ` [PATCH 1/3] ARM: dts: alpine: remove 0x's from the uart1 node unit address Antoine Tenart
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox