linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Felix Fietkau <nbd@nbd.name>
Cc: kbuild-all@01.org, linux-wireless@vger.kernel.org, kvalo@codeaurora.org
Subject: Re: [PATCH v7 3/3] mt76: add driver code for MT76x2e
Date: Thu, 23 Nov 2017 15:05:04 +0800	[thread overview]
Message-ID: <201711231440.jGwH44nB%fengguang.wu@intel.com> (raw)
In-Reply-To: <20171120193549.80831-4-nbd@nbd.name>

[-- Attachment #1: Type: text/plain, Size: 4427 bytes --]

Hi Felix,

I love your patch! Yet something to improve:

[auto build test ERROR on wireless-drivers-next/master]
[also build test ERROR on v4.14 next-20171122]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Felix-Fietkau/dt-bindings-net-add-mt76-wireless-device-binding/20171123-125723
base:   https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/wireless-drivers-next.git master
config: tile-allyesconfig (attached as .config)
compiler: tilegx-linux-gcc (GCC) 4.6.2
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=tile 

All errors (new ones prefixed by >>):

   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c: In function 'mt76x2_mac_process_rate':
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:49:20: error: 'RX_FLAG_SHORTPRE' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:49:20: note: each undeclared identifier is reported only once for each function it appears in
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:58:19: error: 'RX_FLAG_HT_GF' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:61:19: error: 'RX_FLAG_HT' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:65:19: error: 'RX_FLAG_VHT' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:67:9: error: 'struct ieee80211_rx_status' has no member named 'vht_nss'
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:75:19: error: 'RX_FLAG_LDPC' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:78:19: error: 'RX_FLAG_SHORT_GI' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:81:24: error: 'RX_FLAG_STBC_SHIFT' undeclared (first use in this function)
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:87:19: error: 'RX_FLAG_40MHZ' undeclared (first use in this function)
>> drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:90:9: error: 'struct ieee80211_rx_status' has no member named 'vht_flag'
   drivers/net/wireless/mediatek/mt76/mt76x2_mac.c:90:23: error: 'RX_VHT_FLAG_80MHZ' undeclared (first use in this function)

vim +90 drivers/net/wireless/mediatek/mt76/mt76x2_mac.c

    30	
    31	static void
    32	mt76x2_mac_process_rate(struct ieee80211_rx_status *status, u16 rate)
    33	{
    34		u8 idx = FIELD_GET(MT_RXWI_RATE_INDEX, rate);
    35	
    36		switch (FIELD_GET(MT_RXWI_RATE_PHY, rate)) {
    37		case MT_PHY_TYPE_OFDM:
    38			if (idx >= 8)
    39				idx = 0;
    40	
    41			if (status->band == NL80211_BAND_2GHZ)
    42				idx += 4;
    43	
    44			status->rate_idx = idx;
    45			return;
    46		case MT_PHY_TYPE_CCK:
    47			if (idx >= 8) {
    48				idx -= 8;
  > 49				status->flag |= RX_FLAG_SHORTPRE;
    50			}
    51	
    52			if (idx >= 4)
    53				idx = 0;
    54	
    55			status->rate_idx = idx;
    56			return;
    57		case MT_PHY_TYPE_HT_GF:
    58			status->flag |= RX_FLAG_HT_GF;
    59			/* fall through */
    60		case MT_PHY_TYPE_HT:
    61			status->flag |= RX_FLAG_HT;
    62			status->rate_idx = idx;
    63			break;
    64		case MT_PHY_TYPE_VHT:
    65			status->flag |= RX_FLAG_VHT;
    66			status->rate_idx = FIELD_GET(MT_RATE_INDEX_VHT_IDX, idx);
    67			status->vht_nss = FIELD_GET(MT_RATE_INDEX_VHT_NSS, idx) + 1;
    68			break;
    69		default:
    70			WARN_ON(1);
    71			return;
    72		}
    73	
    74		if (rate & MT_RXWI_RATE_LDPC)
    75			status->flag |= RX_FLAG_LDPC;
    76	
    77		if (rate & MT_RXWI_RATE_SGI)
    78			status->flag |= RX_FLAG_SHORT_GI;
    79	
    80		if (rate & MT_RXWI_RATE_STBC)
    81			status->flag |= 1 << RX_FLAG_STBC_SHIFT;
    82	
    83		switch (FIELD_GET(MT_RXWI_RATE_BW, rate)) {
    84		case MT_PHY_BW_20:
    85			break;
    86		case MT_PHY_BW_40:
    87			status->flag |= RX_FLAG_40MHZ;
    88			break;
    89		case MT_PHY_BW_80:
  > 90			status->vht_flag |= RX_VHT_FLAG_80MHZ;
    91			break;
    92		default:
    93			break;
    94		}
    95	}
    96	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 50543 bytes --]

  reply	other threads:[~2017-11-23  7:05 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-20 19:35 [PATCH v7 0/3] mt76: add new wireless driver for MediaTek MT76x2 PCIe chips Felix Fietkau
2017-11-20 19:35 ` [PATCH v7 1/3] dt-bindings: net: add mt76 wireless device binding Felix Fietkau
2017-11-20 21:34   ` Rob Herring
2017-11-21  7:08   ` [v7,1/3] " Kalle Valo
     [not found]   ` <20171121070837.AA354607EA@smtp.codeaurora.org>
2017-11-21  8:45     ` Felix Fietkau
2017-11-20 19:35 ` [PATCH v7 2/3] mt76: add common code shared between multiple chipsets Felix Fietkau
2017-11-20 19:35 ` [PATCH v7 3/3] mt76: add driver code for MT76x2e Felix Fietkau
2017-11-23  7:05   ` kbuild test robot [this message]
2017-11-23  7:42     ` Kalle Valo
2017-11-23  7:27   ` kbuild test robot
2017-11-23  7:27   ` [PATCH] mt76: fix debugfs_simple_attr.cocci warnings kbuild test robot
2017-12-07 13:42     ` Kalle Valo
2017-12-08  1:29       ` Fengguang Wu
2017-11-23  7:27   ` [PATCH] mt76: fix returnvar.cocci warnings kbuild test robot
2017-12-07 13:43     ` Kalle Valo

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=201711231440.jGwH44nB%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@01.org \
    --cc=kvalo@codeaurora.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=nbd@nbd.name \
    /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).