Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1412572 > unrolled thread

[PATCH 1/6] clk: Add missing clk_get_sys() stub

Started byDaniel Lezcano <daniel.lezcano@linaro.org>
First post2016-06-02 23:50 +0200
Last post2016-06-06 09:20 +0200
Articles 6 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 1/6] clk: Add missing clk_get_sys() stub Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-06-02 23:50 +0200
    [PATCH 2/6] clocksource/drivers/bcm_kona: Remove useless header <asm/mach/time.h> Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-06-02 23:50 +0200
    [PATCH 4/6] clocksource/drivers/armv7m_systick: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-06-02 23:50 +0200
      Re: [PATCH 4/6] clocksource/drivers/armv7m_systick: Add the  COMPILE_TEST option kbuild test robot <lkp@intel.com> - 2016-06-03 07:20 +0200
    [PATCH 6/6] clocksource/drivers/clps_711x: Add the COMPILE_TEST option Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-06-02 23:50 +0200
    Re: [PATCH 1/6] clk: Add missing clk_get_sys() stub Daniel Lezcano <daniel.lezcano@linaro.org> - 2016-06-06 09:20 +0200

#1412572 — [PATCH 1/6] clk: Add missing clk_get_sys() stub

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2016-06-02 23:50 +0200
Subject[PATCH 1/6] clk: Add missing clk_get_sys() stub
Message-ID<rFIF3-2WQ-3@gated-at.bofh.it>
When compiling with the COMPILE_TEST option set, the clps711x does not
compile because of the clk_get_sys() noop stub missing.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 include/linux/clk.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/include/linux/clk.h b/include/linux/clk.h
index 0df4a51..834179f 100644
--- a/include/linux/clk.h
+++ b/include/linux/clk.h
@@ -461,6 +461,10 @@ static inline struct clk *clk_get_parent(struct clk *clk)
 	return NULL;
 }
 
+static inline struct clk *clk_get_sys(const char *dev_id, const char *con_id)
+{
+	return NULL;
+}
 #endif
 
 /* clk_prepare_enable helps cases using clk_enable in non-atomic context. */
-- 
1.9.1

[toc] | [next] | [standalone]

#1412574 — [PATCH 2/6] clocksource/drivers/bcm_kona: Remove useless header <asm/mach/time.h>

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2016-06-02 23:50 +0200
Subject[PATCH 2/6] clocksource/drivers/bcm_kona: Remove useless header <asm/mach/time.h>
Message-ID<rFIF3-2WQ-15@gated-at.bofh.it>
In reply to#1412572
The driver includes the <asm/mach/time.h> header but it is pointless.

Remove it.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/clocksource/bcm_kona_timer.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/clocksource/bcm_kona_timer.c b/drivers/clocksource/bcm_kona_timer.c
index e717e87..fd11d59 100644
--- a/drivers/clocksource/bcm_kona_timer.c
+++ b/drivers/clocksource/bcm_kona_timer.c
@@ -20,7 +20,6 @@
 #include <linux/clk.h>
 
 #include <linux/io.h>
-#include <asm/mach/time.h>
 
 #include <linux/of.h>
 #include <linux/of_address.h>
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1412575 — [PATCH 4/6] clocksource/drivers/armv7m_systick: Add the COMPILE_TEST option

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2016-06-02 23:50 +0200
Subject[PATCH 4/6] clocksource/drivers/armv7m_systick: Add the COMPILE_TEST option
Message-ID<rFIF3-2WQ-21@gated-at.bofh.it>
In reply to#1412572
In order to increase the compilation test coverage, add the COMPILE_TEST
so the driver can be compiled even if it does not belong to the platform
or the architecture.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 drivers/clocksource/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
index 34053ab..a1e0cf9 100644
--- a/drivers/clocksource/Kconfig
+++ b/drivers/clocksource/Kconfig
@@ -240,7 +240,7 @@ config CLKSRC_ARM_GLOBAL_TIMER_SCHED_CLOCK
 	 Use ARM global timer clock source as sched_clock
 
 config ARMV7M_SYSTICK
