| Message ID | 20231215153706.2087883-1-gregory.clement@bootlin.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | cpufreq: armada-8k: Fix parameter type warning | expand |
On Fri, Dec 15, 2023 at 04:37:06PM +0100, Gregory CLEMENT wrote: > The second parameter of clk_get() is of type 'const char *', so use > NULL instead of the integer 0 to resolve a sparse warning: > > drivers/cpufreq/armada-8k-cpufreq.c:60:40: warning: Using plain integer as NULL pointer > drivers/cpufreq/armada-8k-cpufreq.c:168:40: warning: Using plain integer as NULL pointer > > Reported-by: kernel test robot <lkp@intel.com> > Closes: https://lore.kernel.org/oe-kbuild-all/202312141651.lCAXGAZ2-lkp@intel.com/ > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Andrew
On 15-12-23, 16:37, Gregory CLEMENT wrote: > The second parameter of clk_get() is of type 'const char *', so use > NULL instead of the integer 0 to resolve a sparse warning: > > drivers/cpufreq/armada-8k-cpufreq.c:60:40: warning: Using plain integer as NULL pointer > drivers/cpufreq/armada-8k-cpufreq.c:168:40: warning: Using plain integer as NULL pointer > > Reported-by: kernel test robot <lkp@intel.com> > Closes: https://lore.kernel.org/oe-kbuild-all/202312141651.lCAXGAZ2-lkp@intel.com/ > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > --- > drivers/cpufreq/armada-8k-cpufreq.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Applied. Thanks.
diff --git a/drivers/cpufreq/armada-8k-cpufreq.c b/drivers/cpufreq/armada-8k-cpufreq.c index 8afefdea4d80..ce5a5641b6dd 100644 --- a/drivers/cpufreq/armada-8k-cpufreq.c +++ b/drivers/cpufreq/armada-8k-cpufreq.c @@ -57,7 +57,7 @@ static void __init armada_8k_get_sharing_cpus(struct clk *cur_clk, continue; } - clk = clk_get(cpu_dev, 0); + clk = clk_get(cpu_dev, NULL); if (IS_ERR(clk)) { pr_warn("Cannot get clock for CPU %d\n", cpu); } else { @@ -165,7 +165,7 @@ static int __init armada_8k_cpufreq_init(void) continue; } - clk = clk_get(cpu_dev, 0); + clk = clk_get(cpu_dev, NULL); if (IS_ERR(clk)) { pr_err("Cannot get clock for CPU %d\n", cpu);
The second parameter of clk_get() is of type 'const char *', so use NULL instead of the integer 0 to resolve a sparse warning: drivers/cpufreq/armada-8k-cpufreq.c:60:40: warning: Using plain integer as NULL pointer drivers/cpufreq/armada-8k-cpufreq.c:168:40: warning: Using plain integer as NULL pointer Reported-by: kernel test robot <lkp@intel.com> Closes: https://lore.kernel.org/oe-kbuild-all/202312141651.lCAXGAZ2-lkp@intel.com/ Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> --- drivers/cpufreq/armada-8k-cpufreq.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)