Re: [PATCH v15 2/3] cpufreq: Add of_perf_domain_get_sharing_cpumask

From: kernel test robot
Date: Sat Sep 04 2021 - 18:33:43 EST

Next message: Josh Poimboeuf: "Re: [PATCH v2 bpf-next 1/3] perf: enable branch record for software events"
Previous message: Fabio M. De Francesco: "Re: [PATCH v2 0/3] staging: r8188eu: Shorten and simplify calls chain"
In reply to: Hector Yuan: "[PATCH v15 2/3] cpufreq: Add of_perf_domain_get_sharing_cpumask"
Next in thread: Viresh Kumar: "Re: [PATCH v15 2/3] cpufreq: Add of_perf_domain_get_sharing_cpumask"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Hector,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on soc/for-next]
[also build test WARNING on robh/for-next v5.14]
[cannot apply to pm/linux-next next-20210903]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url: https://github.com/0day-ci/linux/commits/Hector-Yuan/dt-bindings-cpufreq-add-bindings-for-MediaTek-cpufreq-HW/20210903-174004
base: https://git.kernel.org/pub/scm/linux/kernel/git/soc/soc.git for-next
config: i386-randconfig-r002-20210903 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 1104e3258b5064e7110cc297e2cec60ac9acfc0a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/0day-ci/linux/commit/47a2ae7200a52148136871a5a9c85eadd4c6c620
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Hector-Yuan/dt-bindings-cpufreq-add-bindings-for-MediaTek-cpufreq-HW/20210903-174004
git checkout 47a2ae7200a52148136871a5a9c85eadd4c6c620
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=i386

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All warnings (new ones prefixed by >>):

