Skip to contentSkip to conversation

ARM: mvebu: Add RTC support for Armada 375

PATCH New Gregory CLEMENT 2 participants 4 messages Jul 31, 2014 Device tree bindings

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: mvebu: Add RTC support for Armada 375

The Armada 375 XP Socs have the same controller that the one used in the orion platforms. This patch updates the device tree for these SoCs.

---
Hi Jason,

We missed it for the initial support of the SoC.

I know that it is very late for including for 3.17, so if it is postpone for 3.18, it's fine.

However given the little change, maybe it is still possible to push it in 3.17 ^_^

Thanks,

Gregory

 arch/arm/boot/dts/armada-375.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)
arch/arm/boot/dts/armada-375.dtsi +6 -0
index 893e2a93eba0..f6fe2c270938 100644
@@ -185,6 +185,12 @@
 				};
 			};
 
+			rtc at 10300 {
+				compatible = "marvell,orion-rtc";
+				reg = <0x10300 0x20>;
+				interrupts = <GIC_SPI 21 IRQ_TYPE_LEVEL_HIGH>;
+			};
+
 			spi0: spi at 10600 {
 				compatible = "marvell,orion-spi";
 				reg = <0x10600 0x50>;
Signature · 1 line
1.8.1.2
#1 lore raw

PATCH 1/1 ARM: mvebu: Add RTC support for Armada 375

Hi,

of course it only when the patch is setn that I found a typo:

Quoted 1 line · Gregory CLEMENT

The Armada 375 XP Socs have the same controller that the one used in

It is Armada 375 not Armada 375 XP!

Thanks,

Gregory

Quoted 41 lines

the orion platforms. This patch updates the device tree for these SoCs.

---
Hi Jason,

We missed it for the initial support of the SoC.

I know that it is very late for including for 3.17, so if it is postpone for 3.18, it's fine.

However given the little change, maybe it is still possible to push it in 3.17 ^_^

Thanks,

Gregory

 arch/arm/boot/dts/armada-375.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)
arch/arm/boot/dts/armada-375.dtsi +6 -0
index 893e2a93eba0..f6fe2c270938 100644
@@ -185,6 +185,12 @@
 				};
 			};
 
+			rtc at 10300 {
+				compatible = "marvell,orion-rtc";
+				reg = <0x10300 0x20>;
+				interrupts = <GIC_SPI 21 IRQ_TYPE_LEVEL_HIGH>;
+			};
+
 			spi0: spi at 10600 {
 				compatible = "marvell,orion-spi";
 				reg = <0x10600 0x50>;
Signature · 4 lines
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#2 lore raw

PATCH 1/1 ARM: mvebu: Add RTC support for Armada 375

In reply to Gregory CLEMENT

Dear Gregory CLEMENT,

Quoted 3 lines · Gregory CLEMENT

The Armada 375 XP Socs have the same controller that the one used in the orion platforms. This patch updates the device tree for these SoCs.

Actually there's only one Armada 375 SoC, so singular would be good. And s/that/than/ as well. And "controller" is a bit vague. Proposal:

"""
The Armada 375 SoC has the same real time clock as the one used in other Marvell EBU platforms. This patch consequently updates the Device Tree of the Armada 375 SoC to describe the internal RTC.
"""

Thanks,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
#3 lore raw

PATCH 1/1 ARM: mvebu: Add RTC support for Armada 375

Hi Thomas,

Quoted 9 lines · Thomas Petazzoni

Dear Gregory CLEMENT,

Quoted 3 lines · Gregory CLEMENT

The Armada 375 XP Socs have the same controller that the one used in the orion platforms. This patch updates the device tree for these SoCs.

Actually there's only one Armada 375 SoC, so singular would be good.

On this point I was referring to the chips not to the family. But I agree that it can be understood in both way.

Quoted 7 lines

And s/that/than/ as well. And "controller" is a bit vague. Proposal:

"""
The Armada 375 SoC has the same real time clock as the one used in other Marvell EBU platforms. This patch consequently updates the Device Tree of the Armada 375 SoC to describe the internal RTC.
"""

I agree with your proposal I will sent a new version soon.

Thanks,

Gregory

Quoted 5 lines

Thanks,

Thomas

Signature · 4 lines
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com