From kvalo at kernel.org  Fri Apr  1 04:49:58 2022
From: kvalo at kernel.org (Kalle Valo)
Date: Fri,  1 Apr 2022 11:49:58 +0000 (UTC)
Subject: [PATCH v3] wcn36xx: Improve readability of wcn36xx_caps_name
In-Reply-To: <20220328212912.283393-1-benni@stuerz.xyz>
References: <20220328212912.283393-1-benni@stuerz.xyz>
Message-ID: <164881379683.6665.4776663816367931792.kvalo@kernel.org>

Benjamin St?rz <benni at stuerz.xyz> wrote:

> Use macros to force strict ordering of the elements.
> 
> Signed-off-by: Benjamin St?rz <benni at stuerz.xyz>
> Reviewed-by: Jeff Johnson <quic_jjohnson at quicinc.com>
> Signed-off-by: Kalle Valo <quic_kvalo at quicinc.com>

Patch applied to ath-next branch of ath.git, thanks.

9149a94adad2 wcn36xx: Improve readability of wcn36xx_caps_name

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20220328212912.283393-1-benni at stuerz.xyz/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches



From f_penna at alice.it  Sun Apr  3 21:12:52 2022
From: f_penna at alice.it (We have an offer to invest in your country under a joint venture partnership please reply for more details)
Date: 03 Apr 2022 21:12:52 -0700
Subject: No subject
Message-ID: <20220403211252.A905E71EEDF5545A@alice.it>



From yang.lee at linux.alibaba.com  Thu Apr  7 17:01:12 2022
From: yang.lee at linux.alibaba.com (Yang Li)
Date: Fri,  8 Apr 2022 08:01:12 +0800
Subject: [PATCH -next 1/2] wcn36xx: clean up some inconsistent indenting
Message-ID: <20220408000113.129906-1-yang.lee@linux.alibaba.com>

Eliminate the follow smatch warning:
drivers/net/wireless/ath/wcn36xx/smd.c:3151
wcn36xx_smd_gtk_offload_get_info_rsp() warn: inconsistent indenting

Reported-by: Abaci Robot <abaci at linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee at linux.alibaba.com>
---
 drivers/net/wireless/ath/wcn36xx/smd.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/wcn36xx/smd.c b/drivers/net/wireless/ath/wcn36xx/smd.c
index 872b37875c57..691e637b94f1 100644
--- a/drivers/net/wireless/ath/wcn36xx/smd.c
+++ b/drivers/net/wireless/ath/wcn36xx/smd.c
@@ -3148,9 +3148,9 @@ static int wcn36xx_smd_gtk_offload_get_info_rsp(struct wcn36xx *wcn,
 			cpu_to_le64(rsp->key_replay_counter);
 		ieee80211_gtk_rekey_notify(vif, vif->bss_conf.bssid,
 					   (void *)&replay_ctr, GFP_KERNEL);
-		 wcn36xx_dbg(WCN36XX_DBG_HAL,
-			     "GTK replay counter increment %llu\n",
-			     rsp->key_replay_counter);
+		wcn36xx_dbg(WCN36XX_DBG_HAL,
+			    "GTK replay counter increment %llu\n",
+			    rsp->key_replay_counter);
 	}
 
 	wcn36xx_dbg(WCN36XX_DBG_HAL,
-- 
2.20.1.7.g153144c



From yang.lee at linux.alibaba.com  Thu Apr  7 17:01:13 2022
From: yang.lee at linux.alibaba.com (Yang Li)
Date: Fri,  8 Apr 2022 08:01:13 +0800
Subject: [PATCH -next 2/2] ath9k: Remove unnecessary print function dev_err()
In-Reply-To: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
References: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
Message-ID: <20220408000113.129906-2-yang.lee@linux.alibaba.com>

The print function dev_err() is redundant because
platform_get_irq_byname() already prints an error.

Eliminate the follow coccicheck warning:
./drivers/net/wireless/ath/ath9k/ahb.c:103:2-9: line 103 is redundant
because platform_get_irq() already prints an error

Reported-by: Abaci Robot <abaci at linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee at linux.alibaba.com>
---
 drivers/net/wireless/ath/ath9k/ahb.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/ath9k/ahb.c b/drivers/net/wireless/ath/ath9k/ahb.c
index c9b853af41d1..9cd12b20b18d 100644
--- a/drivers/net/wireless/ath/ath9k/ahb.c
+++ b/drivers/net/wireless/ath/ath9k/ahb.c
@@ -99,10 +99,8 @@ static int ath_ahb_probe(struct platform_device *pdev)
 	}
 
 	irq = platform_get_irq(pdev, 0);
-	if (irq < 0) {
-		dev_err(&pdev->dev, "no IRQ resource found\n");
+	if (irq < 0)
 		return irq;
-	}
 
 	ath9k_fill_chanctx_ops();
 	hw = ieee80211_alloc_hw(sizeof(struct ath_softc), &ath9k_ops);
