Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support
@ 2017-09-11  2:22 Andreas Färber
  2017-09-11  2:22 ` [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7 Andreas Färber
                   ` (4 more replies)
  0 siblings, 5 replies; 12+ messages in thread
From: Andreas Färber @ 2017-09-11  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

Hello,

This series prepares the Actions Semi S700 SoC and Cubietech CubieBoard7.
It boots equally far as the S900 based Bubblegum-96 these patches are based on,
i.e. into an initrd with serial and all four cores up.

I have not yet managed to download the CubieBoard7 sources from Baidu, and the
Mega download has not yet been fixed, so this is based on the S700 datasheet
and /proc/device-tree/ from the preinstalled Android.

Lacking sources and instructions to replace the bootloader on eMMC or to try
booting from SD, the only working way I've found to boot mainline kernels
appears to be booting into Android, then issuing "reboot bootloader". U-Boot
is lacking the saveenv command, so it wasn't possible to change the bootdelay
to facilitate this.

Cf. https://en.opensuse.org/HCL:CubieBoard7

Work branch:
https://github.com/afaerber/linux/commits/bg96-next

Have a lot of fun!

Cheers,
Andreas

Cc: Ahha Lee <lee@cubietech.com>
Cc: support at cubietech.com

Cc: devicetree at vger.kernel.org
Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Thomas Gleixner <tglx@linutronix.de>

Andreas F?rber (4):
  dt-bindings: arm: actions: Add S700 and CubieBoard7
  dt-bindings: timer: Add Actions Semi S700
  clocksource: owl: Prepare S700
  arm64: dts: actions: Add S700 and CubieBoard7

 Documentation/devicetree/bindings/arm/actions.txt  |  15 ++
 .../bindings/timer/actions,owl-timer.txt           |   1 +
 arch/arm64/boot/dts/actions/Makefile               |   2 +
 arch/arm64/boot/dts/actions/s700-cubieboard7.dts   |  47 ++++++
 arch/arm64/boot/dts/actions/s700.dtsi              | 164 +++++++++++++++++++++
 drivers/clocksource/owl-timer.c                    |   1 +
 6 files changed, 230 insertions(+)
 create mode 100644 arch/arm64/boot/dts/actions/s700-cubieboard7.dts
 create mode 100644 arch/arm64/boot/dts/actions/s700.dtsi

