From: kernel test robot <[email protected]>
To: Vasanthakumar Thiagarajan <[email protected]>
Cc: [email protected],
GNU/Weeb Mailing List <[email protected]>,
[email protected], Kalle Valo <[email protected]>,
Bhagavathi Perumal S <[email protected]>,
Karthikeyan Periyasamy <[email protected]>,
Pradeep Kumar Chitrapu <[email protected]>,
P Praneesh <[email protected]>,
Ramya Gnanasekar <[email protected]>,
Sriram R <[email protected]>
Subject: [ammarfaizi2-block:kvalo/ath/ath12k-bringup 29/38] drivers/net/wireless/ath/ath12k/hal_rx.c:404:25: warning: cast to pointer from integer of different size
Date: Sat, 30 Apr 2022 02:20:37 +0800 [thread overview]
Message-ID: <[email protected]> (raw)
Hi Vasanthakumar,
FYI, the error/warning still remains.
tree: https://github.com/ammarfaizi2/linux-block kvalo/ath/ath12k-bringup
head: 910edb5ff5fbe5ad5460c750c7a62e92c4eb0aa5
commit: f40abb4788a2a3868606a29d99583421e0874350 [29/38] ath12k: New driver for Qualcomm 11be hw family
config: arc-allyesconfig (https://download.01.org/0day-ci/archive/20220430/[email protected]/config)
compiler: arceb-elf-gcc (GCC) 11.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/ammarfaizi2/linux-block/commit/f40abb4788a2a3868606a29d99583421e0874350
git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
git fetch --no-tags ammarfaizi2-block kvalo/ath/ath12k-bringup
git checkout f40abb4788a2a3868606a29d99583421e0874350
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=arc SHELL=/bin/bash drivers/net/wireless/ath/ath12k/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>
All warnings (new ones prefixed by >>):
drivers/net/wireless/ath/ath12k/hal_rx.c: In function 'ath12k_hal_wbm_desc_parse_err':
>> drivers/net/wireless/ath/ath12k/hal_rx.c:404:25: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
404 | (struct ath12k_rx_desc_info *)((u64)wbm_cc_desc->buf_va_hi << 32 |
| ^
--
drivers/net/wireless/ath/ath12k/dp_tx.c: In function 'ath12k_dp_tx_completion_handler':
>> drivers/net/wireless/ath/ath12k/dp_tx.c:605:35: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
605 | tx_desc = (struct ath12k_tx_desc_info *)
| ^
In file included from include/linux/bitops.h:6,
from include/linux/kernel.h:13,
from include/linux/interrupt.h:6,
from drivers/net/wireless/ath/ath12k/core.h:11,
from drivers/net/wireless/ath/ath12k/dp_tx.c:7:
drivers/net/wireless/ath/ath12k/dp_tx.c: In function 'ath12k_dp_tx_htt_h2t_vdev_stats_ol_req':
include/linux/bits.h:35:29: warning: left shift count >= width of type [-Wshift-count-overflow]
35 | (((~UL(0)) - (UL(1) << (l)) + 1) & \
| ^~
include/linux/bits.h:38:38: note: in expansion of macro '__GENMASK'
38 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^~~~~~~~~
drivers/net/wireless/ath/ath12k/dp.h:1921:49: note: in expansion of macro 'GENMASK'
1921 | #define HTT_H2T_VDEV_TXRX_HI_BITMASK GENMASK(63, 32)
| ^~~~~~~
drivers/net/wireless/ath/ath12k/dp_tx.c:1074:37: note: in expansion of macro 'HTT_H2T_VDEV_TXRX_HI_BITMASK'
1074 | HTT_H2T_VDEV_TXRX_HI_BITMASK) >> 32);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/bits.h:36:18: warning: right shift count is negative [-Wshift-count-negative]
36 | (~UL(0) >> (BITS_PER_LONG - 1 - (h))))
| ^~
include/linux/bits.h:38:38: note: in expansion of macro '__GENMASK'
38 | (GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
| ^~~~~~~~~
drivers/net/wireless/ath/ath12k/dp.h:1921:49: note: in expansion of macro 'GENMASK'
1921 | #define HTT_H2T_VDEV_TXRX_HI_BITMASK GENMASK(63, 32)
| ^~~~~~~
drivers/net/wireless/ath/ath12k/dp_tx.c:1074:37: note: in expansion of macro 'HTT_H2T_VDEV_TXRX_HI_BITMASK'
1074 | HTT_H2T_VDEV_TXRX_HI_BITMASK) >> 32);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
drivers/net/wireless/ath/ath12k/dp_rx.c: In function 'ath12k_dp_process_rx':
>> drivers/net/wireless/ath/ath12k/dp_rx.c:2739:29: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
2739 | desc_info = (struct ath12k_rx_desc_info *)((u64)desc.buf_va_hi << 32 |
| ^
drivers/net/wireless/ath/ath12k/dp_rx.c: In function 'ath12k_dp_process_rx_err_buf':
drivers/net/wireless/ath/ath12k/dp_rx.c:3413:21: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
3413 | desc_info = (struct ath12k_rx_desc_info *)((u64)desc.buf_va_hi << 32 |
| ^
vim +404 drivers/net/wireless/ath/ath12k/hal_rx.c
352
353 int ath12k_hal_wbm_desc_parse_err(struct ath12k_base *ab, void *desc,
354 struct hal_rx_wbm_rel_info *rel_info)
355 {
356 struct hal_wbm_release_ring *wbm_desc = desc;
357 struct hal_wbm_release_ring_cc_rx *wbm_cc_desc = desc;
358 enum hal_wbm_rel_desc_type type;
359 enum hal_wbm_rel_src_module rel_src;
360 bool hw_cc_done;
361
362 type = FIELD_GET(HAL_WBM_RELEASE_INFO0_DESC_TYPE,
363 wbm_desc->info0);
364 /* We expect only WBM_REL buffer type */
365 if (type != HAL_WBM_REL_DESC_TYPE_REL_MSDU) {
366 WARN_ON(1);
367 return -EINVAL;
368 }
369
370 rel_src = FIELD_GET(HAL_WBM_RELEASE_INFO0_REL_SRC_MODULE,
371 wbm_desc->info0);
372 if (rel_src != HAL_WBM_REL_SRC_MODULE_RXDMA &&
373 rel_src != HAL_WBM_REL_SRC_MODULE_REO)
374 return -EINVAL;
375
376 /* The format of wbm rel ring desc changes based on the
377 * hw cookie conversion status
378 */
379 hw_cc_done = FIELD_GET(HAL_WBM_RELEASE_RX_INFO0_CC_STATUS,
380 wbm_desc->info0);
381
382 if (!hw_cc_done) {
383 if (FIELD_GET(BUFFER_ADDR_INFO1_RET_BUF_MGR,
384 wbm_desc->buf_addr_info.info1) != HAL_RX_BUF_RBM_SW3_BM) {
385 ab->soc_stats.invalid_rbm++;
386 return -EINVAL;
387 }
388
389 rel_info->cookie = FIELD_GET(BUFFER_ADDR_INFO1_SW_COOKIE,
390 wbm_desc->buf_addr_info.info1);
391
392 rel_info->rx_desc = NULL;
393 } else {
394 if (FIELD_GET(HAL_WBM_RELEASE_RX_CC_INFO0_RBM,
395 wbm_cc_desc->info0) != HAL_RX_BUF_RBM_SW3_BM) {
396 ab->soc_stats.invalid_rbm++;
397 return -EINVAL;
398 }
399
400 rel_info->cookie = FIELD_GET(HAL_WBM_RELEASE_RX_CC_INFO1_COOKIE,
401 wbm_cc_desc->info1);
402
403 rel_info->rx_desc =
> 404 (struct ath12k_rx_desc_info *)((u64)wbm_cc_desc->buf_va_hi << 32 |
405 wbm_cc_desc->buf_va_lo);
406 }
407
408 rel_info->err_rel_src = rel_src;
409 rel_info->hw_cc_done = hw_cc_done;
410
411 if (rel_info->err_rel_src == HAL_WBM_REL_SRC_MODULE_REO) {
412 rel_info->push_reason =
413 FIELD_GET(HAL_WBM_RELEASE_INFO0_REO_PUSH_REASON,
414 wbm_desc->info0);
415 rel_info->err_code =
416 FIELD_GET(HAL_WBM_RELEASE_INFO0_REO_ERROR_CODE,
417 wbm_desc->info0);
418 } else {
419 rel_info->push_reason =
420 FIELD_GET(HAL_WBM_RELEASE_INFO0_RXDMA_PUSH_REASON,
421 wbm_desc->info0);
422 rel_info->err_code =
423 FIELD_GET(HAL_WBM_RELEASE_INFO0_RXDMA_ERROR_CODE,
424 wbm_desc->info0);
425 }
426
427 return 0;
428 }
429
--
0-DAY CI Kernel Test Service
https://01.org/lkp
reply other threads:[~2022-04-29 18:21 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
[email protected] \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox