WARNING: line length of 82 exceeds 80 columns #56: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:35: +#define BCM4323X_RX_FRAME_OFFSET (sizeof(struct bcm4323x_rxhdr) + D11_PHY_HDR_LEN) WARNING: line length of 96 exceeds 80 columns #59: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:38: +static bool bcm4323x_rx_key_match(struct bcm4323x *dev, struct sk_buff *skb, unsigned int index) WARNING: line length of 86 exceeds 80 columns #69: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:48: + if (ieee80211_get_hdrlen_from_skb(skb) < sizeof(struct ieee80211_hdr_3addr) || WARNING: line length of 86 exceeds 80 columns #70: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:49: + !ieee80211_has_protected(hdr->frame_control) || index >= BCM4323X_HW_KEYS) WARNING: line length of 90 exceeds 80 columns #85: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:64: + frame_bssid = ieee80211_get_bssid((void *)skb->data, skb->len, vif->type); WARNING: line length of 89 exceeds 80 columns #86: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:65: + addressed = frame_bssid && bssid && ether_addr_equal(frame_bssid, bssid); WARNING: line length of 90 exceeds 80 columns #101: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:80: +static void bcm4323x_set_rx_rate(struct bcm4323x *dev, struct ieee80211_rx_status *status, WARNING: line length of 81 exceeds 80 columns #105: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:84: + const struct ieee80211_supported_band *sband = &dev->bands[status->band]; WARNING: line length of 87 exceeds 80 columns #129: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:108: + /* The HT-SIG byte has the STBC field where plcp3_issgi() finds SGI. */ WARNING: line length of 88 exceeds 80 columns #130: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:109: + status->enc_flags |= rspec_stc(plcp[3] << 16) << RX_ENC_FLAG_STBC_SHIFT; WARNING: line length of 83 exceeds 80 columns #149: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:128: +/* Whether a procedure 0x51 body holds a whole frame, on a channel of the device */ WARNING: line length of 87 exceeds 80 columns #150: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:129: +static bool bcm4323x_rx_valid(struct bcm4323x *dev, const u8 *body, size_t body_length) WARNING: line length of 92 exceeds 80 columns #174: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:153: +static unsigned int bcm4323x_rx_channel(struct bcm4323x *dev, const struct d11rxhdr_le *rxh) WARNING: line length of 87 exceeds 80 columns #188: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:167: +static void bcm4323x_rx_80211(struct bcm4323x *dev, const u8 *body, size_t body_length) WARNING: line length of 84 exceeds 80 columns #212: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:191: + /* Aligns the IP header of QoS data, which mac80211 would otherwise move. */ WARNING: line length of 83 exceeds 80 columns #220: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:199: + status->band = le16_to_cpu(rxh->RxChan) & RXS_CHAN_5G ? NL80211_BAND_5GHZ : WARNING: line length of 82 exceeds 80 columns #221: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:200: + NL80211_BAND_2GHZ; WARNING: line length of 81 exceeds 80 columns #225: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:204: + if ((le16_to_cpu(rxh->RxStatus2) & RXS_PHYRXST_VALID) && hdr->rssi < 0) { WARNING: line length of 86 exceeds 80 columns #239: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:218: + bcm4323x_rx_key_match(dev, skb, FIELD_GET(RXS_SECKINDX_MASK, rx_status1))) WARNING: line length of 81 exceeds 80 columns #244: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:223: +static void bcm4323x_handle_handshake_reply(struct bcm4323x *dev, const u8 *body, WARNING: line length of 97 exceeds 80 columns #250: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:229: + if (body_length == sizeof(*reply) && le32_to_cpu(reply->version) == BCM4323X_RPC_VERSION) WARNING: line length of 87 exceeds 80 columns #255: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:234: +static void bcm4323x_rx_noise(struct bcm4323x *dev, const u8 *body, size_t body_length) WARNING: line length of 95 exceeds 80 columns #276: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:255: +static void bcm4323x_handle_runtime_frame(struct bcm4323x *dev, const u8 *frame, size_t length) WARNING: line length of 87 exceeds 80 columns #279: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:258: + u8 rpc_class = FIELD_GET(BCM4323X_RPC_CLASS_MASK, le32_to_cpu(rpc->xid_class)); WARNING: line length of 94 exceeds 80 columns #311: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:290: + if (urb->status == -ENOENT || urb->status == -ECONNRESET || urb->status == -ESHUTDOWN) WARNING: line length of 89 exceeds 80 columns #314: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:293: + dev_err_ratelimited(&dev->intf->dev, "RX URB failed: %d\n", urb->status); WARNING: line length of 85 exceeds 80 columns #337: FILE: drivers/net/wireless/broadcom/brcm80211/bcm4323x/bcm4323x_rx.c:316: + dev_warn_ratelimited(&dev->intf->dev, "RX URB left idle: %d\n", ret); total: 0 errors, 27 warnings, 0 checks, 317 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit a7c52ab6856b ("wifi: bcm4323x: add the receive path") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE FILE_PATH_CHANGES GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 27 warnings, 0 checks, 317 lines checked