-- 
2.13.5

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7
  2017-09-11  2:22 [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Andreas Färber
@ 2017-09-11  2:22 ` Andreas Färber
  2017-09-18 20:45   ` Rob Herring
  2017-09-11  2:22 ` [PATCH 2/4] dt-bindings: timer: Add Actions Semi S700 Andreas Färber
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 12+ messages in thread
From: Andreas Färber @ 2017-09-11  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

Document the Actions Semi S700 SoC and the Cubietech CubieBoard7.

Signed-off-by: Andreas F?rber <afaerber@suse.de>
---
 Documentation/devicetree/bindings/arm/actions.txt | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/actions.txt b/Documentation/devicetree/bindings/arm/actions.txt
index ced764a8549e..544a8855fad5 100644
--- a/Documentation/devicetree/bindings/arm/actions.txt
+++ b/Documentation/devicetree/bindings/arm/actions.txt
@@ -25,6 +25,21 @@ Root node property compatible must contain, depending on board:
  - LeMaker Guitar Base Board rev. B: "lemaker,guitar-bb-rev-b", "lemaker,guitar"
 
 
+S700 SoC
+========
+
+Required root node properties:
+
+- compatible :  must contain "actions,s700"
+
+
+Boards:
+
+Root node property compatible must contain, depending on board:
+
+ - Cubietech CubieBoard7: "cubietech,cubieboard7"
+
+
 S900 SoC
 ========
 
-- 
2.13.5

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/4] dt-bindings: timer: Add Actions Semi S700
  2017-09-11  2:22 [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Andreas Färber
  2017-09-11  2:22 ` [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7 Andreas Färber
@ 2017-09-11  2:22 ` Andreas Färber
  2017-09-18 20:59   ` Rob Herring
  2017-09-11  2:22 ` [PATCH 3/4] clocksource: owl: Prepare S700 Andreas Färber
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 12+ messages in thread
From: Andreas Färber @ 2017-09-11  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

Define a compatible string for the Actions Semi S700 SoC timer.

Signed-off-by: Andreas F?rber <afaerber@suse.de>
---
 Documentation/devicetree/bindings/timer/actions,owl-timer.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/timer/actions,owl-timer.txt b/Documentation/devicetree/bindings/timer/actions,owl-timer.txt
index e3c28da80cb2..977054f87563 100644
--- a/Documentation/devicetree/bindings/timer/actions,owl-timer.txt
+++ b/Documentation/devicetree/bindings/timer/actions,owl-timer.txt
@@ -2,6 +2,7 @@ Actions Semi Owl Timer
 
 Required properties:
 - compatible      :  "actions,s500-timer" for S500
+                     "actions,s700-timer" for S700
                      "actions,s900-timer" for S900
 - reg             :  Offset and length of the register set for the device.
 - interrupts      :  Should contain the interrupts.
-- 
2.13.5

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/4] clocksource: owl: Prepare S700
  2017-09-11  2:22 [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Andreas Färber
  2017-09-11  2:22 ` [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7 Andreas Färber
  2017-09-11  2:22 ` [PATCH 2/4] dt-bindings: timer: Add Actions Semi S700 Andreas Färber
@ 2017-09-11  2:22 ` Andreas Färber
  2017-09-18 21:52   ` Daniel Lezcano
  2017-09-11  2:22 ` [PATCH 4/4] arm64: dts: actions: Add S700 and CubieBoard7 Andreas Färber
  2017-09-18 21:50 ` [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Daniel Lezcano
  4 siblings, 1 reply; 12+ messages in thread
From: Andreas Färber @ 2017-09-11  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

S700 has two 2Hz timers like S500, and four TIMx timers like S900.

Signed-off-by: Andreas F?rber <afaerber@suse.de>
---
 drivers/clocksource/owl-timer.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/clocksource/owl-timer.c b/drivers/clocksource/owl-timer.c
index d19c53c11094..d5f3f72c10ee 100644
--- a/drivers/clocksource/owl-timer.c
+++ b/drivers/clocksource/owl-timer.c
@@ -169,4 +169,5 @@ static int __init owl_timer_init(struct device_node *node)
 	return 0;
 }
 CLOCKSOURCE_OF_DECLARE(owl_s500, "actions,s500-timer", owl_timer_init);
+CLOCKSOURCE_OF_DECLARE(owl_s700, "actions,s700-timer", owl_timer_init);
 CLOCKSOURCE_OF_DECLARE(owl_s900, "actions,s900-timer", owl_timer_init);
-- 
2.13.5

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 4/4] arm64: dts: actions: Add S700 and CubieBoard7
  2017-09-11  2:22 [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Andreas Färber
                   ` (2 preceding siblings ...)
  2017-09-11  2:22 ` [PATCH 3/4] clocksource: owl: Prepare S700 Andreas Färber
@ 2017-09-11  2:22 ` Andreas Färber
  2017-09-18 21:50 ` [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Daniel Lezcano
  4 siblings, 0 replies; 12+ messages in thread
From: Andreas Färber @ 2017-09-11  2:22 UTC (permalink / raw)
  To: linux-arm-kernel

Add Device Trees for S700 SoC and Cubietech CubieBoard7.

Signed-off-by: Andreas F?rber <afaerber@suse.de>
---
 arch/arm64/boot/dts/actions/Makefile             |   2 +
 arch/arm64/boot/dts/actions/s700-cubieboard7.dts |  47 +++++++
 arch/arm64/boot/dts/actions/s700.dtsi            | 164 +++++++++++++++++++++++
 3 files changed, 213 insertions(+)
 create mode 100644 arch/arm64/boot/dts/actions/s700-cubieboard7.dts
 create mode 100644 arch/arm64/boot/dts/actions/s700.dtsi

diff --git a/arch/arm64/boot/dts/actions/Makefile b/arch/arm64/boot/dts/actions/Makefile
index 62922d688ce3..cfd7f2679feb 100644
--- a/arch/arm64/boot/dts/actions/Makefile
+++ b/arch/arm64/boot/dts/actions/Makefile
@@ -1,3 +1,5 @@
+dtb-$(CONFIG_ARCH_ACTIONS) += s700-cubieboard7.dtb
+
 dtb-$(CONFIG_ARCH_ACTIONS) += s900-bubblegum-96.dtb
 
 always		:= $(dtb-y)
diff --git a/arch/arm64/boot/dts/actions/s700-cubieboard7.dts b/arch/arm64/boot/dts/actions/s700-cubieboard7.dts
new file mode 100644
index 000000000000..e562f04c2490
--- /dev/null
+++ b/arch/arm64/boot/dts/actions/s700-cubieboard7.dts
@@ -0,0 +1,47 @@
+/*
+ * Copyright (c) 2017 Andreas F?rber
+ *
+ * SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+ */
+
+/dts-v1/;
+
+#include "s700.dtsi"
+
+/ {
+	compatible = "cubietech,cubieboard7", "actions,s700";
+	model = "CubieBoard7";
+
+	aliases {
+		serial3 = &uart3;
+	};
+
+	chosen {
+		stdout-path = "serial3:115200n8";
+	};
+
+	memory at 0 {
+		device_type = "memory";
+		reg = <0x0 0x0 0x0 0x80000000>;
+	};
+
+	memory at 1,e0000000 {
+		device_type = "memory";
+		reg = <0x1 0xe0000000 0x0 0x0>;
+	};
+
+	uart3_clk: uart3-clk {
+		compatible = "fixed-clock";
+		clock-frequency = <921600>;
+		#clock-cells = <0>;
+	};
+};
+
+&timer {
+	clocks = <&hosc>;
+};
+
+&uart3 {
+	status = "okay";
+	clocks = <&uart3_clk>;
+};
diff --git a/arch/arm64/boot/dts/actions/s700.dtsi b/arch/arm64/boot/dts/actions/s700.dtsi
new file mode 100644
index 000000000000..b54df405aaeb
--- /dev/null
+++ b/arch/arm64/boot/dts/actions/s700.dtsi
@@ -0,0 +1,164 @@
+/*
+ * Copyright (c) 2017 Andreas F?rber
+ *
+ * SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+ */
+
+#include <dt-bindings/interrupt-controller/arm-gic.h>
+
+/ {
+	compatible = "actions,s700";
+	interrupt-parent = <&gic>;
+	#address-cells = <2>;
+	#size-cells = <2>;
+
+	cpus {
+		#address-cells = <2>;
+		#size-cells = <0>;
+
+		cpu0: cpu at 0 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a53", "arm,armv8";
+			reg = <0x0 0x0>;
+			enable-method = "psci";
+		};
+
+		cpu1: cpu at 1 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a53", "arm,armv8";
+			reg = <0x0 0x1>;
+			enable-method = "psci";
+		};
+
+		cpu2: cpu at 2 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a53", "arm,armv8";
+			reg = <0x0 0x2>;
+			enable-method = "psci";
+		};
+
+		cpu3: cpu at 3 {
+			device_type = "cpu";
+			compatible = "arm,cortex-a53", "arm,armv8";
+			reg = <0x0 0x3>;
+			enable-method = "psci";
+		};
+	};
+
+	reserved-memory {
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		secmon at 1f000000 {
+			reg = <0x0 0x1f000000 0x0 0x1000000>;
+			no-map;
+		};
+	};
+
+	psci {
+		compatible = "arm,psci-0.2";
+		method = "smc";
+	};
+
+	arm-pmu {
+		compatible = "arm,cortex-a53-pmu";
+		interrupts = <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>,
+		             <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>,
+		             <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>,
+		             <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
+		interrupt-affinity = <&cpu0>, <&cpu1>, <&cpu2>, <&cpu3>;
+	};
+
+	timer {
+		compatible = "arm,armv8-timer";
+		interrupts = <GIC_PPI 13
+			(GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
+			     <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)>;
+	};
+
+	hosc: hosc {
+		compatible = "fixed-clock";
+		clock-frequency = <24000000>;
+		#clock-cells = <0>;
+	};
+
+	soc {
+		compatible = "simple-bus";
+		#address-cells = <2>;
+		#size-cells = <2>;
+		ranges;
+
+		gic: interrupt-controller at e00f1000 {
+			compatible = "arm,gic-400";
+			reg = <0x0 0xe00f1000 0x0 0x1000>,
+			      <0x0 0xe00f2000 0x0 0x2000>,
+			      <0x0 0xe00f4000 0x0 0x2000>,
+			      <0x0 0xe00f6000 0x0 0x2000>;
+			interrupts = <GIC_PPI 9 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>;
+			interrupt-controller;
+			#interrupt-cells = <3>;
+		};
+
+		uart0: serial at e0120000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe0120000 0x0 0x2000>;
+			interrupts = <GIC_SPI 29 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		uart1: serial at e0122000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe0122000 0x0 0x2000>;
+			interrupts = <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		uart2: serial at e0124000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe0124000 0x0 0x2000>;
+			interrupts = <GIC_SPI 31 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		uart3: serial at e0126000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe0126000 0x0 0x2000>;
+			interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		uart4: serial at e0128000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe0128000 0x0 0x2000>;
+			interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		uart5: serial at e012a000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe012a000 0x0 0x2000>;
+			interrupts = <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		uart6: serial at e012c000 {
+			compatible = "actions,s900-uart", "actions,owl-uart";
+			reg = <0x0 0xe012c000 0x0 0x2000>;
+			interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>;
+			status = "disabled";
+		};
+
+		timer: timer at e024c000 {
+			compatible = "actions,s700-timer";
+			reg = <0x0 0xe024c000 0x0 0x4000>;
+			interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>;
+			interrupt-names = "timer1";
+		};
+	};
+};
-- 
2.13.5

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7
  2017-09-11  2:22 ` [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7 Andreas Färber
@ 2017-09-18 20:45   ` Rob Herring
  0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2017-09-18 20:45 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Sep 11, 2017 at 04:22:38AM +0200, Andreas F?rber wrote:
> Document the Actions Semi S700 SoC and the Cubietech CubieBoard7.
> 
> Signed-off-by: Andreas F?rber <afaerber@suse.de>
> ---
>  Documentation/devicetree/bindings/arm/actions.txt | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 2/4] dt-bindings: timer: Add Actions Semi S700
  2017-09-11  2:22 ` [PATCH 2/4] dt-bindings: timer: Add Actions Semi S700 Andreas Färber
@ 2017-09-18 20:59   ` Rob Herring
  0 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2017-09-18 20:59 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Sep 11, 2017 at 04:22:39AM +0200, Andreas F?rber wrote:
> Define a compatible string for the Actions Semi S700 SoC timer.
> 
> Signed-off-by: Andreas F?rber <afaerber@suse.de>
> ---
>  Documentation/devicetree/bindings/timer/actions,owl-timer.txt | 1 +
>  1 file changed, 1 insertion(+)

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support
  2017-09-11  2:22 [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Andreas Färber
                   ` (3 preceding siblings ...)
  2017-09-11  2:22 ` [PATCH 4/4] arm64: dts: actions: Add S700 and CubieBoard7 Andreas Färber
@ 2017-09-18 21:50 ` Daniel Lezcano
  2017-09-19  5:08   ` Andreas Färber
  4 siblings, 1 reply; 12+ messages in thread
From: Daniel Lezcano @ 2017-09-18 21:50 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Andreas,


On 11/09/2017 04:22, Andreas F?rber wrote:
> Hello,
> 
> This series prepares the Actions Semi S700 SoC and Cubietech CubieBoard7.
> It boots equally far as the S900 based Bubblegum-96 these patches are based on,
> i.e. into an initrd with serial and all four cores up.
> 
> I have not yet managed to download the CubieBoard7 sources from Baidu, and the
> Mega download has not yet been fixed, so this is based on the S700 datasheet
> and /proc/device-tree/ from the preinstalled Android.
> 
> Lacking sources and instructions to replace the bootloader on eMMC or to try
> booting from SD, the only working way I've found to boot mainline kernels
> appears to be booting into Android, then issuing "reboot bootloader". U-Boot
> is lacking the saveenv command, so it wasn't possible to change the bootdelay
> to facilitate this.
> 
> Cf. https://en.opensuse.org/HCL:CubieBoard7
> 
> Work branch:
> https://github.com/afaerber/linux/commits/bg96-next
> 
> Have a lot of fun!
> 
> Cheers,
> Andreas
> 
> Cc: Ahha Lee <lee@cubietech.com>
> Cc: support at cubietech.com
> 
> Cc: devicetree at vger.kernel.org
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> 
> Andreas F?rber (4):
>   dt-bindings: arm: actions: Add S700 and CubieBoard7
>   dt-bindings: timer: Add Actions Semi S700
>   clocksource: owl: Prepare S700
>   arm64: dts: actions: Add S700 and CubieBoard7
> 
>  Documentation/devicetree/bindings/arm/actions.txt  |  15 ++
>  .../bindings/timer/actions,owl-timer.txt           |   1 +

Usually sub system maintainers are also Cc'ed with Dt bindings
definitions and usage.

>  arch/arm64/boot/dts/actions/Makefile               |   2 +
>  arch/arm64/boot/dts/actions/s700-cubieboard7.dts   |  47 ++++++
>  arch/arm64/boot/dts/actions/s700.dtsi              | 164 +++++++++++++++++++++
>  drivers/clocksource/owl-timer.c                    |   1 +
>  6 files changed, 230 insertions(+)
>  create mode 100644 arch/arm64/boot/dts/actions/s700-cubieboard7.dts
>  create mode 100644 arch/arm64/boot/dts/actions/s700.dtsi
> 


-- 
 <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 3/4] clocksource: owl: Prepare S700
  2017-09-11  2:22 ` [PATCH 3/4] clocksource: owl: Prepare S700 Andreas Färber
@ 2017-09-18 21:52   ` Daniel Lezcano
  0 siblings, 0 replies; 12+ messages in thread
From: Daniel Lezcano @ 2017-09-18 21:52 UTC (permalink / raw)
  To: linux-arm-kernel

On 11/09/2017 04:22, Andreas F?rber wrote:
> S700 has two 2Hz timers like S500, and four TIMx timers like S900.
> 
> Signed-off-by: Andreas F?rber <afaerber@suse.de>
> ---
>  drivers/clocksource/owl-timer.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/clocksource/owl-timer.c b/drivers/clocksource/owl-timer.c
> index d19c53c11094..d5f3f72c10ee 100644
> --- a/drivers/clocksource/owl-timer.c
> +++ b/drivers/clocksource/owl-timer.c
> @@ -169,4 +169,5 @@ static int __init owl_timer_init(struct device_node *node)
>  	return 0;
>  }
>  CLOCKSOURCE_OF_DECLARE(owl_s500, "actions,s500-timer", owl_timer_init);
> +CLOCKSOURCE_OF_DECLARE(owl_s700, "actions,s700-timer", owl_timer_init);
>  CLOCKSOURCE_OF_DECLARE(owl_s900, "actions,s900-timer", owl_timer_init);

Can you take the opportunity to fix the deprecated
CLOCKSOURCE_OF_DECLARE by replacing it by TIMER_OF_DECLARE ?

Thanks.

  -- Daniel


-- 
 <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support
  2017-09-18 21:50 ` [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Daniel Lezcano
@ 2017-09-19  5:08   ` Andreas Färber
  2017-09-19  5:37     ` Daniel Lezcano
  2017-09-19  6:12     ` Daniel Lezcano
  0 siblings, 2 replies; 12+ messages in thread
From: Andreas Färber @ 2017-09-19  5:08 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Daniel,

Am 18.09.2017 um 23:50 schrieb Daniel Lezcano:
> On 11/09/2017 04:22, Andreas F?rber wrote:
>> Cc: devicetree at vger.kernel.org
>> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
>> Cc: Thomas Gleixner <tglx@linutronix.de>
>>
>> Andreas F?rber (4):
>>   dt-bindings: arm: actions: Add S700 and CubieBoard7
>>   dt-bindings: timer: Add Actions Semi S700
>>   clocksource: owl: Prepare S700
>>   arm64: dts: actions: Add S700 and CubieBoard7
>>
>>  Documentation/devicetree/bindings/arm/actions.txt  |  15 ++
>>  .../bindings/timer/actions,owl-timer.txt           |   1 +
> 
> Usually sub system maintainers are also Cc'ed with Dt bindings
> definitions and usage.

Your MAINTAINERS entry is lacking an entry for the bindings. Can you
please fix that in linux-next? Then the next patches should reach you.

Actually I'm even surprised that it works for the drivers - I would
expect a trailing "/" for the directory (see line 95ff).

Regards,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N?rnberg)

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support
  2017-09-19  5:08   ` Andreas Färber
@ 2017-09-19  5:37     ` Daniel Lezcano
  2017-09-19  6:12     ` Daniel Lezcano
  1 sibling, 0 replies; 12+ messages in thread
From: Daniel Lezcano @ 2017-09-19  5:37 UTC (permalink / raw)
  To: linux-arm-kernel

On 19/09/2017 07:08, Andreas F?rber wrote:
> Hi Daniel,
> 
> Am 18.09.2017 um 23:50 schrieb Daniel Lezcano:
>> On 11/09/2017 04:22, Andreas F?rber wrote:
>>> Cc: devicetree at vger.kernel.org
>>> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
>>> Cc: Thomas Gleixner <tglx@linutronix.de>
>>>
>>> Andreas F?rber (4):
>>>   dt-bindings: arm: actions: Add S700 and CubieBoard7
>>>   dt-bindings: timer: Add Actions Semi S700
>>>   clocksource: owl: Prepare S700
>>>   arm64: dts: actions: Add S700 and CubieBoard7
>>>
>>>  Documentation/devicetree/bindings/arm/actions.txt  |  15 ++
>>>  .../bindings/timer/actions,owl-timer.txt           |   1 +
>>
>> Usually sub system maintainers are also Cc'ed with Dt bindings
>> definitions and usage.
> 
> Your MAINTAINERS entry is lacking an entry for the bindings. Can you
> please fix that in linux-next? Then the next patches should reach you.
> 
> Actually I'm even surprised that it works for the drivers - I would
> expect a trailing "/" for the directory (see line 95ff).

Right, thanks for pointing this out, I will fix it.

  -- Daniel


-- 
 <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support
  2017-09-19  5:08   ` Andreas Färber
  2017-09-19  5:37     ` Daniel Lezcano
@ 2017-09-19  6:12     ` Daniel Lezcano
  1 sibling, 0 replies; 12+ messages in thread
From: Daniel Lezcano @ 2017-09-19  6:12 UTC (permalink / raw)
  To: linux-arm-kernel

On 19/09/2017 07:08, Andreas F?rber wrote:
> Hi Daniel,
> 
> Am 18.09.2017 um 23:50 schrieb Daniel Lezcano:
>> On 11/09/2017 04:22, Andreas F?rber wrote:
>>> Cc: devicetree at vger.kernel.org
>>> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
>>> Cc: Thomas Gleixner <tglx@linutronix.de>
>>>
>>> Andreas F?rber (4):
>>>   dt-bindings: arm: actions: Add S700 and CubieBoard7
>>>   dt-bindings: timer: Add Actions Semi S700
>>>   clocksource: owl: Prepare S700
>>>   arm64: dts: actions: Add S700 and CubieBoard7
>>>
>>>  Documentation/devicetree/bindings/arm/actions.txt  |  15 ++
>>>  .../bindings/timer/actions,owl-timer.txt           |   1 +
>>
>> Usually sub system maintainers are also Cc'ed with Dt bindings
>> definitions and usage.
> 
> Your MAINTAINERS entry is lacking an entry for the bindings. Can you
> please fix that in linux-next? Then the next patches should reach you.
> 
> Actually I'm even surprised that it works for the drivers - I would
> expect a trailing "/" for the directory (see line 95ff).

Done.

https://git.linaro.org/people/daniel.lezcano/linux.git/commit/?h=clockevents/next&id=acdbb48efecb03bc5a6e8e82447208fdd342eb89

Should hit linux-next soon.


-- 
 <http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2017-09-19  6:12 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-11  2:22 [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Andreas Färber
2017-09-11  2:22 ` [PATCH 1/4] dt-bindings: arm: actions: Add S700 and CubieBoard7 Andreas Färber
2017-09-18 20:45   ` Rob Herring
2017-09-11  2:22 ` [PATCH 2/4] dt-bindings: timer: Add Actions Semi S700 Andreas Färber
2017-09-18 20:59   ` Rob Herring
2017-09-11  2:22 ` [PATCH 3/4] clocksource: owl: Prepare S700 Andreas Färber
2017-09-18 21:52   ` Daniel Lezcano
2017-09-11  2:22 ` [PATCH 4/4] arm64: dts: actions: Add S700 and CubieBoard7 Andreas Färber
2017-09-18 21:50 ` [PATCH 0/4] arm64: Add initial Actions Semi S700 and CubieBoard7 support Daniel Lezcano
2017-09-19  5:08   ` Andreas Färber
2017-09-19  5:37     ` Daniel Lezcano
2017-09-19  6:12     ` Daniel Lezcano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox