diff mbox

[v15,01/13] clocksource/drivers/arm_arch_timer: Move enums and defines to header file

Message ID 1479215615-26950-2-git-send-email-fu.wei@linaro.org
State Superseded
Headers show

Commit Message

Fu Wei Fu Nov. 15, 2016, 1:13 p.m. UTC 
From: Fu Wei <fu.wei@linaro.org>


To support the arm_arch_timer via ACPI we need to share defines and enums
between the driver and the ACPI parser code.

Split out the relevant defines and enums into arm_arch_timer.h, and
change "enum ppi_nr" to "enum arch_timer_ppi_nr" to avoid the potential
name clashes.
No functional change.

Signed-off-by: Fu Wei <fu.wei@linaro.org>

Acked-by: Mark Rutland <mark.rutland@arm.com>

---
 drivers/clocksource/arm_arch_timer.c | 11 -----------
 include/clocksource/arm_arch_timer.h | 12 ++++++++++++
 2 files changed, 12 insertions(+), 11 deletions(-)

-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Fu Wei Fu Nov. 16, 2016, 2:15 a.m. UTC | #1 
Hi Marc,

On 16 November 2016 at 03:23, kbuild test robot <lkp@intel.com> wrote:
> Hi Fu,

>

> [auto build test ERROR on tip/timers/core]

> [also build test ERROR on v4.9-rc5 next-20161115]

> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

>

> url:    https://github.com/0day-ci/linux/commits/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161116-004444

> config: arm-hisi_defconfig (attached as .config)

> compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705

> reproduce:

>         wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross

>         chmod +x ~/bin/make.cross

>         # save the attached .config to linux build tree

>         make.cross ARCH=arm

>

> Note: the linux-review/fu-wei-linaro-org/acpi-clocksource-add-GTDT-driver-and-GTDT-support-in-arm_arch_timer/20161116-004444 HEAD 47004b0566d2fd65c05836078319b02c5af2a1fc builds fine.

>       It only hurts bisectibility.

>

> All error/warnings (new ones prefixed by >>):

>

>>> drivers/clocksource/arm_arch_timer.c:70:13: error: variable 'arch_timer_uses_ppi' has initializer but incomplete type

>     static enum ppi_nr arch_timer_uses_ppi = VIRT_PPI;


I have figured out this,  I should move a change from patch 4/15 to
here to avoid the problem.

it should be static enum arch_timer_ppi_nr arch_timer_uses_ppi = VIRT_PPI;
will improve this

Thanks

>                 ^~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function '__arch_timer_setup':

>>> drivers/clocksource/arm_arch_timer.c:358:3: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>       clk->irq = arch_timer_ppi[arch_timer_uses_ppi];

>       ^~~