-	bool
+	bool "Support for the ARMv7M system time" if COMPILE_TEST
 	select CLKSRC_OF if OF
 	select CLKSRC_MMIO
 	help
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1412806 — Re: [PATCH 4/6] clocksource/drivers/armv7m_systick: Add the COMPILE_TEST option

Fromkbuild test robot <lkp@intel.com>
Date2016-06-03 07:20 +0200
SubjectRe: [PATCH 4/6] clocksource/drivers/armv7m_systick: Add the COMPILE_TEST option
Message-ID<rFPGx-7sZ-11@gated-at.bofh.it>
In reply to#1412575

[Multipart message — attachments visible in raw view] — view raw

Hi,

[auto build test ERROR on tip/timers/core]
[also build test ERROR on v4.7-rc1 next-20160602]
[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/Daniel-Lezcano/clk-Add-missing-clk_get_sys-stub/20160603-054643
config: um-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
        # save the attached .config to linux build tree
        make ARCH=um 

All errors (new ones prefixed by >>):

   drivers/clocksource/armv7m_systick.c: In function 'system_timer_of_register':
>> drivers/clocksource/armv7m_systick.c:74:2: error: implicit declaration of function 'iounmap' [-Werror=implicit-function-declaration]
     iounmap(base);
     ^~~~~~~
   cc1: some warnings being treated as errors

vim +/iounmap +74 drivers/clocksource/armv7m_systick.c

4958ebb3 Maxime Coquelin 2015-05-09  58  		pr_err("failed to init clocksource (%d)\n", ret);
4958ebb3 Maxime Coquelin 2015-05-09  59  		if (clk)
4958ebb3 Maxime Coquelin 2015-05-09  60  			goto out_clk_disable;
4958ebb3 Maxime Coquelin 2015-05-09  61  		else
4958ebb3 Maxime Coquelin 2015-05-09  62  			goto out_unmap;
4958ebb3 Maxime Coquelin 2015-05-09  63  	}
4958ebb3 Maxime Coquelin 2015-05-09  64  
4958ebb3 Maxime Coquelin 2015-05-09  65  	pr_info("ARM System timer initialized as clocksource\n");
4958ebb3 Maxime Coquelin 2015-05-09  66  
4958ebb3 Maxime Coquelin 2015-05-09  67  	return;
4958ebb3 Maxime Coquelin 2015-05-09  68  
4958ebb3 Maxime Coquelin 2015-05-09  69  out_clk_disable:
4958ebb3 Maxime Coquelin 2015-05-09  70  	clk_disable_unprepare(clk);
4958ebb3 Maxime Coquelin 2015-05-09  71  out_clk_put:
4958ebb3 Maxime Coquelin 2015-05-09  72  	clk_put(clk);
4958ebb3 Maxime Coquelin 2015-05-09  73  out_unmap:
4958ebb3 Maxime Coquelin 2015-05-09 @74  	iounmap(base);
4958ebb3 Maxime Coquelin 2015-05-09  75  	pr_warn("ARM System timer register failed (%d)\n", ret);
4958ebb3 Maxime Coquelin 2015-05-09  76  }
4958ebb3 Maxime Coquelin 2015-05-09  77  
4958ebb3 Maxime Coquelin 2015-05-09  78  CLOCKSOURCE_OF_DECLARE(arm_systick, "arm,armv7m-systick",
4958ebb3 Maxime Coquelin 2015-05-09  79  			system_timer_of_register);

:::::: The code at line 74 was first introduced by commit
:::::: 4958ebb3d027886c46b936453745dba59b09c578 clocksource/drivers/armv7m_systick: Add ARM System timer driver

:::::: TO: Maxime Coquelin <mcoquelin.stm32@gmail.com>
:::::: CC: Daniel Lezcano <daniel.lezcano@linaro.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[toc] | [prev] | [next] | [standalone]

#1412577 — [PATCH 6/6] clocksource/drivers/clps_711x: Add the COMPILE_TEST option

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2016-06-02 23:50 +0200
Subject[PATCH 6/6] clocksource/drivers/clps_711x: Add the COMPILE_TEST option
Message-ID<rFIF3-2WQ-13@gated-at.bofh.it>
In reply to#1412572
Change the Kconfig option logic to fullfil with the current approach.

