From: kernel test robot <lkp@intel.com>
To: David Mosberger-Tang <davidm@egauge.net>,
Ajay Singh <ajay.kathat@microchip.com>
Cc: kbuild-all@lists.01.org,
Claudiu Beznea <claudiu.beznea@microchip.com>,
Kalle Valo <kvalo@kernel.org>, Jakub Kicinski <kuba@kernel.org>,
linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org,
David Mosberger-Tang <davidm@egauge.net>
Subject: Re: [PATCH v2 32/50] wilc1000: introduce vmm_table_entry() helper function
Date: Tue, 28 Dec 2021 20:47:48 +0800 [thread overview]
Message-ID: <202112282030.iPTnhHRr-lkp@intel.com> (raw)
In-Reply-To: <20211223011358.4031459-33-davidm@egauge.net>
Hi David,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on kvalo-wireless-drivers-next/master]
[also build test WARNING on kvalo-wireless-drivers/master v5.16-rc7 next-20211224]
[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/David-Mosberger-Tang/wilc1000-rework-tx-path-to-use-sk_buffs-throughout/20211223-091915
base: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
config: sparc-randconfig-s031-20211228 (https://download.01.org/0day-ci/archive/20211228/202112282030.iPTnhHRr-lkp@intel.com/config)
compiler: sparc64-linux-gcc (GCC) 11.2.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# apt-get install sparse
# sparse version: v0.6.4-dirty
# https://github.com/0day-ci/linux/commit/9339519807fd005c22f3299f859edc615e540d3f
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review David-Mosberger-Tang/wilc1000-rework-tx-path-to-use-sk_buffs-throughout/20211223-091915
git checkout 9339519807fd005c22f3299f859edc615e540d3f
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=sparc SHELL=/bin/bash drivers/net/wireless/microchip/wilc1000/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
sparse warnings: (new ones prefixed by >>)
>> drivers/net/wireless/microchip/wilc1000/wlan.c:640:16: sparse: sparse: incorrect type in return expression (different base types) @@ expected unsigned int @@ got restricted __le32 [usertype] @@
drivers/net/wireless/microchip/wilc1000/wlan.c:640:16: sparse: expected unsigned int
drivers/net/wireless/microchip/wilc1000/wlan.c:640:16: sparse: got restricted __le32 [usertype]
vim +640 drivers/net/wireless/microchip/wilc1000/wlan.c
631
632 static u32 vmm_table_entry(struct sk_buff *tqe, u32 vmm_sz)
633 {
634 struct wilc_skb_tx_cb *tx_cb = WILC_SKB_TX_CB(tqe);
635 u32 entry;
636
637 entry = vmm_sz / 4;
638 if (tx_cb->type == WILC_CFG_PKT)
639 entry |= WILC_VMM_CFG_PKT;
> 640 return cpu_to_le32(entry);
641 }
642
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
next prev parent reply other threads:[~2021-12-28 12:48 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-23 1:14 [PATCH v2 00/50] wilc1000: rework tx path to use sk_buffs throughout David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 04/50] wilc1000: factor common code in wilc_wlan_cfg_set() and wilc_wlan_cfg_get() David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 01/50] wilc1000: don't hold txq_spinlock while initializing AC queue limits David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 31/50] wilc1000: eliminate another magic constant David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 29/50] wilc1000: factor header length calculation into a new function David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 18/50] wilc1000: split huge tx handler into subfunctions David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 03/50] wilc1000: move receive-queue stats from txq to wilc structure David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 11/50] wilc1000: convert tqx_entries from "int" to "atomic_t" David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 07/50] wilc1000: increment tx_dropped stat counter on tx packet drop David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 15/50] wilc1000: add struct wilc_skb_tx_cb as an alias of struct txq_entry_t David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 27/50] wilc1000: simplify ac_balance() a bit David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 20/50] wilc1000: eliminate "max_size_over" variable in fill_vmm_table David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 25/50] wilc1000: replace txq_spinlock with ack_filter_lock mutex David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 19/50] wilc1000: don't tell the chip to go to sleep while copying tx packets David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 13/50] wilc1000: sanitize config packet sequence number management a bit David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 22/50] wilc1000: minor syntax cleanup David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 12/50] wilc1000: refactor wilc_wlan_cfg_commit() a bit David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 14/50] wilc1000: if there is no tx packet, don't increment packets-sent counter David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 10/50] wilc1000: factor initialization of tx queue-specific packet fields David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 02/50] wilc1000: switch txq_event from completion to waitqueue David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 09/50] wilc1000: prepare wilc_wlan_tx_packet_done() for sk_buff changes David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 16/50] wilc1000: switch tx queue to normal sk_buff entries David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 23/50] wilc1000: introduce symbolic names for two tx-related control bits David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 24/50] wilc1000: protect tx_q_limit with a mutex instead of a spinlock David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 06/50] wilc1000: move tx packet drop code into its own function David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 28/50] wilc1000: improve send_packets() a bit David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 08/50] wilc1000: fix management packet type inconsistency David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 30/50] wilc1000: use more descriptive variable names David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 17/50] wilc1000: remove no longer used "vif" argument from init_txq_entry() David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 21/50] wilc1000: declare read-only ac_preserve_ratio as static and const David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 48/50] wilc1000: introduce function to find and check DMA response David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 26/50] wilc1000: reduce amount of time ack_filter_lock is held David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 45/50] wilc1000: move struct wilc_spi declaration David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 33/50] wilc1000: move ac_desired_ratio calculation to where its needed David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 37/50] wilc1000: introduce set_header() function David Mosberger-Tang
2021-12-28 15:00 ` kernel test robot
2021-12-23 1:14 ` [PATCH v2 46/50] wilc1000: remove duplicate CRC calculation code David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 47/50] wilc1000: factor SPI DMA command initialization code into a function David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 42/50] wilc1000: simplify code by adding header/padding to skb David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 40/50] wilc1000: introduce schedule_packets() function David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 34/50] wilc1000: restructure wilc-wlan_handle_txq() for clarity David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 50/50] wilc1000: add module parameter "disable_zero_copy_tx" to SPI driver David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 49/50] wilc1000: implement zero-copy transmit support for SPI David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 32/50] wilc1000: introduce vmm_table_entry() helper function David Mosberger-Tang
2021-12-28 12:47 ` kernel test robot [this message]
2021-12-23 1:14 ` [PATCH v2 39/50] wilc1000: eliminate txq_add_to_head_cs mutex David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 44/50] wilc1000: don't allocate tx_buffer when zero-copy is available David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 36/50] wilc1000: introduce transmit path chip queue David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 38/50] wilc1000: take advantage of " David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 05/50] wilc1000: add wilc_wlan_tx_packet_done() function David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 41/50] wilc1000: use more descriptive variable name David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 43/50] wilc1000: add support for zero-copy transmit of tx packets David Mosberger-Tang
2021-12-23 1:14 ` [PATCH v2 35/50] wilc1000: introduce copy_and_send_packets() helper function David Mosberger-Tang
2021-12-23 6:16 ` [PATCH v2 00/50] wilc1000: rework tx path to use sk_buffs throughout Ajay.Kathat
2021-12-23 6:41 ` Kalle Valo
2021-12-23 15:23 ` David Mosberger-Tang
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 \
--in-reply-to=202112282030.iPTnhHRr-lkp@intel.com \
--to=lkp@intel.com \
--cc=ajay.kathat@microchip.com \
--cc=claudiu.beznea@microchip.com \
--cc=davidm@egauge.net \
--cc=kbuild-all@lists.01.org \
--cc=kuba@kernel.org \
--cc=kvalo@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).