2021-06-09 12:32:45

by kernel test robot

[permalink] [raw]
Subject: Re: [PATCH 1/5] firmware: qcom_scm: Introduce SCM calls to access LMh

Hi Thara,

I love your patch! Perhaps something to improve:

[auto build test WARNING on robh/for-next]
[also build test WARNING on pm/linux-next linus/master v5.13-rc5 next-20210608]
[cannot apply to thermal/next]
[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

url: https://github.com/0day-ci/linux/commits/Thara-Gopinath/Introduce-LMh-driver-for-Qualcomm-SoCs/20210609-063135
base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: powerpc-randconfig-r024-20210608 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project d2012d965d60c3258b3a69d024491698f8aec386)
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
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
# https://github.com/0day-ci/linux/commit/bd39209abfb69da45acdc24c969d69620106f24e
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Thara-Gopinath/Introduce-LMh-driver-for-Qualcomm-SoCs/20210609-063135
git checkout bd39209abfb69da45acdc24c969d69620106f24e
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>

All warnings (new ones prefixed by >>):

In file included from drivers/mmc/host/sdhci-msm.c:8:
In file included from include/linux/module.h:12:
In file included from include/linux/list.h:9:
In file included from include/linux/kernel.h:12:
In file included from include/linux/bitops.h:32:
In file included from arch/powerpc/include/asm/bitops.h:62:
arch/powerpc/include/asm/barrier.h:49:9: warning: '__lwsync' macro redefined [-Wmacro-redefined]
#define __lwsync() __asm__ __volatile__ (stringify_in_c(LWSYNC) : : :"memory")
<built-in>:308:9: note: previous definition is here
#define __lwsync __builtin_ppc_lwsync
In file included from drivers/mmc/host/sdhci-msm.c:16:
>> include/linux/qcom_scm.h:180:5: warning: no previous prototype for function 'qcom_scm_lmh_dcvsh' [-Wmissing-prototypes]
int qcom_scm_lmh_dcvsh(u32 *payload, u32 payload_size, u64 limit_node,
include/linux/qcom_scm.h:180:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int qcom_scm_lmh_dcvsh(u32 *payload, u32 payload_size, u64 limit_node,
>> include/linux/qcom_scm.h:183:5: warning: no previous prototype for function 'qcom_scm_lmh_profile_change' [-Wmissing-prototypes]
int qcom_scm_lmh_profile_change(u32 profile_id) { return -ENODEV; }
include/linux/qcom_scm.h:183:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int qcom_scm_lmh_profile_change(u32 profile_id) { return -ENODEV; }
>> include/linux/qcom_scm.h:185:6: warning: no previous prototype for function 'qcom_scm_lmh_dcvsh_available' [-Wmissing-prototypes]
bool qcom_scm_lmh_dcvsh_available(void) { return -ENODEV; }
include/linux/qcom_scm.h:185:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
bool qcom_scm_lmh_dcvsh_available(void) { return -ENODEV; }
4 warnings generated.

Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for HOTPLUG_CPU
Selected by

vim +/qcom_scm_lmh_dcvsh +180 include/linux/qcom_scm.h

141 static inline int qcom_scm_io_readl(phys_addr_t addr, unsigned int *val)
142 { return -ENODEV; }
143 static inline int qcom_scm_io_writel(phys_addr_t addr, unsigned int val)
144 { return -ENODEV; }
146 static inline bool qcom_scm_restore_sec_cfg_available(void) { return false; }
147 static inline int qcom_scm_restore_sec_cfg(u32 device_id, u32 spare)
148 { return -ENODEV; }
149 static inline int qcom_scm_iommu_secure_ptbl_size(u32 spare, size_t *size)
150 { return -ENODEV; }
151 static inline int qcom_scm_iommu_secure_ptbl_init(u64 addr, u32 size, u32 spare)
152 { return -ENODEV; }
153 extern inline int qcom_scm_mem_protect_video_var(u32 cp_start, u32 cp_size,
154 u32 cp_nonpixel_start,
155 u32 cp_nonpixel_size)
156 { return -ENODEV; }
157 static inline int qcom_scm_assign_mem(phys_addr_t mem_addr, size_t mem_sz,
158 unsigned int *src, const struct qcom_scm_vmperm *newvm,
159 unsigned int dest_cnt) { return -ENODEV; }
161 static inline bool qcom_scm_ocmem_lock_available(void) { return false; }
162 static inline int qcom_scm_ocmem_lock(enum qcom_scm_ocmem_client id, u32 offset,
163 u32 size, u32 mode) { return -ENODEV; }
164 static inline int qcom_scm_ocmem_unlock(enum qcom_scm_ocmem_client id,
165 u32 offset, u32 size) { return -ENODEV; }
167 static inline bool qcom_scm_ice_available(void) { return false; }
168 static inline int qcom_scm_ice_invalidate_key(u32 index) { return -ENODEV; }
169 static inline int qcom_scm_ice_set_key(u32 index, const u8 *key, u32 key_size,
170 enum qcom_scm_ice_cipher cipher,
171 u32 data_unit_size) { return -ENODEV; }
173 static inline bool qcom_scm_hdcp_available(void) { return false; }
174 static inline int qcom_scm_hdcp_req(struct qcom_scm_hdcp_req *req, u32 req_cnt,
175 u32 *resp) { return -ENODEV; }
177 static inline int qcom_scm_qsmmu500_wait_safe_toggle(bool en)
178 { return -ENODEV; }
> 180 int qcom_scm_lmh_dcvsh(u32 *payload, u32 payload_size, u64 limit_node,
181 u32 node_id, u64 version)
182 { return -ENODEV; }
> 183 int qcom_scm_lmh_profile_change(u32 profile_id) { return -ENODEV; }
> 185 bool qcom_scm_lmh_dcvsh_available(void) { return -ENODEV; }

0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

(No filename) (6.49 kB)
.config.gz (35.23 kB)
Download all attachments