public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [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 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 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

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

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