Re: [PATCH 2/2] blackfin-cpufreq: Mark cpu_set_cclk() as static

2016-01-02 Thread Rafael J. Wysocki
On Tuesday, December 22, 2015 11:05:43 AM Viresh Kumar wrote:
> On 21-12-15, 22:32, SF Markus Elfring wrote:
> > From: Markus Elfring 
> > Date: Mon, 21 Dec 2015 22:12:26 +0100
> > 
> > The cpu_set_cclk() function was only used in a single source file so far.
> > Indicate this setting also by the corresponding linkage specifier.
> > 
> > Signed-off-by: Markus Elfring 
> > Acked-by: Viresh Kumar 
> 
> Well, I never Acked this patch as such and you shouldn't have
> mentioned it by yourself. But it looks fine otherwise and so
> 
> Acked-by: Viresh Kumar 

Series applied, thanks!

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/2] blackfin-cpufreq: Mark cpu_set_cclk() as static

2016-01-02 Thread Rafael J. Wysocki
On Tuesday, December 22, 2015 11:05:43 AM Viresh Kumar wrote:
> On 21-12-15, 22:32, SF Markus Elfring wrote:
> > From: Markus Elfring 
> > Date: Mon, 21 Dec 2015 22:12:26 +0100
> > 
> > The cpu_set_cclk() function was only used in a single source file so far.
> > Indicate this setting also by the corresponding linkage specifier.
> > 
> > Signed-off-by: Markus Elfring 
> > Acked-by: Viresh Kumar 
> 
> Well, I never Acked this patch as such and you shouldn't have
> mentioned it by yourself. But it looks fine otherwise and so
> 
> Acked-by: Viresh Kumar 

Series applied, thanks!

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/2] blackfin-cpufreq: Mark cpu_set_cclk() as static

2015-12-21 Thread Viresh Kumar
On 21-12-15, 22:32, SF Markus Elfring wrote:
> From: Markus Elfring 
> Date: Mon, 21 Dec 2015 22:12:26 +0100
> 
> The cpu_set_cclk() function was only used in a single source file so far.
> Indicate this setting also by the corresponding linkage specifier.
> 
> Signed-off-by: Markus Elfring 
> Acked-by: Viresh Kumar 

Well, I never Acked this patch as such and you shouldn't have
mentioned it by yourself. But it looks fine otherwise and so

Acked-by: Viresh Kumar 

-- 
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 2/2] blackfin-cpufreq: Mark cpu_set_cclk() as static

2015-12-21 Thread SF Markus Elfring
From: Markus Elfring 
Date: Mon, 21 Dec 2015 22:12:26 +0100

The cpu_set_cclk() function was only used in a single source file so far.
Indicate this setting also by the corresponding linkage specifier.

Signed-off-by: Markus Elfring 
Acked-by: Viresh Kumar 
---
 drivers/cpufreq/blackfin-cpufreq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/cpufreq/blackfin-cpufreq.c 
b/drivers/cpufreq/blackfin-cpufreq.c
index 2a6f3ac..12e97d8 100644
--- a/drivers/cpufreq/blackfin-cpufreq.c
+++ b/drivers/cpufreq/blackfin-cpufreq.c
@@ -112,7 +112,7 @@ static unsigned int bfin_getfreq_khz(unsigned int cpu)
 }
 
 #ifdef CONFIG_BF60x
-int cpu_set_cclk(int cpu, unsigned long new)
+static int cpu_set_cclk(int cpu, unsigned long new)
 {
struct clk *clk;
int ret;
-- 
2.6.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 2/2] blackfin-cpufreq: Mark cpu_set_cclk() as static

2015-12-21 Thread SF Markus Elfring
From: Markus Elfring 
Date: Mon, 21 Dec 2015 22:12:26 +0100

The cpu_set_cclk() function was only used in a single source file so far.
Indicate this setting also by the corresponding linkage specifier.

Signed-off-by: Markus Elfring 
Acked-by: Viresh Kumar 
---
 drivers/cpufreq/blackfin-cpufreq.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/cpufreq/blackfin-cpufreq.c 
b/drivers/cpufreq/blackfin-cpufreq.c
index 2a6f3ac..12e97d8 100644
--- a/drivers/cpufreq/blackfin-cpufreq.c
+++ b/drivers/cpufreq/blackfin-cpufreq.c
@@ -112,7 +112,7 @@ static unsigned int bfin_getfreq_khz(unsigned int cpu)
 }
 
 #ifdef CONFIG_BF60x
-int cpu_set_cclk(int cpu, unsigned long new)
+static int cpu_set_cclk(int cpu, unsigned long new)
 {
struct clk *clk;
int ret;
-- 
2.6.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 2/2] blackfin-cpufreq: Mark cpu_set_cclk() as static

2015-12-21 Thread Viresh Kumar
On 21-12-15, 22:32, SF Markus Elfring wrote:
> From: Markus Elfring 
> Date: Mon, 21 Dec 2015 22:12:26 +0100
> 
> The cpu_set_cclk() function was only used in a single source file so far.
> Indicate this setting also by the corresponding linkage specifier.
> 
> Signed-off-by: Markus Elfring 
> Acked-by: Viresh Kumar 

Well, I never Acked this patch as such and you shouldn't have
mentioned it by yourself. But it looks fine otherwise and so

Acked-by: Viresh Kumar 

-- 
viresh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/