-- 
2.20.1.7.g153144c



From loic.poulain at linaro.org  Thu Apr  7 23:42:00 2022
From: loic.poulain at linaro.org (Loic Poulain)
Date: Fri, 8 Apr 2022 08:42:00 +0200
Subject: [PATCH -next 1/2] wcn36xx: clean up some inconsistent indenting
In-Reply-To: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
References: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
Message-ID: <CAMZdPi9kA0iLB64B6pCwt+2tUL2Lv9nYbU_N-0J6tSORSfq9_A@mail.gmail.com>

On Fri, 8 Apr 2022 at 02:01, Yang Li <yang.lee at linux.alibaba.com> wrote:
>
> Eliminate the follow smatch warning:
> drivers/net/wireless/ath/wcn36xx/smd.c:3151
> wcn36xx_smd_gtk_offload_get_info_rsp() warn: inconsistent indenting
>
> Reported-by: Abaci Robot <abaci at linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee at linux.alibaba.com>

Acked-by: Loic Poulain <loic.poulain at linaro.org>


From toke at toke.dk  Fri Apr  8 04:53:35 2022
From: toke at toke.dk (Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?=)
Date: Fri, 08 Apr 2022 13:53:35 +0200
Subject: [PATCH -next 2/2] ath9k: Remove unnecessary print function
 dev_err()
In-Reply-To: <20220408000113.129906-2-yang.lee@linux.alibaba.com>
References: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
 <20220408000113.129906-2-yang.lee@linux.alibaba.com>
Message-ID: <87h77394g0.fsf@toke.dk>

Yang Li <yang.lee at linux.alibaba.com> writes:

> The print function dev_err() is redundant because
> platform_get_irq_byname() already prints an error.
>
> Eliminate the follow coccicheck warning:
> ./drivers/net/wireless/ath/ath9k/ahb.c:103:2-9: line 103 is redundant
> because platform_get_irq() already prints an error
>
> Reported-by: Abaci Robot <abaci at linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee at linux.alibaba.com>

Heh, so I was the one originally insisting on keeping that err print,
but looking a bit closer now it does really appear to be redundant, so:

Acked-by: Toke H?iland-J?rgensen <toke at toke.dk>


From kvalo at kernel.org  Tue Apr 12 06:13:14 2022
From: kvalo at kernel.org (Kalle Valo)
Date: Tue, 12 Apr 2022 13:13:14 +0000 (UTC)
Subject: [PATCH -next 1/2] wcn36xx: clean up some inconsistent indenting
In-Reply-To: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
References: <20220408000113.129906-1-yang.lee@linux.alibaba.com>
Message-ID: <164976919327.15500.7970749626802998679.kvalo@kernel.org>

Yang Li <yang.lee at linux.alibaba.com> wrote:

> Eliminate the follow smatch warning:
> drivers/net/wireless/ath/wcn36xx/smd.c:3151
> wcn36xx_smd_gtk_offload_get_info_rsp() warn: inconsistent indenting
> 
> Reported-by: Abaci Robot <abaci at linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee at linux.alibaba.com>
> Acked-by: Loic Poulain <loic.poulain at linaro.org>
> Signed-off-by: Kalle Valo <quic_kvalo at quicinc.com>

Patch applied to ath-next branch of ath.git, thanks.

2578171ff85e wcn36xx: clean up some inconsistent indenting

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20220408000113.129906-1-yang.lee at linux.alibaba.com/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches



From kvalo at kernel.org  Tue Apr 12 06:13:48 2022
From: kvalo at kernel.org (Kalle Valo)
Date: Tue, 12 Apr 2022 13:13:48 +0000 (UTC)
Subject: [PATCH -next 2/2] ath9k: Remove unnecessary print function
 dev_err()
In-Reply-To: <20220408000113.129906-2-yang.lee@linux.alibaba.com>
References: <20220408000113.129906-2-yang.lee@linux.alibaba.com>
Message-ID: <164976922680.15500.1095811162535053166.kvalo@kernel.org>

Yang Li <yang.lee at linux.alibaba.com> wrote:

> The print function dev_err() is redundant because
> platform_get_irq_byname() already prints an error.
> 
> Eliminate the follow coccicheck warning:
> ./drivers/net/wireless/ath/ath9k/ahb.c:103:2-9: line 103 is redundant
> because platform_get_irq() already prints an error
> 
> Reported-by: Abaci Robot <abaci at linux.alibaba.com>
> Signed-off-by: Yang Li <yang.lee at linux.alibaba.com>
> Acked-by: Toke H?iland-J?rgensen <toke at toke.dk>
> Signed-off-by: Kalle Valo <quic_kvalo at quicinc.com>

Patch applied to ath-next branch of ath.git, thanks.

d7ceee8051ba ath9k: Remove unnecessary print function dev_err()

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20220408000113.129906-2-yang.lee at linux.alibaba.com/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches



