Re: [PATCH v3 4/4] staging: rtl8723bs: fix line length and alignment issues
From: Greg KH
Date: Fri Jan 16 2026 - 08:14:22 EST
On Fri, Jan 16, 2026 at 11:00:52AM +0530, Khushal Chitturi wrote:
> /* (1) RTS_Threshold is compared to the MPDU, not MSDU. */
> - /* (2) If there are more than one frag in this MSDU, only the first frag uses protection frame. */
> + /* (2) If there are more than one frag in this MSDU,
> + * only the first frag uses protection frame.
> + */
Break a sentance on the line length, not the ',' character.
Otherwise, you get odd looking lines like this one:
> @@ -656,7 +672,9 @@ static s32 update_attrib(struct adapter *padapter, struct sk_buff *pkt, struct p
> pattrib->pktlen = pktfile.pkt_len;
>
> if (pattrib->ether_type == ETH_P_IP) {
> - /* The following is for DHCP and ARP packet, we use cck1M to tx these packets and let LPS awake some time */
> + /* The following is for DHCP and ARP packet,
> + * we use cck1M to tx these packets and let LPS awake some time
> + */
Thanks,
greg k-h