linux-kernel - Re: [PATCH] wifi: ath11k: Remove redundant semicolons

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <b2efb1f2-df89-4c88-8f97-e164e89c7532@oss.qualcomm.com>
Date: Wed, 17 Sep 2025 13:29:47 -0700
From: Jeff Johnson <jeff.johnson@....qualcomm.com>
To: Liao Yuanhong <liaoyuanhong@...o.com>, Jeff Johnson
 <jjohnson@...nel.org>,
        "open list:QUALCOMM ATHEROS ATH11K WIRELESS DRIVER"
 <linux-wireless@...r.kernel.org>,
        "open list:QUALCOMM ATHEROS ATH11K WIRELESS DRIVER"
 <ath11k@...ts.infradead.org>,
        open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] wifi: ath11k: Remove redundant semicolons

On 8/12/2025 12:52 AM, Liao Yuanhong wrote:
> Remove unnecessary semicolons.

There is only one so s/semicolons/semicolon/ here and in subject

> 
> Fixes: d5c65159f2895 ("ath11k: driver for Qualcomm IEEE 802.11ax devices")

Drop this since this isn't a bug fix that needs to be backported

> Signed-off-by: Liao Yuanhong <liaoyuanhong@...o.com>
> ---
>  drivers/net/wireless/ath/ath11k/dp_rx.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/ath11k/dp_rx.c b/drivers/net/wireless/ath/ath11k/dp_rx.c
> index ffc7482c77b6..b9e976ddcbbf 100644
> --- a/drivers/net/wireless/ath/ath11k/dp_rx.c
> +++ b/drivers/net/wireless/ath/ath11k/dp_rx.c
> @@ -4615,7 +4615,6 @@ static void ath11k_hal_rx_msdu_list_get(struct ath11k *ar,
>  			      msdu_details[i].buf_addr_info.info0) == 0) {
>  			msdu_desc_info = &msdu_details[i - 1].rx_msdu_info;
>  			msdu_desc_info->info0 |= last;
> -			;
>  			break;
>  		}
>  		msdu_desc_info = &msdu_details[i].rx_msdu_info;
No need to repost -- I'll incorporate these changes when I process the patch.

/jeff