Re: [PATCH 02/19] wifi: rtw88: rx: mark zero length packets on RTL8723B

[Bitterblue Smith]

On 24/07/2026 21:18, luka.gejak@xxxxxxxxx wrote:
> From: Luka Gejak <luka.gejak@xxxxxxxxx>
> 
> Like the RTL8703B, the RTL8723B reports receive descriptors with a zero
> packet length, which the vendor driver drops outright. rtw88 already
> flags these as having no PSDU for the RTL8703B, so extend the same
> handling to the RTL8723B rather than passing an empty frame up.
> 

I wonder if this only happens with SDIO? I don't think RTL8723BU does it.

> Signed-off-by: Luka Gejak <luka.gejak@xxxxxxxxx>
> ---
>  drivers/net/wireless/realtek/rtw88/rx.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/wireless/realtek/rtw88/rx.c b/drivers/net/wireless/realtek/rtw88/rx.c
> index 01fd299abb7f..57facbc1b41f 100644
> --- a/drivers/net/wireless/realtek/rtw88/rx.c
> +++ b/drivers/net/wireless/realtek/rtw88/rx.c
> @@ -253,10 +253,12 @@ static void rtw_rx_fill_rx_status(struct rtw_dev *rtwdev,
>  
>  	rtw_rx_addr_match(rtwdev, pkt_stat, hdr);
>  
> -	/* Rtl8723cs driver checks for size < 14 or size > 8192 and
> -	 * simply drops the packet.
> +	/* Rtl8723cs and rtl8723bs drivers check for size < 14 or size > 8192
> +	 * and simply drop the packet.
>  	 */
> -	if (rtwdev->chip->id == RTW_CHIP_TYPE_8703B && pkt_stat->pkt_len == 0) {
> +	if ((rtwdev->chip->id == RTW_CHIP_TYPE_8703B ||
> +	     rtwdev->chip->id == RTW_CHIP_TYPE_8723B) &&
> +	    pkt_stat->pkt_len == 0) {
>  		rx_status->flag |= RX_FLAG_NO_PSDU;
>  		rtw_dbg(rtwdev, RTW_DBG_RX, "zero length packet");
>  	}