public inbox for [email protected]
 help / color / mirror / Atom feed
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