static inline void register_hugetlbfs_with_node(node_registration_func_t reg,
^
include/linux/node.h:177:32: warning: unused parameter 'unreg' [-Wunused-parameter]
node_registration_func_t unreg)
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:6:
In file included from include/linux/cpufreq.h:17:
In file included from include/linux/of_device.h:5:
In file included from include/linux/cpu.h:20:
include/linux/cpuhotplug.h:417:55: warning: unused parameter 'state' [-Wunused-parameter]
static inline void cpuhp_online_idle(enum cpuhp_state state) { }
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:6:
In file included from include/linux/cpufreq.h:17:
In file included from include/linux/of_device.h:5:
include/linux/cpu.h:111:40: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline int add_cpu(unsigned int cpu) { return 0;}
^
include/linux/cpu.h:142:43: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline int remove_cpu(unsigned int cpu) { return -EPERM; }
^
include/linux/cpu.h:143:59: warning: unused parameter 'primary_cpu' [-Wunused-parameter]
static inline void smp_shutdown_nonboot_cpus(unsigned int primary_cpu) { }
^
include/linux/cpu.h:226:41: warning: unused parameter 'force' [-Wunused-parameter]
static inline void cpu_smt_disable(bool force) { }
^
include/linux/cpu.h:230:60: warning: unused parameter 'ctrlval' [-Wunused-parameter]
static inline int cpuhp_smt_disable(enum cpuhp_smt_control ctrlval) { return 0; }
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:6:
In file included from include/linux/cpufreq.h:17:
In file included from include/linux/of_device.h:6:
include/linux/platform_device.h:353:71: warning: unused parameter 'pdev' [-Wunused-parameter]
static inline int is_sh_early_platform_device(struct platform_device *pdev)
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:6:
include/linux/cpufreq.h:190:71: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline struct cpufreq_policy *cpufreq_cpu_get_raw(unsigned int cpu)
^
include/linux/cpufreq.h:194:67: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline struct cpufreq_policy *cpufreq_cpu_get(unsigned int cpu)
^
include/linux/cpufreq.h:198:59: warning: unused parameter 'policy' [-Wunused-parameter]
static inline void cpufreq_cpu_put(struct cpufreq_policy *policy) { }
^
include/linux/cpufreq.h:232:53: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline unsigned int cpufreq_get(unsigned int cpu)
^
include/linux/cpufreq.h:236:59: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline unsigned int cpufreq_quick_get(unsigned int cpu)
^
include/linux/cpufreq.h:240:63: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline unsigned int cpufreq_quick_get_max(unsigned int cpu)
^
include/linux/cpufreq.h:244:65: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline unsigned int cpufreq_get_hw_max_freq(unsigned int cpu)
^
include/linux/cpufreq.h:261:70: warning: unused parameter 'policy' [-Wunused-parameter]
static inline void cpufreq_stats_create_table(struct cpufreq_policy *policy) { }
^
include/linux/cpufreq.h:262:68: warning: unused parameter 'policy' [-Wunused-parameter]
static inline void cpufreq_stats_free_table(struct cpufreq_policy *policy) { }
^
include/linux/cpufreq.h:263:75: warning: unused parameter 'policy' [-Wunused-parameter]
static inline void cpufreq_stats_record_transition(struct cpufreq_policy *policy,
^
include/linux/cpufreq.h:264:23: warning: unused parameter 'new_freq' [-Wunused-parameter]
unsigned int new_freq) { }
^
include/linux/cpufreq.h:500:68: warning: unused parameter 'nb' [-Wunused-parameter]
static inline int cpufreq_register_notifier(struct notifier_block *nb,
^
include/linux/cpufreq.h:501:20: warning: unused parameter 'list' [-Wunused-parameter]
unsigned int list)
^
include/linux/cpufreq.h:505:70: warning: unused parameter 'nb' [-Wunused-parameter]
static inline int cpufreq_unregister_notifier(struct notifier_block *nb,
^
include/linux/cpufreq.h:506:20: warning: unused parameter 'list' [-Wunused-parameter]
unsigned int list)
^
include/linux/cpufreq.h:676:64: warning: unused parameter 'dev' [-Wunused-parameter]
static inline int dev_pm_opp_init_cpufreq_table(struct device *dev,
^
include/linux/cpufreq.h:678:9: warning: unused parameter 'table' [-Wunused-parameter]
**table)
^
include/linux/cpufreq.h:683:65: warning: unused parameter 'dev' [-Wunused-parameter]
static inline void dev_pm_opp_free_cpufreq_table(struct device *dev,
^
include/linux/cpufreq.h:685:10: warning: unused parameter 'table' [-Wunused-parameter]
**table)
^
include/linux/cpufreq.h:1038:51: warning: unused parameter 'state' [-Wunused-parameter]
static inline int cpufreq_boost_trigger_state(int state)
^
include/linux/cpufreq.h:1052:65: warning: unused parameter 'policy' [-Wunused-parameter]
static inline bool policy_has_boost_freq(struct cpufreq_policy *policy)
^
>> include/linux/cpufreq.h:1057:58: warning: unused parameter 'pcpu' [-Wunused-parameter]
static inline int of_perf_domain_get_sharing_cpumask(int pcpu, const char *list_name,
^
>> include/linux/cpufreq.h:1057:76: warning: unused parameter 'list_name' [-Wunused-parameter]
static inline int of_perf_domain_get_sharing_cpumask(int pcpu, const char *list_name,
^
>> include/linux/cpufreq.h:1058:24: warning: unused parameter 'cell_name' [-Wunused-parameter]
const char *cell_name, struct cpumask *cpumask)
^
>> include/linux/cpufreq.h:1058:51: warning: unused parameter 'cpumask' [-Wunused-parameter]
const char *cell_name, struct cpumask *cpumask)
^
include/linux/cpufreq.h:1068:73: warning: unused parameter 'policy' [-Wunused-parameter]
static inline void sched_cpufreq_governor_change(struct cpufreq_policy *policy,
^
include/linux/cpufreq.h:1069:29: warning: unused parameter 'old_gov' [-Wunused-parameter]
struct cpufreq_governor *old_gov) { }
^
include/linux/cpufreq.h:1077:48: warning: unused parameter 'cpus' [-Wunused-parameter]
void arch_set_freq_scale(const struct cpumask *cpus,
^
include/linux/cpufreq.h:1078:19: warning: unused parameter 'cur_freq' [-Wunused-parameter]
unsigned long cur_freq,
^
include/linux/cpufreq.h:1079:19: warning: unused parameter 'max_freq' [-Wunused-parameter]
unsigned long max_freq)
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:8:
In file included from drivers/gpu/drm/i915/i915_drv.h:36:
In file included from arch/x86/include/asm/hypervisor.h:37:
In file included from arch/x86/include/asm/kvm_para.h:7:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:5:
include/linux/context_tracking_state.h:51:53: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline bool context_tracking_enabled_cpu(int cpu) { return false; }
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:8:
In file included from drivers/gpu/drm/i915/i915_drv.h:36:
In file included from arch/x86/include/asm/hypervisor.h:37:
In file included from arch/x86/include/asm/kvm_para.h:7:
In file included from include/linux/interrupt.h:11:
In file included from include/linux/hardirq.h:10:
include/linux/vtime.h:28:57: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_user_enter(struct task_struct *tsk) { }
^
include/linux/vtime.h:29:56: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_user_exit(struct task_struct *tsk) { }
^
include/linux/vtime.h:30:58: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_guest_enter(struct task_struct *tsk) { }
^
include/linux/vtime.h:31:57: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_guest_exit(struct task_struct *tsk) { }
^
include/linux/vtime.h:32:56: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_init_idle(struct task_struct *tsk, int cpu) { }
^
include/linux/vtime.h:32:65: warning: unused parameter 'cpu' [-Wunused-parameter]
static inline void vtime_init_idle(struct task_struct *tsk, int cpu) { }
^
include/linux/vtime.h:41:58: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_account_irq(struct task_struct *tsk, unsigned int offset) { }
^
include/linux/vtime.h:41:76: warning: unused parameter 'offset' [-Wunused-parameter]
static inline void vtime_account_irq(struct task_struct *tsk, unsigned int offset) { }
^
include/linux/vtime.h:42:62: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_account_softirq(struct task_struct *tsk) { }
^
include/linux/vtime.h:43:62: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_account_hardirq(struct task_struct *tsk) { }
^
include/linux/vtime.h:44:52: warning: unused parameter 'tsk' [-Wunused-parameter]
static inline void vtime_flush(struct task_struct *tsk) { }
^
include/linux/vtime.h:117:58: warning: unused parameter 'prev' [-Wunused-parameter]
static inline void vtime_task_switch(struct task_struct *prev) { }
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:8:
In file included from drivers/gpu/drm/i915/i915_drv.h:36:
In file included from arch/x86/include/asm/hypervisor.h:37:
In file included from arch/x86/include/asm/kvm_para.h:7:
In file included from include/linux/interrupt.h:20:
In file included from arch/x86/include/asm/sections.h:7:
include/asm-generic/sections.h:70:53: warning: unused parameter 'addr' [-Wunused-parameter]
static inline int arch_is_kernel_text(unsigned long addr)
^
include/asm-generic/sections.h:77:53: warning: unused parameter 'addr' [-Wunused-parameter]
static inline int arch_is_kernel_data(unsigned long addr)
^
In file included from drivers/gpu/drm/i915/gt/intel_llc.c:8:
In file included from drivers/gpu/drm/i915/i915_drv.h:36:
In file included from arch/x86/include/asm/hypervisor.h:37:
In file included from arch/x86/include/asm/kvm_para.h:7:
include/linux/interrupt.h:346:49: warning: unused parameter 'irq' [-Wunused-parameter]
static inline int irq_set_affinity(unsigned int irq, const struct cpumask *m)
^
include/linux/interrupt.h:346:76: warning: unused parameter 'm' [-Wunused-parameter]
static inline int irq_set_affinity(unsigned int irq, const struct cpumask *m)
^
include/linux/interrupt.h:351:51: warning: unused parameter 'irq' [-Wunused-parameter]
static inline int irq_force_affinity(unsigned int irq, const struct cpumask *cpumask)
^
include/linux/interrupt.h:351:78: warning: unused parameter 'cpumask' [-Wunused-parameter]
static inline int irq_force_affinity(unsigned int irq, const struct cpumask *cpumask)
^
include/linux/interrupt.h:356:53: warning: unused parameter 'irq' [-Wunused-parameter]
static inline int irq_can_set_affinity(unsigned int irq)
^
include/linux/interrupt.h:361:52: warning: unused parameter 'irq' [-Wunused-parameter]

vim +/pcpu +1057 include/linux/cpufreq.h

1000
1001 static inline int of_perf_domain_get_sharing_cpumask(int pcpu, const char *list_name,
1002 const char *cell_name, struct cpumask *cpumask)
1003 {
1004 struct device_node *cpu_np;
1005 struct of_phandle_args args;
1006 int target_idx;
1007 int cpu, ret;
1008
1009 cpu_np = of_cpu_device_node_get(pcpu);
1010 of_parse_phandle_with_args(cpu_np, list_name,
1011 cell_name, 0, &args);
1012 of_node_put(cpu_np);
1013 target_idx = args.args[0];
1014
1015 for_each_possible_cpu(cpu) {
1016 if (cpu == pcpu)
1017 continue;
1018
1019 cpu_np = of_cpu_device_node_get(cpu);
1020 if (!cpu_np)
1021 continue;
1022
1023 ret = of_parse_phandle_with_args(cpu_np, list_name,
1024 cell_name, 0,
1025 &args);
1026
1027 of_node_put(cpu_np);
1028 if (ret < 0)
1029 continue;
1030
1031 if (target_idx == args.args[0])
1032 cpumask_set_cpu(cpu, cpumask);
1033 }
1034
1035 return target_idx;
1036 }
1037 #else
> 1038 static inline int cpufreq_boost_trigger_state(int state)
1039 {
1040 return 0;
1041 }
1042 static inline int cpufreq_boost_enabled(void)
1043 {
1044 return 0;
1045 }
1046
1047 static inline int cpufreq_enable_boost_support(void)
1048 {
1049 return -EINVAL;
1050 }
1051
> 1052 static inline bool policy_has_boost_freq(struct cpufreq_policy *policy)
1053 {
1054 return false;
1055 }
1056
> 1057 static inline int of_perf_domain_get_sharing_cpumask(int pcpu, const char *list_name,
> 1058 const char *cell_name, struct cpumask *cpumask)
1059 {
1060 return -EOPNOTSUPP;
1061 }
1062 #endif
1063

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

Next message: Josh Poimboeuf: "Re: [PATCH v2 bpf-next 1/3] perf: enable branch record for software events"
Previous message: Fabio M. De Francesco: "Re: [PATCH v2 0/3] staging: r8188eu: Shorten and simplify calls chain"
In reply to: Hector Yuan: "[PATCH v15 2/3] cpufreq: Add of_perf_domain_get_sharing_cpumask"
Next in thread: Viresh Kumar: "Re: [PATCH v15 2/3] cpufreq: Add of_perf_domain_get_sharing_cpumask"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]