Re: [PATCH 4/5] clk: qcom: Add SDM660 GPU Clock Controller (GPUCC) driver
by Philip Li
On Wed, Sep 30, 2020 at 11:58:50PM +0200, AngeloGioacchino Del Regno wrote:
> The kernel test robot is wrong: this commit builds fine by
Thanks for the info and sorry for the wrong report. Currently it is
hard to detect the base in this case automatically, if possible, we
recommend to use --base option when git format patch, this provides
extra hint to us when looking for the base.
> merging in the required patch series (464958) named
> "Qualcomm clock fixes and preparation for SDM660"
> https://lore.kernel.org/patchwork/project/lkml/list/?series=464958
>
> In that series there is a commit that generalizes the MSM8996-only
> gfx3d clock ops and introduces the definition of clk_rcg2_gfx3d.
>
> Il giorno mer 30 set 2020 alle ore 22:46 kernel test robot
> <lkp(a)intel.com> ha scritto:
> >
> > Hi,
> >
> > Thank you for the patch! Yet something to improve:
> >
> > [auto build test ERROR on clk/clk-next]
> > [also build test ERROR on robh/for-next linux/master linus/master v5.9-rc7 next-20200930]
> > [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/kholk11-gmail-com/SDM630-660-Mul...
> > base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next
> > config: m68k-allmodconfig (attached as .config)
> > compiler: m68k-linux-gcc (GCC) 9.3.0
> > 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/0551514b1ab723837163a8c4cd84d2283...
> > git remote add linux-review https://github.com/0day-ci/linux
> > git fetch --no-tags linux-review kholk11-gmail-com/SDM630-660-Multimedia-and-GPU-clock-controllers/20200926-210437
> > git checkout 0551514b1ab723837163a8c4cd84d22831e38019
> > # save the attached .config to linux build tree
> > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=m68k
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp(a)intel.com>
> >
> > All errors (new ones prefixed by >>):
> >
> > >> drivers/clk/qcom/gpucc-sdm660.c:106:15: error: variable 'gfx3d_clk_src' has initializer but incomplete type
> > 106 | static struct clk_rcg2_gfx3d gfx3d_clk_src = {
> > | ^~~~~~~~~~~~~~
> > >> drivers/clk/qcom/gpucc-sdm660.c:107:3: error: 'struct clk_rcg2_gfx3d' has no member named 'div'
> > 107 | .div = 2,
> > | ^~~
> > drivers/clk/qcom/gpucc-sdm660.c:107:9: warning: excess elements in struct initializer
> > 107 | .div = 2,
> > | ^
> > drivers/clk/qcom/gpucc-sdm660.c:107:9: note: (near initialization for 'gfx3d_clk_src')
> > >> drivers/clk/qcom/gpucc-sdm660.c:108:3: error: 'struct clk_rcg2_gfx3d' has no member named 'rcg'
> > 108 | .rcg = {
> > | ^~~
> > >> drivers/clk/qcom/gpucc-sdm660.c:108:9: error: extra brace group at end of initializer
> > 108 | .rcg = {
> > | ^
> > drivers/clk/qcom/gpucc-sdm660.c:108:9: note: (near initialization for 'gfx3d_clk_src')
> > drivers/clk/qcom/gpucc-sdm660.c:108:9: warning: excess elements in struct initializer
> > drivers/clk/qcom/gpucc-sdm660.c:108:9: note: (near initialization for 'gfx3d_clk_src')
> > >> drivers/clk/qcom/gpucc-sdm660.c:121:3: error: 'struct clk_rcg2_gfx3d' has no member named 'hws'
> > 121 | .hws = (struct clk_hw*[]){
> > | ^~~
> > drivers/clk/qcom/gpucc-sdm660.c:121:9: warning: excess elements in struct initializer
> > 121 | .hws = (struct clk_hw*[]){
> > | ^
> > drivers/clk/qcom/gpucc-sdm660.c:121:9: note: (near initialization for 'gfx3d_clk_src')
> > >> drivers/clk/qcom/gpucc-sdm660.c:139:25: error: invalid use of undefined type 'struct clk_rcg2_gfx3d'
> > 139 | .hw = &gfx3d_clk_src.rcg.clkr.hw,
> > | ^
> > drivers/clk/qcom/gpucc-sdm660.c:257:45: error: 'NO_RET_PERIPH' undeclared here (not in a function)
> > 257 | .flags = CLAMP_IO | SW_RESET | AON_RESET | NO_RET_PERIPH,
> > | ^~~~~~~~~~~~~
> > drivers/clk/qcom/gpucc-sdm660.c:277:34: error: invalid use of undefined type 'struct clk_rcg2_gfx3d'
> > 277 | [GFX3D_CLK_SRC] = &gfx3d_clk_src.rcg.clkr,
> > | ^
> > >> drivers/clk/qcom/gpucc-sdm660.c:106:30: error: storage size of 'gfx3d_clk_src' isn't known
> > 106 | static struct clk_rcg2_gfx3d gfx3d_clk_src = {
> > | ^~~~~~~~~~~~~
> >
> > vim +/gfx3d_clk_src +106 drivers/clk/qcom/gpucc-sdm660.c
> >
> > 105
> > > 106 static struct clk_rcg2_gfx3d gfx3d_clk_src = {
> > > 107 .div = 2,
> > > 108 .rcg = {
> > 109 .cmd_rcgr = 0x1070,
> > 110 .mnd_width = 0,
> > 111 .hid_width = 5,
> > 112 .parent_map = gpucc_parent_map_1,
> > 113 .clkr.hw.init = &(struct clk_init_data){
> > 114 .name = "gfx3d_clk_src",
> > 115 .parent_data = gpucc_parent_data_1,
> > 116 .num_parents = 4,
> > 117 .ops = &clk_gfx3d_ops,
> > 118 .flags = CLK_SET_RATE_PARENT | CLK_OPS_PARENT_ENABLE,
> > 119 },
> > 120 },
> > > 121 .hws = (struct clk_hw*[]){
> > 122 &gpucc_cxo_clk.clkr.hw,
> > 123 &gpu_pll0_pll_out_main.clkr.hw,
> > 124 &gpu_pll1_pll_out_main.clkr.hw,
> > 125 }
> > 126 };
> > 127
> > 128 static struct clk_branch gpucc_gfx3d_clk = {
> > 129 .halt_reg = 0x1098,
> > 130 .halt_check = BRANCH_HALT,
> > 131 .hwcg_reg = 0x1098,
> > 132 .hwcg_bit = 1,
> > 133 .clkr = {
> > 134 .enable_reg = 0x1098,
> > 135 .enable_mask = BIT(0),
> > 136 .hw.init = &(struct clk_init_data){
> > 137 .name = "gpucc_gfx3d_clk",
> > 138 .parent_data = &(const struct clk_parent_data){
> > > 139 .hw = &gfx3d_clk_src.rcg.clkr.hw,
> > 140 },
> > 141 .num_parents = 1,
> > 142 .ops = &clk_branch2_ops,
> > 143 .flags = CLK_SET_RATE_PARENT,
> > 144 },
> > 145 },
> > 146 };
> > 147
> >
> > ---
> > 0-DAY CI Kernel Test Service, Intel Corporation
> > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
>
1 year, 10 months
Re: [linux-next:master 12268/12330] ld.lld: warning: drivers/built-in.a(hwtracing/coresight/coresight-core.o):(.ARM.extab) is being placed in '.ARM.extab'
by Philip Li
On Thu, Oct 01, 2020 at 10:12:59AM -0700, Nathan Chancellor wrote:
> On Thu, Oct 01, 2020 at 04:00:25PM +0800, kernel test robot wrote:
> > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
> > head: de69ee6df1cfbf3c67787d8504fd21b59da39572
> > commit: ad32dae3950bd32340f4b7c9ee05f0633d9e8e32 [12268/12330] Merge remote-tracking branch 'char-misc/char-misc-next' into master
> > config: arm-randconfig-r011-20200930 (attached as .config)
> > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
> > 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 arm cross compiling tool for clang build
> > # apt-get install binutils-arm-linux-gnueabi
> > # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commi...
> > git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> > git fetch --no-tags linux-next master
> > git checkout ad32dae3950bd32340f4b7c9ee05f0633d9e8e32
> > # save the attached .config to linux build tree
> > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
> >
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kernel test robot <lkp(a)intel.com>
> >
> > All warnings (new ones prefixed by >>):
> >
> > ld.lld: warning: drivers/built-in.a(crypto/ccree/cc_hash.o):(.ARM.extab) is being placed in '.ARM.extab'
> > ld.lld: warning: drivers/built-in.a(crypto/ccree/cc_aead.o):(.ARM.extab) is being placed in '.ARM.extab'
> > ld.lld: warning: drivers/built-in.a(crypto/ccree/cc_sram_mgr.o):(.ARM.extab) is being placed in '.ARM.extab'
>
> Should be fixed with
> https://lore.kernel.org/r/20200928224854.3224862-1-natechancellor@gmail.com/,
> obviously not caused by anything in that merge. Still waiting for review
thanks, we will look into this to avoid false positive.
> and acceptance.
let me disable this warning for a while to reduce duplicated (and wrong bad commit)
reports.
>
> Cheers,
> Nathan
>
1 year, 10 months
Re: [PATCH v5 03/15] misc: bcm-vk: add autoload support
by kernel test robot
Hi Scott,
I love your patch! Yet something to improve:
[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on soc/for-next kees/for-next/pstore linus/master v5.9-rc7 next-20200930]
[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/Scott-Branden/Add-Broadcom-VK-dr...
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git c471bf4b11c7df0f0f9f42b5aeec424dc62d0c7e
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
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/f04a1673933a1af94e44bc650533bbb27...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Scott-Branden/Add-Broadcom-VK-driver/20201001-093119
git checkout f04a1673933a1af94e44bc650533bbb271e3dcc1
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/misc/bcm-vk/bcm_vk_dev.c: In function 'bcm_vk_load_image_by_type':
>> drivers/misc/bcm-vk/bcm_vk_dev.c:284:8: error: implicit declaration of function 'request_partial_firmware_into_buf'; did you mean 'request_firmware_into_buf'? [-Werror=implicit-function-declaration]
284 | ret = request_partial_firmware_into_buf(&fw, filename, dev,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| request_firmware_into_buf
cc1: some warnings being treated as errors
vim +284 drivers/misc/bcm-vk/bcm_vk_dev.c
206
207 static int bcm_vk_load_image_by_type(struct bcm_vk *vk, u32 load_type,
208 const char *filename)
209 {
210 struct device *dev = &vk->pdev->dev;
211 const struct firmware *fw = NULL;
212 void *bufp = NULL;
213 size_t max_buf, offset;
214 int ret;
215 u64 offset_codepush;
216 u32 codepush;
217 u32 value;
218 dma_addr_t boot_dma_addr;
219 bool is_stdalone;
220
221 if (load_type == VK_IMAGE_TYPE_BOOT1) {
222 /*
223 * After POR, enable VK soft BOOTSRC so bootrom do not clear
224 * the pushed image (the TCM memories).
225 */
226 value = vkread32(vk, BAR_0, BAR_BOOTSRC_SELECT);
227 value |= BOOTSRC_SOFT_ENABLE;
228 vkwrite32(vk, value, BAR_0, BAR_BOOTSRC_SELECT);
229
230 codepush = CODEPUSH_BOOTSTART + CODEPUSH_BOOT1_ENTRY;
231 offset_codepush = BAR_CODEPUSH_SBL;
232
233 /* Write a 1 to request SRAM open bit */
234 vkwrite32(vk, CODEPUSH_BOOTSTART, BAR_0, offset_codepush);
235
236 /* Wait for VK to respond */
237 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS, SRAM_OPEN,
238 SRAM_OPEN, LOAD_IMAGE_TIMEOUT_MS);
239 if (ret < 0) {
240 dev_err(dev, "boot1 wait SRAM err - ret(%d)\n", ret);
241 goto err_buf_out;
242 }
243
244 max_buf = SZ_256K;
245 bufp = dma_alloc_coherent(dev,
246 max_buf,
247 &boot_dma_addr, GFP_KERNEL);
248 if (!bufp) {
249 dev_err(dev, "Error allocating 0x%zx\n", max_buf);
250 ret = -ENOMEM;
251 goto err_buf_out;
252 }
253 } else if (load_type == VK_IMAGE_TYPE_BOOT2) {
254 codepush = CODEPUSH_BOOT2_ENTRY;
255 offset_codepush = BAR_CODEPUSH_SBI;
256
257 /* Wait for VK to respond */
258 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS, DDR_OPEN,
259 DDR_OPEN, LOAD_IMAGE_TIMEOUT_MS);
260 if (ret < 0) {
261 dev_err(dev, "boot2 wait DDR open error - ret(%d)\n",
262 ret);
263 goto err_buf_out;
264 }
265
266 max_buf = SZ_4M;
267 bufp = dma_alloc_coherent(dev,
268 max_buf,
269 &boot_dma_addr, GFP_KERNEL);
270 if (!bufp) {
271 dev_err(dev, "Error allocating 0x%zx\n", max_buf);
272 ret = -ENOMEM;
273 goto err_buf_out;
274 }
275
276 bcm_vk_buf_notify(vk, bufp, boot_dma_addr, max_buf);
277 } else {
278 dev_err(dev, "Error invalid image type 0x%x\n", load_type);
279 ret = -EINVAL;
280 goto err_buf_out;
281 }
282
283 offset = 0;
> 284 ret = request_partial_firmware_into_buf(&fw, filename, dev,
285 bufp, max_buf, offset);
286 if (ret) {
287 dev_err(dev, "Error %d requesting firmware file: %s\n",
288 ret, filename);
289 goto err_firmware_out;
290 }
291 dev_dbg(dev, "size=0x%zx\n", fw->size);
292 if (load_type == VK_IMAGE_TYPE_BOOT1)
293 memcpy_toio(vk->bar[BAR_1] + BAR1_CODEPUSH_BASE_BOOT1,
294 bufp,
295 fw->size);
296
297 dev_dbg(dev, "Signaling 0x%x to 0x%llx\n", codepush, offset_codepush);
298 vkwrite32(vk, codepush, BAR_0, offset_codepush);
299
300 if (load_type == VK_IMAGE_TYPE_BOOT1) {
301 u32 boot_status;
302
303 /* wait until done */
304 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS,
305 BOOT1_RUNNING,
306 BOOT1_RUNNING,
307 BOOT1_STARTUP_TIMEOUT_MS);
308
309 boot_status = vkread32(vk, BAR_0, BAR_BOOT_STATUS);
310 is_stdalone = !BCM_VK_INTF_IS_DOWN(boot_status) &&
311 (boot_status & BOOT_STDALONE_RUNNING);
312 if (ret && !is_stdalone) {
313 dev_err(dev,
314 "Timeout %ld ms waiting for boot1 to come up - ret(%d)\n",
315 BOOT1_STARTUP_TIMEOUT_MS, ret);
316 goto err_firmware_out;
317 } else if (is_stdalone) {
318 u32 reg;
319
320 reg = vkread32(vk, BAR_0, BAR_BOOT1_STDALONE_PROGRESS);
321 if ((reg & BOOT1_STDALONE_PROGRESS_MASK) ==
322 BOOT1_STDALONE_SUCCESS) {
323 dev_info(dev, "Boot1 standalone success\n");
324 ret = 0;
325 } else {
326 dev_err(dev, "Timeout %ld ms - Boot1 standalone failure\n",
327 BOOT1_STARTUP_TIMEOUT_MS);
328 ret = -EINVAL;
329 goto err_firmware_out;
330 }
331 }
332 } else if (load_type == VK_IMAGE_TYPE_BOOT2) {
333 unsigned long timeout;
334
335 timeout = jiffies + msecs_to_jiffies(LOAD_IMAGE_TIMEOUT_MS);
336
337 /* To send more data to VK than max_buf allowed at a time */
338 do {
339 /*
340 * Check for ack from card. when Ack is received,
341 * it means all the data is received by card.
342 * Exit the loop after ack is received.
343 */
344 ret = bcm_vk_wait(vk, BAR_0, BAR_BOOT_STATUS,
345 FW_LOADER_ACK_RCVD_ALL_DATA,
346 FW_LOADER_ACK_RCVD_ALL_DATA,
347 TXFR_COMPLETE_TIMEOUT_MS);
348 if (ret == 0) {
349 dev_dbg(dev, "Exit boot2 download\n");
350 break;
351 } else if (ret == -EFAULT) {
352 dev_err(dev, "Error detected during ACK waiting");
353 goto err_firmware_out;
354 }
355
356 /* exit the loop, if there is no response from card */
357 if (time_after(jiffies, timeout)) {
358 dev_err(dev, "Error. No reply from card\n");
359 ret = -ETIMEDOUT;
360 goto err_firmware_out;
361 }
362
363 /* Wait for VK to open BAR space to copy new data */
364 ret = bcm_vk_wait(vk, BAR_0, offset_codepush,
365 codepush, 0,
366 TXFR_COMPLETE_TIMEOUT_MS);
367 if (ret == 0) {
368 offset += max_buf;
369 ret = request_partial_firmware_into_buf
370 (&fw,
371 filename,
372 dev, bufp,
373 max_buf,
374 offset);
375 if (ret) {
376 dev_err(dev,
377 "Error %d requesting firmware file: %s offset: 0x%zx\n",
378 ret, filename, offset);
379 goto err_firmware_out;
380 }
381 dev_dbg(dev, "size=0x%zx\n", fw->size);
382 dev_dbg(dev, "Signaling 0x%x to 0x%llx\n",
383 codepush, offset_codepush);
384 vkwrite32(vk, codepush, BAR_0, offset_codepush);
385 /* reload timeout after every codepush */
386 timeout = jiffies +
387 msecs_to_jiffies(LOAD_IMAGE_TIMEOUT_MS);
388 } else if (ret == -EFAULT) {
389 dev_err(dev, "Error detected waiting for transfer\n");
390 goto err_firmware_out;
391 }
392 } while (1);
393
394 /* wait for fw status bits to indicate app ready */
395 ret = bcm_vk_wait(vk, BAR_0, VK_BAR_FWSTS,
396 VK_FWSTS_READY,
397 VK_FWSTS_READY,
398 BOOT2_STARTUP_TIMEOUT_MS);
399 if (ret < 0) {
400 dev_err(dev, "Boot2 not ready - ret(%d)\n", ret);
401 goto err_firmware_out;
402 }
403
404 is_stdalone = vkread32(vk, BAR_0, BAR_BOOT_STATUS) &
405 BOOT_STDALONE_RUNNING;
406 if (!is_stdalone) {
407 ret = bcm_vk_intf_ver_chk(vk);
408 if (ret) {
409 dev_err(dev, "failure in intf version check\n");
410 goto err_firmware_out;
411 }
412
413 /* sync & channel other info */
414 ret = bcm_vk_sync_card_info(vk);
415 if (ret) {
416 dev_err(dev, "Syncing Card Info failure\n");
417 goto err_firmware_out;
418 }
419 }
420 }
421
422 err_firmware_out:
423 release_firmware(fw);
424
425 err_buf_out:
426 if (bufp)
427 dma_free_coherent(dev, max_buf, bufp, boot_dma_addr);
428
429 return ret;
430 }
431
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
drivers/gpu/drm/i915/gt/intel_workarounds.c:1082:20: error: function 'is_nonpriv_flags_valid' is not needed and will not be emitted
by kernel test robot
Hi Nick,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 60e720931556fc1034d0981460164dcf02697679
commit: 9f4069b055d1508c833115df7493b6e0001e5c9b drm/i915: re-disable -Wframe-address
date: 5 months ago
config: x86_64-randconfig-a005-20201001 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
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 x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 9f4069b055d1508c833115df7493b6e0001e5c9b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
>> drivers/gpu/drm/i915/gt/intel_workarounds.c:1082:20: error: function 'is_nonpriv_flags_valid' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration]
static inline bool is_nonpriv_flags_valid(u32 flags)
^
1 error generated.
--
>> drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c:259:19: error: unused function 'rq_prio' [-Werror,-Wunused-function]
static inline int rq_prio(const struct i915_request *rq)
^
1 error generated.
--
>> drivers/gpu/drm/i915/gvt/gtt.c:267:19: error: unused function 'get_pt_type' [-Werror,-Wunused-function]
static inline int get_pt_type(int type)
^
>> drivers/gpu/drm/i915/gvt/gtt.c:590:20: error: unused function 'ppgtt_set_guest_root_entry' [-Werror,-Wunused-function]
static inline void ppgtt_set_guest_root_entry(struct intel_vgpu_mm *mm,
^
2 errors generated.
vim +/is_nonpriv_flags_valid +1082 drivers/gpu/drm/i915/gt/intel_workarounds.c
094304beb4e1f4 drivers/gpu/drm/i915/intel_workarounds.c Tvrtko Ursulin 2018-12-03 1081
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 @1082 static inline bool is_nonpriv_flags_valid(u32 flags)
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1083 {
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1084 /* Check only valid flag bits are set */
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1085 if (flags & ~RING_FORCE_TO_NONPRIV_MASK_VALID)
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1086 return false;
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1087
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1088 /* NB: Only 3 out of 4 enum values are valid for access field */
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1089 if ((flags & RING_FORCE_TO_NONPRIV_ACCESS_MASK) ==
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1090 RING_FORCE_TO_NONPRIV_ACCESS_INVALID)
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1091 return false;
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1092
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1093 return true;
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1094 }
1e2b7f497c28a4 drivers/gpu/drm/i915/gt/intel_workarounds.c John Harrison 2019-07-12 1095
:::::: The code at line 1082 was first introduced by commit
:::::: 1e2b7f497c28a47793a95d8f0cc8e135899827f1 drm/i915: Add test for invalid flag bits in whitelist entries
:::::: TO: John Harrison <John.C.Harrison(a)Intel.com>
:::::: CC: Tvrtko Ursulin <tvrtko.ursulin(a)intel.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
Re: [PATCH v3 08/13] ASoC: audio-graph: Expose helpers from audio graph
by kernel test robot
Hi Sameer,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on tegra/for-next]
[also build test WARNING on v5.9-rc7 next-20201001]
[cannot apply to asoc/for-next robh/for-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
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Sameer-Pujar/Audio-graph-card-up...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git for-next
config: x86_64-randconfig-r034-20200930 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
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 x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/b7b97805bc967aae0ce3009c1bbf17b0f...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Sameer-Pujar/Audio-graph-card-updates-and-usage-with-Tegra210-audio/20201002-013648
git checkout b7b97805bc967aae0ce3009c1bbf17b0f232e18b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
sound/soc/generic/audio-graph-card.c:116:28: error: implicit declaration of function 'snd_soc_find_dai_with_mutex' [-Werror,-Wimplicit-function-declaration]
struct snd_soc_dai *dai = snd_soc_find_dai_with_mutex(dlc);
^
sound/soc/generic/audio-graph-card.c:116:22: warning: incompatible integer to pointer conversion initializing 'struct snd_soc_dai *' with an expression of type 'int' [-Wint-conversion]
struct snd_soc_dai *dai = snd_soc_find_dai_with_mutex(dlc);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> sound/soc/generic/audio-graph-card.c:532:5: warning: no previous prototype for function 'graph_parse_of' [-Wmissing-prototypes]
int graph_parse_of(struct asoc_simple_priv *priv)
^
sound/soc/generic/audio-graph-card.c:532:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int graph_parse_of(struct asoc_simple_priv *priv)
^
static
>> sound/soc/generic/audio-graph-card.c:608:6: warning: no previous prototype for function 'graph_get_dais_count' [-Wmissing-prototypes]
void graph_get_dais_count(struct asoc_simple_priv *priv,
^
sound/soc/generic/audio-graph-card.c:608:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void graph_get_dais_count(struct asoc_simple_priv *priv,
^
static
>> sound/soc/generic/audio-graph-card.c:667:5: warning: no previous prototype for function 'graph_card_probe' [-Wmissing-prototypes]
int graph_card_probe(struct snd_soc_card *card)
^
sound/soc/generic/audio-graph-card.c:667:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int graph_card_probe(struct snd_soc_card *card)
^
static
4 warnings and 1 error generated.
vim +/graph_parse_of +532 sound/soc/generic/audio-graph-card.c
531
> 532 int graph_parse_of(struct asoc_simple_priv *priv)
533 {
534 struct snd_soc_card *card = simple_priv_to_card(priv);
535 struct link_info li;
536 int ret;
537
538 ret = asoc_simple_parse_widgets(card, NULL);
539 if (ret < 0)
540 return ret;
541
542 ret = asoc_simple_parse_routing(card, NULL);
543 if (ret < 0)
544 return ret;
545
546 memset(&li, 0, sizeof(li));
547 for (li.cpu = 1; li.cpu >= 0; li.cpu--) {
548 /*
549 * Detect all CPU first, and Detect all Codec 2nd.
550 *
551 * In Normal sound case, all DAIs are detected
552 * as "CPU-Codec".
553 *
554 * In DPCM sound case,
555 * all CPUs are detected as "CPU-dummy", and
556 * all Codecs are detected as "dummy-Codec".
557 * To avoid random sub-device numbering,
558 * detect "dummy-Codec" in last;
559 */
560 ret = graph_for_each_link(priv, &li,
561 graph_dai_link_of,
562 graph_dai_link_of_dpcm);
563 if (ret < 0)
564 return ret;
565 }
566
567 return asoc_simple_parse_card_name(card, NULL);
568 }
569 EXPORT_SYMBOL_GPL(graph_parse_of);
570
571 static int graph_count_noml(struct asoc_simple_priv *priv,
572 struct device_node *cpu_ep,
573 struct device_node *codec_ep,
574 struct link_info *li)
575 {
576 struct device *dev = simple_priv_to_dev(priv);
577
578 li->link += 1; /* 1xCPU-Codec */
579 li->dais += 2; /* 1xCPU + 1xCodec */
580
581 dev_dbg(dev, "Count As Normal\n");
582
583 return 0;
584 }
585
586 static int graph_count_dpcm(struct asoc_simple_priv *priv,
587 struct device_node *cpu_ep,
588 struct device_node *codec_ep,
589 struct link_info *li,
590 int dup_codec)
591 {
592 struct device *dev = simple_priv_to_dev(priv);
593
594 li->link++; /* 1xCPU-dummy */
595 li->dais++; /* 1xCPU */
596
597 if (!dup_codec && codec_ep) {
598 li->link++; /* 1xdummy-Codec */
599 li->conf++; /* 1xdummy-Codec */
600 li->dais++; /* 1xCodec */
601 }
602
603 dev_dbg(dev, "Count As DPCM\n");
604
605 return 0;
606 }
607
> 608 void graph_get_dais_count(struct asoc_simple_priv *priv,
609 struct link_info *li)
610 {
611 struct device *dev = simple_priv_to_dev(priv);
612
613 /*
614 * link_num : number of links.
615 * CPU-Codec / CPU-dummy / dummy-Codec
616 * dais_num : number of DAIs
617 * ccnf_num : number of codec_conf
618 * same number for "dummy-Codec"
619 *
620 * ex1)
621 * CPU0 --- Codec0 link : 5
622 * CPU1 --- Codec1 dais : 7
623 * CPU2 -/ ccnf : 1
624 * CPU3 --- Codec2
625 *
626 * => 5 links = 2xCPU-Codec + 2xCPU-dummy + 1xdummy-Codec
627 * => 7 DAIs = 4xCPU + 3xCodec
628 * => 1 ccnf = 1xdummy-Codec
629 *
630 * ex2)
631 * CPU0 --- Codec0 link : 5
632 * CPU1 --- Codec1 dais : 6
633 * CPU2 -/ ccnf : 1
634 * CPU3 -/
635 *
636 * => 5 links = 1xCPU-Codec + 3xCPU-dummy + 1xdummy-Codec
637 * => 6 DAIs = 4xCPU + 2xCodec
638 * => 1 ccnf = 1xdummy-Codec
639 *
640 * ex3)
641 * CPU0 --- Codec0 link : 6
642 * CPU1 -/ dais : 6
643 * CPU2 --- Codec1 ccnf : 2
644 * CPU3 -/
645 *
646 * => 6 links = 0xCPU-Codec + 4xCPU-dummy + 2xdummy-Codec
647 * => 6 DAIs = 4xCPU + 2xCodec
648 * => 2 ccnf = 2xdummy-Codec
649 *
650 * ex4)
651 * CPU0 --- Codec0 (convert-rate) link : 3
652 * CPU1 --- Codec1 dais : 4
653 * ccnf : 1
654 *
655 * => 3 links = 1xCPU-Codec + 1xCPU-dummy + 1xdummy-Codec
656 * => 4 DAIs = 2xCPU + 2xCodec
657 * => 1 ccnf = 1xdummy-Codec
658 */
659 graph_for_each_link(priv, li,
660 graph_count_noml,
661 graph_count_dpcm);
662 dev_dbg(dev, "link %d, dais %d, ccnf %d\n",
663 li->link, li->dais, li->conf);
664 }
665 EXPORT_SYMBOL_GPL(graph_get_dais_count);
666
> 667 int graph_card_probe(struct snd_soc_card *card)
668 {
669 struct asoc_simple_priv *priv = snd_soc_card_get_drvdata(card);
670 int ret;
671
672 ret = asoc_simple_init_hp(card, &priv->hp_jack, NULL);
673 if (ret < 0)
674 return ret;
675
676 ret = asoc_simple_init_mic(card, &priv->mic_jack, NULL);
677 if (ret < 0)
678 return ret;
679
680 return 0;
681 }
682 EXPORT_SYMBOL_GPL(graph_card_probe);
683
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
Re: [PATCH v3 08/13] ASoC: audio-graph: Expose helpers from audio graph
by kernel test robot
Hi Sameer,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on tegra/for-next]
[also build test WARNING on v5.9-rc7 next-20201001]
[cannot apply to asoc/for-next robh/for-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
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Sameer-Pujar/Audio-graph-card-up...
base: https://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux.git for-next
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 9.3.0
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/b7b97805bc967aae0ce3009c1bbf17b0f...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Sameer-Pujar/Audio-graph-card-updates-and-usage-with-Tegra210-audio/20201002-013648
git checkout b7b97805bc967aae0ce3009c1bbf17b0f232e18b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
sound/soc/generic/audio-graph-card.c: In function 'soc_component_is_pcm':
sound/soc/generic/audio-graph-card.c:116:28: error: implicit declaration of function 'snd_soc_find_dai_with_mutex' [-Werror=implicit-function-declaration]
116 | struct snd_soc_dai *dai = snd_soc_find_dai_with_mutex(dlc);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
sound/soc/generic/audio-graph-card.c:116:28: warning: initialization of 'struct snd_soc_dai *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
sound/soc/generic/audio-graph-card.c: At top level:
>> sound/soc/generic/audio-graph-card.c:532:5: warning: no previous prototype for 'graph_parse_of' [-Wmissing-prototypes]
532 | int graph_parse_of(struct asoc_simple_priv *priv)
| ^~~~~~~~~~~~~~
>> sound/soc/generic/audio-graph-card.c:608:6: warning: no previous prototype for 'graph_get_dais_count' [-Wmissing-prototypes]
608 | void graph_get_dais_count(struct asoc_simple_priv *priv,
| ^~~~~~~~~~~~~~~~~~~~
>> sound/soc/generic/audio-graph-card.c:667:5: warning: no previous prototype for 'graph_card_probe' [-Wmissing-prototypes]
667 | int graph_card_probe(struct snd_soc_card *card)
| ^~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/graph_parse_of +532 sound/soc/generic/audio-graph-card.c
531
> 532 int graph_parse_of(struct asoc_simple_priv *priv)
533 {
534 struct snd_soc_card *card = simple_priv_to_card(priv);
535 struct link_info li;
536 int ret;
537
538 ret = asoc_simple_parse_widgets(card, NULL);
539 if (ret < 0)
540 return ret;
541
542 ret = asoc_simple_parse_routing(card, NULL);
543 if (ret < 0)
544 return ret;
545
546 memset(&li, 0, sizeof(li));
547 for (li.cpu = 1; li.cpu >= 0; li.cpu--) {
548 /*
549 * Detect all CPU first, and Detect all Codec 2nd.
550 *
551 * In Normal sound case, all DAIs are detected
552 * as "CPU-Codec".
553 *
554 * In DPCM sound case,
555 * all CPUs are detected as "CPU-dummy", and
556 * all Codecs are detected as "dummy-Codec".
557 * To avoid random sub-device numbering,
558 * detect "dummy-Codec" in last;
559 */
560 ret = graph_for_each_link(priv, &li,
561 graph_dai_link_of,
562 graph_dai_link_of_dpcm);
563 if (ret < 0)
564 return ret;
565 }
566
567 return asoc_simple_parse_card_name(card, NULL);
568 }
569 EXPORT_SYMBOL_GPL(graph_parse_of);
570
571 static int graph_count_noml(struct asoc_simple_priv *priv,
572 struct device_node *cpu_ep,
573 struct device_node *codec_ep,
574 struct link_info *li)
575 {
576 struct device *dev = simple_priv_to_dev(priv);
577
578 li->link += 1; /* 1xCPU-Codec */
579 li->dais += 2; /* 1xCPU + 1xCodec */
580
581 dev_dbg(dev, "Count As Normal\n");
582
583 return 0;
584 }
585
586 static int graph_count_dpcm(struct asoc_simple_priv *priv,
587 struct device_node *cpu_ep,
588 struct device_node *codec_ep,
589 struct link_info *li,
590 int dup_codec)
591 {
592 struct device *dev = simple_priv_to_dev(priv);
593
594 li->link++; /* 1xCPU-dummy */
595 li->dais++; /* 1xCPU */
596
597 if (!dup_codec && codec_ep) {
598 li->link++; /* 1xdummy-Codec */
599 li->conf++; /* 1xdummy-Codec */
600 li->dais++; /* 1xCodec */
601 }
602
603 dev_dbg(dev, "Count As DPCM\n");
604
605 return 0;
606 }
607
> 608 void graph_get_dais_count(struct asoc_simple_priv *priv,
609 struct link_info *li)
610 {
611 struct device *dev = simple_priv_to_dev(priv);
612
613 /*
614 * link_num : number of links.
615 * CPU-Codec / CPU-dummy / dummy-Codec
616 * dais_num : number of DAIs
617 * ccnf_num : number of codec_conf
618 * same number for "dummy-Codec"
619 *
620 * ex1)
621 * CPU0 --- Codec0 link : 5
622 * CPU1 --- Codec1 dais : 7
623 * CPU2 -/ ccnf : 1
624 * CPU3 --- Codec2
625 *
626 * => 5 links = 2xCPU-Codec + 2xCPU-dummy + 1xdummy-Codec
627 * => 7 DAIs = 4xCPU + 3xCodec
628 * => 1 ccnf = 1xdummy-Codec
629 *
630 * ex2)
631 * CPU0 --- Codec0 link : 5
632 * CPU1 --- Codec1 dais : 6
633 * CPU2 -/ ccnf : 1
634 * CPU3 -/
635 *
636 * => 5 links = 1xCPU-Codec + 3xCPU-dummy + 1xdummy-Codec
637 * => 6 DAIs = 4xCPU + 2xCodec
638 * => 1 ccnf = 1xdummy-Codec
639 *
640 * ex3)
641 * CPU0 --- Codec0 link : 6
642 * CPU1 -/ dais : 6
643 * CPU2 --- Codec1 ccnf : 2
644 * CPU3 -/
645 *
646 * => 6 links = 0xCPU-Codec + 4xCPU-dummy + 2xdummy-Codec
647 * => 6 DAIs = 4xCPU + 2xCodec
648 * => 2 ccnf = 2xdummy-Codec
649 *
650 * ex4)
651 * CPU0 --- Codec0 (convert-rate) link : 3
652 * CPU1 --- Codec1 dais : 4
653 * ccnf : 1
654 *
655 * => 3 links = 1xCPU-Codec + 1xCPU-dummy + 1xdummy-Codec
656 * => 4 DAIs = 2xCPU + 2xCodec
657 * => 1 ccnf = 1xdummy-Codec
658 */
659 graph_for_each_link(priv, li,
660 graph_count_noml,
661 graph_count_dpcm);
662 dev_dbg(dev, "link %d, dais %d, ccnf %d\n",
663 li->link, li->dais, li->conf);
664 }
665 EXPORT_SYMBOL_GPL(graph_get_dais_count);
666
> 667 int graph_card_probe(struct snd_soc_card *card)
668 {
669 struct asoc_simple_priv *priv = snd_soc_card_get_drvdata(card);
670 int ret;
671
672 ret = asoc_simple_init_hp(card, &priv->hp_jack, NULL);
673 if (ret < 0)
674 return ret;
675
676 ret = asoc_simple_init_mic(card, &priv->mic_jack, NULL);
677 if (ret < 0)
678 return ret;
679
680 return 0;
681 }
682 EXPORT_SYMBOL_GPL(graph_card_probe);
683
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c:426:32: warning: unused variable 'aq_pm_ops'
by kernel test robot
Hi zhengbin,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 60e720931556fc1034d0981460164dcf02697679
commit: 0ee0bbb018938addf87b54d447cc5633d2e53490 net: atlantic: make symbol 'aq_pm_ops' static
date: 11 months ago
config: x86_64-randconfig-a003-20201001 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
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 x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 0ee0bbb018938addf87b54d447cc5633d2e53490
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c:426:32: warning: unused variable 'aq_pm_ops' [-Wunused-const-variable]
static const struct dev_pm_ops aq_pm_ops = {
^
1 warning generated.
vim +/aq_pm_ops +426 drivers/net/ethernet/aquantia/atlantic/aq_pci_func.c
425
> 426 static const struct dev_pm_ops aq_pm_ops = {
427 .suspend = aq_pm_suspend_poweroff,
428 .poweroff = aq_pm_suspend_poweroff,
429 .freeze = aq_pm_freeze,
430 .resume = aq_pm_resume_restore,
431 .restore = aq_pm_resume_restore,
432 .thaw = aq_pm_thaw,
433 };
434
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
[arm-integrator:kasan 1/6] arch/arm/mm/mmu.c:1302:13: warning: no previous prototype for function 'arm_mm_clear_atag_pages'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git kasan
head: fb5e749f76335be35afcb3ea74ad3fa0926900db
commit: 0ffa763acb72e5e1a0b75b27faa91767d971ba08 [1/6] ARM: Handle a device tree in lowmem
config: arm-randconfig-r012-20200930 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
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 arm cross compiling tool for clang build
# apt-get install binutils-arm-linux-gnueabi
# https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.g...
git remote add arm-integrator https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
git fetch --no-tags arm-integrator kasan
git checkout 0ffa763acb72e5e1a0b75b27faa91767d971ba08
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
arch/arm/mm/mmu.c:119:13: warning: no previous prototype for function 'init_default_cache_policy' [-Wmissing-prototypes]
void __init init_default_cache_policy(unsigned long pmd)
^
arch/arm/mm/mmu.c:119:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init init_default_cache_policy(unsigned long pmd)
^
static
arch/arm/mm/mmu.c:1157:13: warning: no previous prototype for function 'adjust_lowmem_bounds' [-Wmissing-prototypes]
void __init adjust_lowmem_bounds(void)
^
arch/arm/mm/mmu.c:1157:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init adjust_lowmem_bounds(void)
^
static
>> arch/arm/mm/mmu.c:1302:13: warning: no previous prototype for function 'arm_mm_clear_atag_pages' [-Wmissing-prototypes]
void __init arm_mm_clear_atag_pages(void)
^
arch/arm/mm/mmu.c:1302:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init arm_mm_clear_atag_pages(void)
^
static
arch/arm/mm/mmu.c:1667:13: warning: no previous prototype for function 'paging_init' [-Wmissing-prototypes]
void __init paging_init(const struct machine_desc *mdesc)
^
arch/arm/mm/mmu.c:1667:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init paging_init(const struct machine_desc *mdesc)
^
static
arch/arm/mm/mmu.c:1691:13: warning: no previous prototype for function 'early_mm_init' [-Wmissing-prototypes]
void __init early_mm_init(const struct machine_desc *mdesc)
^
arch/arm/mm/mmu.c:1691:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void __init early_mm_init(const struct machine_desc *mdesc)
^
static
5 warnings generated.
vim +/arm_mm_clear_atag_pages +1302 arch/arm/mm/mmu.c
1301
> 1302 void __init arm_mm_clear_atag_pages(void)
1303 {
1304 unsigned long addr;
1305 unsigned long atags;
1306 phys_addr_t end;
1307
1308 /*
1309 * Clear out the kernel space mappings used by the attached DTB if
1310 * it happens to end up in lowmem.
1311 */
1312 end = memblock.memory.regions[0].base + memblock.memory.regions[0].size;
1313 if (end >= arm_lowmem_limit)
1314 end = arm_lowmem_limit;
1315 atags = __phys_to_virt(__atags_pointer);
1316 atags &= PMD_MASK;
1317 for (addr = __phys_to_virt(end);
1318 addr < VMALLOC_START; addr += PMD_SIZE) {
1319 if (addr >= atags && addr < (atags + 2 * PMD_SIZE)) {
1320 pr_info("Clear ATAGs/DTB PMD @0x%08lx\n", addr);
1321 pmd_clear(pmd_off_k(addr));
1322 }
1323 }
1324 }
1325
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
[chrome-os:chromeos-4.4 7/9] /tmp/hostap_info-311473.s:1912: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
by kernel test robot
tree: https://chromium.googlesource.com/chromiumos/third_party/kernel chromeos-4.4
head: 2b50676552be09f1dbb6db4d40f20eeeca61f51c
commit: 765b1cd9b693a948690a676b811a9c28bf33d5c6 [7/9] CHROMIUM: media/rockchip-vpu: remove support for V4L2_CID_MPEG_VIDEO_H264_SPS_PPS_BEFORE_IDR
config: x86_64-randconfig-a003-20200930 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project bcd05599d0e53977a963799d6ee4f6e0bc21331b)
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 x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
git remote add chrome-os https://chromium.googlesource.com/chromiumos/third_party/kernel
git fetch --no-tags chrome-os chromeos-4.4
git checkout 765b1cd9b693a948690a676b811a9c28bf33d5c6
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
arch/x86/include/asm/percpu.h:478:35: note: expanded from macro 'this_cpu_add_8'
#define this_cpu_add_8(pcp, val) percpu_add_op((pcp), val)
^~~~~~~~~~~~~~~~~~~~~~~~~
arch/x86/include/asm/percpu.h:130:31: note: expanded from macro 'percpu_add_op'
((val) == 1 || (val) == -1)) ? \
~~~ ^ ~~
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:3:
In file included from include/linux/if_arp.h:26:
In file included from include/linux/skbuff.h:31:
In file included from include/net/checksum.h:25:
arch/x86/include/asm/uaccess.h:754:26: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (likely(sz < 0 || sz >= n)) {
~~ ^ ~
include/linux/compiler.h:181:40: note: expanded from macro 'likely'
# define likely(x) __builtin_expect(!!(x), 1)
^
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:3:
In file included from include/linux/if_arp.h:26:
In file included from include/linux/skbuff.h:31:
In file included from include/net/checksum.h:25:
arch/x86/include/asm/uaccess.h:773:26: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (likely(sz < 0 || sz >= n)) {
~~ ^ ~
include/linux/compiler.h:181:40: note: expanded from macro 'likely'
# define likely(x) __builtin_expect(!!(x), 1)
^
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:3:
In file included from include/linux/if_arp.h:26:
include/linux/skbuff.h:2675:23: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
&csum, from) == copy) {
~~~~~~~~~~~~ ^ ~~~~
include/linux/skbuff.h:2679:60: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
} else if (copy_from_iter(skb_put(skb, copy), copy, from) == copy)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~
include/linux/skbuff.h:2693:14: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
off == frag->page_offset + skb_frag_size(frag);
~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/skbuff.h:2921:51: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
return copy_from_iter(data, len, &msg->msg_iter) == len ? 0 : -EFAULT;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~
include/linux/skbuff.h:2926:49: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
return copy_to_iter(data, len, &msg->msg_iter) == len ? 0 : -EFAULT;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:3:
In file included from include/linux/if_arp.h:27:
In file included from include/uapi/linux/if_arp.h:26:
In file included from include/linux/netdevice.h:43:
In file included from include/net/net_namespace.h:17:
In file included from include/net/netns/ipv4.h:9:
In file included from include/net/inet_frag.h:4:
include/linux/rhashtable.h:264:34: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
return atomic_read(&ht->nelems) > (tbl->size / 4 * 3) &&
~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~
include/linux/rhashtable.h:277:34: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
return atomic_read(&ht->nelems) < (tbl->size * 3 / 10) &&
~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~
include/linux/rhashtable.h:289:34: warning: comparison of integers of different signs: 'int' and 'const unsigned int' [-Wsign-compare]
return atomic_read(&ht->nelems) > tbl->size &&
~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~
include/linux/rhashtable.h:302:34: warning: comparison of integers of different signs: 'int' and 'const unsigned int' [-Wsign-compare]
atomic_read(&ht->nelems) >= ht->p.insecure_max_entries;
~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:3:
In file included from include/linux/if_arp.h:27:
In file included from include/uapi/linux/if_arp.h:26:
include/linux/netdevice.h:2413:34: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
skb_checksum_start_offset(skb) <
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
include/linux/netdevice.h:3051:15: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
BUG_ON(index >= dev->num_rx_queues);
~~~~~ ^ ~~~~~~~~~~~~~~~~~~
include/asm-generic/bug.h:58:45: note: expanded from macro 'BUG_ON'
#define BUG_ON(condition) do { if (unlikely(condition)) BUG(); } while (0)
^~~~~~~~~
include/linux/compiler.h:182:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:3:
In file included from include/linux/if_arp.h:27:
In file included from include/uapi/linux/if_arp.h:26:
include/linux/netdevice.h:3325:37: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (debug_value < 0 || debug_value >= (sizeof(u32) * 8))
~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~
In file included from drivers/net/wireless/intersil/hostap/hostap_info.c:8:
In file included from drivers/net/wireless/intersil/hostap/hostap_wlan.h:11:
In file included from include/net/lib80211.h:31:
include/linux/ieee80211.h:2510:15: warning: comparison of integers of different signs: 'unsigned int' and 'int' [-Wsign-compare]
skb->len > (skb_network_offset(skb) + 2)) {
~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/intersil/hostap/hostap_info.c:18:11: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (left < sizeof(struct hfa384x_comm_tallies)) {
~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/intersil/hostap/hostap_info.c:58:11: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (left < sizeof(struct hfa384x_comm_tallies32)) {
~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/net/wireless/intersil/hostap/hostap_info.c:327:15: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (copy_len > sizeof(struct hfa384x_hostscan_result))
~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
22 warnings generated.
/tmp/hostap_info-311473.s: Assembler messages:
>> /tmp/hostap_info-311473.s:1912: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/hostap_info-311473.s:1933: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/hostap_info-311473.s:2173: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
--
include/linux/netdevice.h:2413:34: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
skb_checksum_start_offset(skb) <
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
include/linux/netdevice.h:3051:15: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
BUG_ON(index >= dev->num_rx_queues);
~~~~~ ^ ~~~~~~~~~~~~~~~~~~
include/asm-generic/bug.h:58:45: note: expanded from macro 'BUG_ON'
#define BUG_ON(condition) do { if (unlikely(condition)) BUG(); } while (0)
^~~~~~~~~
include/linux/compiler.h:182:42: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^
In file included from net//bluetooth/l2cap_core.c:36:
In file included from include/net/bluetooth/bluetooth.h:29:
In file included from include/net/sock.h:51:
include/linux/netdevice.h:3325:37: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (debug_value < 0 || debug_value >= (sizeof(u32) * 8))
~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~
In file included from net//bluetooth/l2cap_core.c:36:
In file included from include/net/bluetooth/bluetooth.h:29:
In file included from include/net/sock.h:63:
In file included from include/linux/filter.h:16:
In file included from include/net/sch_generic.h:12:
In file included from include/net/rtnetlink.h:5:
include/net/netlink.h:345:18: warning: comparison of integers of different signs: 'const __u32' (aka 'const unsigned int') and 'int' [-Wsign-compare]
nlh->nlmsg_len <= remaining);
~~~~~~~~~~~~~~ ^ ~~~~~~~~~
include/net/netlink.h:380:21: warning: comparison of integers of different signs: 'const __u32' (aka 'const unsigned int') and 'int' [-Wsign-compare]
if (nlh->nlmsg_len < nlmsg_msg_size(hdrlen))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~
include/net/netlink.h:413:21: warning: comparison of integers of different signs: 'const __u32' (aka 'const unsigned int') and 'int' [-Wsign-compare]
if (nlh->nlmsg_len < nlmsg_msg_size(hdrlen))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~
In file included from net//bluetooth/l2cap_core.c:36:
In file included from include/net/bluetooth/bluetooth.h:29:
In file included from include/net/sock.h:63:
In file included from include/linux/filter.h:16:
In file included from include/net/sch_generic.h:12:
include/net/rtnetlink.h:20:21: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (nlmsg_len(nlh) >= sizeof(struct rtgenmsg))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~
In file included from net//bluetooth/l2cap_core.c:36:
In file included from include/net/bluetooth/bluetooth.h:29:
In file included from include/net/sock.h:63:
In file included from include/linux/filter.h:16:
include/net/sch_generic.h:267:33: warning: comparison of integers of different signs: 'unsigned long' and 'int' [-Wsign-compare]
BUILD_BUG_ON(sizeof(qcb->data) < sz);
~~~~~~~~~~~~~~~~~ ^ ~~
include/linux/build_bug.h:71:19: note: expanded from macro 'BUILD_BUG_ON'
BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
^~~~~~~~~
include/linux/build_bug.h:47:58: note: expanded from macro 'BUILD_BUG_ON_MSG'
#define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
^~~~
include/linux/compiler.h:509:22: note: expanded from macro 'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
^~~~~~~~~
include/linux/compiler.h:497:23: note: expanded from macro '_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^~~~~~~~~
include/linux/compiler.h:489:19: note: expanded from macro '__compiletime_assert'
bool __cond = !(condition); \
^~~~~~~~~
In file included from net//bluetooth/l2cap_core.c:36:
In file included from include/net/bluetooth/bluetooth.h:29:
In file included from include/net/sock.h:63:
include/linux/filter.h:517:16: warning: comparison of integers of different signs: 'const __u32' (aka 'const unsigned int') and 'int' [-Wsign-compare]
if (first->k == SKF_AD_OFF + SKF_AD_ALU_XOR_X)
~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from net//bluetooth/l2cap_core.c:36:
In file included from include/net/bluetooth/bluetooth.h:29:
include/net/sock.h:1882:54: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
if (csum_and_copy_from_iter(to, copy, &csum, from) != copy)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~
include/net/sock.h:1886:46: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
if (copy_from_iter_nocache(to, copy, from) != copy)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~
include/net/sock.h:1888:44: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
} else if (copy_from_iter(to, copy, from) != copy)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~
net//bluetooth/l2cap_core.c:2155:65: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
if (copy_from_iter(skb_put(skb, count), count, &msg->msg_iter) != count)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~
net//bluetooth/l2cap_core.c:2176:24: warning: comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
&msg->msg_iter) != count)
~~~~~~~~~~~~~~~ ^ ~~~~~
net//bluetooth/l2cap_core.c:4118:27: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (chan->conf_len + len > sizeof(chan->conf_req)) {
~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~
net//bluetooth/l2cap_core.c:4255:12: warning: comparison of integers of different signs: 'int' and 'unsigned long' [-Wsign-compare]
if (len > sizeof(req) - sizeof(struct l2cap_conf_req)) {
~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
net//bluetooth/l2cap_core.c:7581:11: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
if (len == skb->len) {
~~~ ^ ~~~~~~~~
net//bluetooth/l2cap_core.c:7589:16: warning: comparison of integers of different signs: 'unsigned int' and 'int' [-Wsign-compare]
if (skb->len > len) {
~~~~~~~~ ^ ~~~
33 warnings generated.
/tmp/l2cap_core-815894.s: Assembler messages:
>> /tmp/l2cap_core-815894.s:12174: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
/tmp/l2cap_core-815894.s:19266: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
/tmp/l2cap_core-815894.s:26495: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
/tmp/l2cap_core-815894.s:39340: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
>> /tmp/l2cap_core-815894.s:47927: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:49064: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:52337: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
/tmp/l2cap_core-815894.s:53273: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
/tmp/l2cap_core-815894.s:60984: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:61027: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:62251: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:63002: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:63057: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:63108: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:66612: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:67065: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:67530: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:68270: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
/tmp/l2cap_core-815894.s:72296: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months
[android-common:android-mainline 1/6] drivers/tty/serial/msm_geni_serial_earlycon.c:307:6: warning: variable 'rx_stale' set but not used
by kernel test robot
tree: https://android.googlesource.com/kernel/common android-mainline
head: a44b31f34c58e6e9dca649a262e0c9986e757ed8
commit: e1be87019fc97f736f2729a2a4198d250d28633b [1/6] ANDROID: serial: msm_geni_serial_earlycon: Add Earlycon support
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
git remote add android-common https://android.googlesource.com/kernel/common
git fetch --no-tags android-common android-mainline
git checkout e1be87019fc97f736f2729a2a4198d250d28633b
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/tty/serial/msm_geni_serial_earlycon.c: In function 'msm_geni_serial_earlycon_setup':
>> drivers/tty/serial/msm_geni_serial_earlycon.c:307:6: warning: variable 'rx_stale' set but not used [-Wunused-but-set-variable]
307 | u32 rx_stale = 0;
| ^~~~~~~~
>> drivers/tty/serial/msm_geni_serial_earlycon.c:305:6: warning: variable 'rx_parity_cfg' set but not used [-Wunused-but-set-variable]
305 | u32 rx_parity_cfg = 0;
| ^~~~~~~~~~~~~
>> drivers/tty/serial/msm_geni_serial_earlycon.c:304:6: warning: variable 'rx_trans_cfg' set but not used [-Wunused-but-set-variable]
304 | u32 rx_trans_cfg = 0;
| ^~~~~~~~~~~~
vim +/rx_stale +307 drivers/tty/serial/msm_geni_serial_earlycon.c
296
297 static int __init
298 msm_geni_serial_earlycon_setup(struct earlycon_device *dev,
299 const char *opt)
300 {
301 int ret = 0;
302 u32 tx_trans_cfg = 0;
303 u32 tx_parity_cfg = 0;
> 304 u32 rx_trans_cfg = 0;
> 305 u32 rx_parity_cfg = 0;
306 u32 stop_bit = 0;
> 307 u32 rx_stale = 0;
308 u32 bits_per_char = 0;
309 unsigned long cfg0, cfg1;
310 struct uart_port *uport = &dev->port;
311
312 if (!uport->membase) {
313 ret = -ENOMEM;
314 goto exit;
315 }
316
317 if (get_se_proto_earlycon(uport->membase) != UART_PROTOCOL) {
318 ret = -ENXIO;
319 goto exit;
320 }
321
322 /*
323 * Ignore Flow control.
324 * Disable Tx Parity.
325 * Don't check Parity during Rx.
326 * Disable Rx Parity.
327 * n = 8.
328 * Stop bit = 0.
329 * Stale timeout in bit-time (3 chars worth).
330 */
331 tx_trans_cfg |= UART_CTS_MASK;
332 tx_parity_cfg = 0;
333 rx_trans_cfg = 0;
334 rx_parity_cfg = 0;
335 bits_per_char = 0x8;
336 stop_bit = 0;
337 rx_stale = 0x18;
338
339 msm_geni_serial_poll_abort_tx(uport);
340
341 se_get_packing_config_earlycon(8, 1, false, &cfg0, &cfg1);
342
343 se_io_init_earlycon(uport->membase);
344
345 geni_se_select_fifo_mode_earlycon(uport->membase);
346 writel_relaxed(cfg0, uport->membase + SE_GENI_TX_PACKING_CFG0);
347 writel_relaxed(cfg1, uport->membase + SE_GENI_TX_PACKING_CFG1);
348 writel_relaxed(tx_trans_cfg, uport->membase + SE_UART_TX_TRANS_CFG);
349 writel_relaxed(tx_parity_cfg, uport->membase + SE_UART_TX_PARITY_CFG);
350 writel_relaxed(bits_per_char, uport->membase + SE_UART_TX_WORD_LEN);
351 writel_relaxed(stop_bit, uport->membase + SE_UART_TX_STOP_BIT_LEN);
352
353 dev->con->write = msm_geni_serial_early_console_write;
354 dev->con->setup = NULL;
355 exit:
356 return ret;
357 }
358
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 year, 10 months