>    drivers/clocksource/arm_arch_timer.c:359:3: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>       switch (arch_timer_uses_ppi) {

>       ^~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_has_nonsecure_ppi':

>    drivers/clocksource/arm_arch_timer.c:447:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      return (arch_timer_uses_ppi == PHYS_SECURE_PPI &&

>      ^~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_starting_cpu':

>    drivers/clocksource/arm_arch_timer.c:471:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      flags = check_ppi_trigger(arch_timer_ppi[arch_timer_uses_ppi]);

>      ^~~~~

>    drivers/clocksource/arm_arch_timer.c:472:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      enable_percpu_irq(arch_timer_ppi[arch_timer_uses_ppi], flags);

>      ^~~~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_banner':

>    drivers/clocksource/arm_arch_timer.c:512:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      pr_info("Architected %s%s%s timer(s) running at %lu.%02luMHz (%s%s%s).\n",

>      ^~~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_counter_register':

>    drivers/clocksource/arm_arch_timer.c:589:3: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>       if (IS_ENABLED(CONFIG_ARM64) || arch_timer_uses_ppi == VIRT_PPI)

>       ^~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_stop':

>    drivers/clocksource/arm_arch_timer.c:624:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      disable_percpu_irq(arch_timer_ppi[arch_timer_uses_ppi]);

>      ^~~~~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_register':

>    drivers/clocksource/arm_arch_timer.c:687:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      ppi = arch_timer_ppi[arch_timer_uses_ppi];

>      ^~~

>    drivers/clocksource/arm_arch_timer.c:688:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      switch (arch_timer_uses_ppi) {

>      ^~~~~~

>    drivers/clocksource/arm_arch_timer.c:737:2: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>      free_percpu_irq(arch_timer_ppi[arch_timer_uses_ppi], arch_timer_evt);

>      ^~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_init':

>    drivers/clocksource/arm_arch_timer.c:838:4: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>        arch_timer_uses_ppi = HYP_PPI;

>        ^~~~~~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c:841:4: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>        arch_timer_uses_ppi = PHYS_SECURE_PPI;

>        ^~~~~~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_of_init':

>    drivers/clocksource/arm_arch_timer.c:897:3: error: 'arch_timer_uses_ppi' has an incomplete type 'enum ppi_nr'

>       arch_timer_uses_ppi = PHYS_SECURE_PPI;

>       ^~~~~~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c: At top level:

>>> drivers/clocksource/arm_arch_timer.c:70:20: error: storage size of 'arch_timer_uses_ppi' isn't known

>     static enum ppi_nr arch_timer_uses_ppi = VIRT_PPI;

>                        ^~~~~~~~~~~~~~~~~~~

>    drivers/clocksource/arm_arch_timer.c: In function 'arch_timer_has_nonsecure_ppi':

>>> drivers/clocksource/arm_arch_timer.c:449:1: warning: control reaches end of non-void function [-Wreturn-type]

>     }

>     ^

>

> vim +/arch_timer_uses_ppi +70 drivers/clocksource/arm_arch_timer.c

>

> 220069945 Stephen Boyd      2013-07-18  64

> 8a4da6e36 Mark Rutland      2012-11-12  65  static u32 arch_timer_rate;

> 8a4da6e36 Mark Rutland      2012-11-12  66  static int arch_timer_ppi[MAX_TIMER_PPI];

> 8a4da6e36 Mark Rutland      2012-11-12  67

> 8a4da6e36 Mark Rutland      2012-11-12  68  static struct clock_event_device __percpu *arch_timer_evt;

> 8a4da6e36 Mark Rutland      2012-11-12  69

> f81f03fa2 Marc Zyngier      2014-02-20 @70  static enum ppi_nr arch_timer_uses_ppi = VIRT_PPI;

> 82a561941 Lorenzo Pieralisi 2014-04-08  71  static bool arch_timer_c3stop;

> 220069945 Stephen Boyd      2013-07-18  72  static bool arch_timer_mem_use_virtual;

> 8a4da6e36 Mark Rutland      2012-11-12  73

>

> :::::: The code at line 70 was first introduced by commit

> :::::: f81f03fa231a8c3aacd580759e73b9238b92ba29 arm64: Allow the arch timer to use the HYP timer

>

> :::::: TO: Marc Zyngier <marc.zyngier@arm.com>

> :::::: CC: Marc Zyngier <marc.zyngier@arm.com>

>

> ---

> 0-DAY kernel test infrastructure                Open Source Technology Center

> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation




-- 
Best regards,

Fu Wei
Software Engineer
Red Hat
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
index 73c487d..9826f06 100644
--- a/drivers/clocksource/arm_arch_timer.c
+++ b/drivers/clocksource/arm_arch_timer.c
@@ -51,8 +51,6 @@ 
 #define CNTV_TVAL	0x38
 #define CNTV_CTL	0x3c
 
-#define ARCH_CP15_TIMER	BIT(0)
-#define ARCH_MEM_TIMER	BIT(1)
 static unsigned arch_timers_present __initdata;
 
 static void __iomem *arch_counter_base;
@@ -65,15 +63,6 @@  struct arch_timer {
 #define to_arch_timer(e) container_of(e, struct arch_timer, evt)
 
 static u32 arch_timer_rate;
-
-enum ppi_nr {
-	PHYS_SECURE_PPI,
-	PHYS_NONSECURE_PPI,
-	VIRT_PPI,
-	HYP_PPI,
-	MAX_TIMER_PPI
-};
-
 static int arch_timer_ppi[MAX_TIMER_PPI];
 
 static struct clock_event_device __percpu *arch_timer_evt;
diff --git a/include/clocksource/arm_arch_timer.h b/include/clocksource/arm_arch_timer.h
index caedb74..557f869 100644
--- a/include/clocksource/arm_arch_timer.h
+++ b/include/clocksource/arm_arch_timer.h
@@ -16,9 +16,13 @@ 
 #ifndef __CLKSOURCE_ARM_ARCH_TIMER_H
 #define __CLKSOURCE_ARM_ARCH_TIMER_H
 
+#include <linux/bitops.h>
 #include <linux/timecounter.h>
 #include <linux/types.h>
 
+#define ARCH_CP15_TIMER			BIT(0)
+#define ARCH_MEM_TIMER			BIT(1)
+
 #define ARCH_TIMER_CTRL_ENABLE		(1 << 0)
 #define ARCH_TIMER_CTRL_IT_MASK		(1 << 1)
 #define ARCH_TIMER_CTRL_IT_STAT		(1 << 2)
@@ -34,6 +38,14 @@  enum arch_timer_reg {
 	ARCH_TIMER_REG_TVAL,
 };
 
+enum arch_timer_ppi_nr {
+	PHYS_SECURE_PPI,
+	PHYS_NONSECURE_PPI,
+	VIRT_PPI,
+	HYP_PPI,
+	MAX_TIMER_PPI
+};
+
 #define ARCH_TIMER_PHYS_ACCESS		0
 #define ARCH_TIMER_VIRT_ACCESS		1
 #define ARCH_TIMER_MEM_PHYS_ACCESS	2