A new Kconfig option is added, CONFIG_BCM_KONA_TIMER and is selected by the
platform. Then the clocksource's Kconfig is changed to make this option
selectable by the user if the COMPILE_TEST option is set. Otherwise, it is
up to the platform's Kconfig to select the timer.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
---
 arch/arm/Kconfig             | 2 +-
 drivers/clocksource/Kconfig  | 7 +++++++
 drivers/clocksource/Makefile | 2 +-
 3 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index cdfa6c2..0c1f546 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -356,10 +356,10 @@ config ARCH_CLPS711X
 	bool "Cirrus Logic CLPS711x/EP721x/EP731x-based"
 	select ARCH_REQUIRE_GPIOLIB
 	select AUTO_ZRELADDR
-	select CLKSRC_MMIO
 	select COMMON_CLK
 	select CPU_ARM720T
 	select GENERIC_CLOCKEVENTS
+	select CLPS711X_TIMER
 	select MFD_SYSCON
 	select SOC_BUS
 	help
diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
index 6714a22..e92a814 100644
--- a/drivers/clocksource/Kconfig
+++ b/drivers/clocksource/Kconfig
@@ -155,6 +155,13 @@ config CLKSRC_DBX500_PRCMU
 	help
 	  Use the always on PRCMU Timer as clocksource
 
+config CLPS711X_TIMER
+	bool "Cirrus logic timer driver" if COMPILE_TEST
+	depends on GENERIC_CLOCKEVENTS
+	select CLKSRC_MMIO
+	help
+	  Enables support for the Cirrus Logic PS711 timer.
+
 config CLKSRC_DBX500_PRCMU_SCHED_CLOCK
 	bool "Clocksource PRCMU Timer sched_clock"
 	depends on (CLKSRC_DBX500_PRCMU && !CLKSRC_NOMADIK_MTU_SCHED_CLOCK)
diff --git a/drivers/clocksource/Makefile b/drivers/clocksource/Makefile
index 662a87c..ae6c09f 100644
--- a/drivers/clocksource/Makefile
+++ b/drivers/clocksource/Makefile
@@ -20,7 +20,7 @@ obj-$(CONFIG_CLKSRC_DBX500_PRCMU)	+= clksrc-dbx500-prcmu.o
 obj-$(CONFIG_ARMADA_370_XP_TIMER)	+= time-armada-370-xp.o
 obj-$(CONFIG_ORION_TIMER)	+= time-orion.o
 obj-$(CONFIG_BCM2835_TIMER)	+= bcm2835_timer.o
-obj-$(CONFIG_ARCH_CLPS711X)	+= clps711x-timer.o
+obj-$(CONFIG_CLPS711X_TIMER)	+= clps711x-timer.o
 obj-$(CONFIG_ARCH_ATLAS7)	+= timer-atlas7.o
 obj-$(CONFIG_ARCH_MOXART)	+= moxart_timer.o
 obj-$(CONFIG_ARCH_MXS)		+= mxs_timer.o
-- 
1.9.1

[toc] | [prev] | [next] | [standalone]

#1414692

FromDaniel Lezcano <daniel.lezcano@linaro.org>
Date2016-06-06 09:20 +0200
Message-ID<rGWZj-1Il-1@gated-at.bofh.it>
In reply to#1412572
On 06/02/2016 11:41 PM, Daniel Lezcano wrote:
> When compiling with the COMPILE_TEST option set, the clps711x does not
> compile because of the clk_get_sys() noop stub missing.
>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>

Added missing Cc [Mike and Stephen]

> ---
>   include/linux/clk.h | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/include/linux/clk.h b/include/linux/clk.h
> index 0df4a51..834179f 100644
> --- a/include/linux/clk.h
> +++ b/include/linux/clk.h
> @@ -461,6 +461,10 @@ static inline struct clk *clk_get_parent(struct clk *clk)
>   	return NULL;
>   }
>
> +static inline struct clk *clk_get_sys(const char *dev_id, const char *con_id)
> +{
> +	return NULL;
> +}
>   #endif
>
>   /* clk_prepare_enable helps cases using clk_enable in non-atomic context. */
>


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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web