Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 4 Jul 2022 21:35:59 +0800
From:   kernel test robot <lkp@intel.com>
To:     Felix Fietkau <nbd@nbd.name>, linux-wireless@vger.kernel.org
Cc:     kbuild-all@lists.01.org
Subject: Re: [PATCH] mt76: remove q->qid
Message-ID: <202207042158.O7hTDMKY-lkp@intel.com>
References: <20220704074336.28043-1-nbd@nbd.name>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220704074336.28043-1-nbd@nbd.name>
Precedence: bulk

Hi Felix,

I love your patch! Yet something to improve:

[auto build test ERROR on wireless-next/main]
[also build test ERROR on wireless/main linus/master v5.19-rc5 next-20220704]
[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/intel-lab-lkp/linux/commits/Felix-Fietkau/mt76-remove-q-qid/20220704-154431
base:   https://git.kernel.org/pub/scm/linux/kernel/git/wireless/wireless-next.git main
config: i386-allyesconfig (https://download.01.org/0day-ci/archive/20220704/202207042158.O7hTDMKY-lkp@intel.com/config)
compiler: gcc-11 (Debian 11.3.0-3) 11.3.0
reproduce (this is a W=1 build):
        # https://github.com/intel-lab-lkp/linux/commit/7cef83a2b5a0f3c5ca3612e34ca2649ac9cf75f4
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Felix-Fietkau/mt76-remove-q-qid/20220704-154431
        git checkout 7cef83a2b5a0f3c5ca3612e34ca2649ac9cf75f4
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        make W=1 O=build_dir ARCH=i386 SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   drivers/net/wireless/mediatek/mt76/usb.c: In function 'mt76u_tx_queue_skb':
>> drivers/net/wireless/mediatek/mt76/usb.c:861:52: error: 'struct mt76_queue' has no member named 'qid'
     861 |         err = dev->drv->tx_prepare_skb(dev, NULL, q->qid, wcid, sta, &tx_info);
         |                                                    ^~
--
   drivers/net/wireless/mediatek/mt76/sdio.c: In function 'mt76s_tx_queue_skb':
>> drivers/net/wireless/mediatek/mt76/sdio.c:531:52: error: 'struct mt76_queue' has no member named 'qid'
     531 |         err = dev->drv->tx_prepare_skb(dev, NULL, q->qid, wcid, sta, &tx_info);
         |                                                    ^~
--
   drivers/net/wireless/mediatek/mt76/dma.c: In function 'mt76_dma_tx_queue_skb':
>> drivers/net/wireless/mediatek/mt76/dma.c:409:52: error: 'struct mt76_queue' has no member named 'qid'
     409 |         ret = dev->drv->tx_prepare_skb(dev, txwi, q->qid, wcid, sta, &tx_info);
         |                                                    ^~


vim +861 drivers/net/wireless/mediatek/mt76/usb.c

b40b15e1521f77 Lorenzo Bianconi 2018-07-31  845  
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  846  static int
8987059475f503 Lorenzo Bianconi 2020-11-11  847  mt76u_tx_queue_skb(struct mt76_dev *dev, struct mt76_queue *q,
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  848  		   struct sk_buff *skb, struct mt76_wcid *wcid,
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  849  		   struct ieee80211_sta *sta)
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  850  {
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  851  	struct mt76_tx_info tx_info = {
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  852  		.skb = skb,
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  853  	};
95f61e17eff3d9 Felix Fietkau    2020-08-23  854  	u16 idx = q->head;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  855  	int err;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  856  
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  857  	if (q->queued == q->ndesc)
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  858  		return -ENOSPC;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  859  
88046b2c9f6d8b Felix Fietkau    2018-10-25  860  	skb->prev = skb->next = NULL;
8987059475f503 Lorenzo Bianconi 2020-11-11 @861  	err = dev->drv->tx_prepare_skb(dev, NULL, q->qid, wcid, sta, &tx_info);
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  862  	if (err < 0)
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  863  		return err;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  864  
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  865  	err = mt76u_tx_setup_buffers(dev, tx_info.skb, q->entry[idx].urb);
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  866  	if (err < 0)
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  867  		return err;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  868  
4de92bf10c6df6 Lorenzo Bianconi 2019-02-20  869  	mt76u_fill_bulk_urb(dev, USB_DIR_OUT, q2ep(q->hw_idx),
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  870  			    q->entry[idx].urb, mt76u_complete_tx,
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  871  			    &q->entry[idx]);
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  872  
95f61e17eff3d9 Felix Fietkau    2020-08-23  873  	q->head = (q->head + 1) % q->ndesc;
cfaae9e67cf130 Lorenzo Bianconi 2019-04-02  874  	q->entry[idx].skb = tx_info.skb;
6d51cae28c8de9 Felix Fietkau    2021-05-09  875  	q->entry[idx].wcid = 0xffff;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  876  	q->queued++;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  877  
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  878  	return idx;
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  879  }
b40b15e1521f77 Lorenzo Bianconi 2018-07-31  880  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp