From barnabas.czeman at mainlining.org  Mon Nov  4 12:00:35 2024
From: barnabas.czeman at mainlining.org (=?utf-8?q?Barnab=C3=A1s_Cz=C3=A9m=C3=A1n?=)
Date: Mon, 04 Nov 2024 21:00:35 +0100
Subject: [PATCH] wifi: wcn36xx: fix channel survey memory allocation size
Message-ID: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>

KASAN reported a memory allocation issue in wcn->chan_survey
due to incorrect size calculation.
This commit uses kcalloc to allocate memory for wcn->chan_survey,
ensuring proper initialization and preventing the use of uninitialized
values when there are no frames on the channel.

Fixes: 29696e0aa413 ("wcn36xx: Track SNR and RSSI for each RX frame")
Signed-off-by: Barnab?s Cz?m?n <barnabas.czeman at mainlining.org>
---
 drivers/net/wireless/ath/wcn36xx/main.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c
index 408776562a7e56da3017aa074396bcd241d62f8c..cd36cab6db75d300f4f6617a6a9e1550f62921c7 100644
--- a/drivers/net/wireless/ath/wcn36xx/main.c
+++ b/drivers/net/wireless/ath/wcn36xx/main.c
@@ -1590,7 +1590,10 @@ static int wcn36xx_probe(struct platform_device *pdev)
 	}
 
 	n_channels = wcn_band_2ghz.n_channels + wcn_band_5ghz.n_channels;
-	wcn->chan_survey = devm_kmalloc(wcn->dev, n_channels, GFP_KERNEL);
+	wcn->chan_survey = devm_kcalloc(wcn->dev,
+					n_channels,
+					sizeof(struct wcn36xx_chan_survey),
+					GFP_KERNEL);
 	if (!wcn->chan_survey) {
 		ret = -ENOMEM;
 		goto out_wq;

---
base-commit: 1ffec08567f426a1c593e038cadc61bdc38cb467
change-id: 20241104-wcn36xx-memory-allocation-803e4e3de9a6

Best regards,
-- 
Barnab?s Cz?m?n <barnabas.czeman at mainlining.org>



From bryan.odonoghue at linaro.org  Mon Nov  4 14:23:22 2024
From: bryan.odonoghue at linaro.org (Bryan O'Donoghue)
Date: Mon, 4 Nov 2024 22:23:22 +0000
Subject: [PATCH] wifi: wcn36xx: fix channel survey memory allocation size
In-Reply-To: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>
References: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>
Message-ID: <78405309-b28d-4a7f-8cca-5410a9c05cb1@linaro.org>

On 04/11/2024 20:00, Barnab?s Cz?m?n wrote:
> KASAN reported a memory allocation issue in wcn->chan_survey
> due to incorrect size calculation.
> This commit uses kcalloc to allocate memory for wcn->chan_survey,
> ensuring proper initialization and preventing the use of uninitialized
> values when there are no frames on the channel.
> 
> Fixes: 29696e0aa413 ("wcn36xx: Track SNR and RSSI for each RX frame")
> Signed-off-by: Barnab?s Cz?m?n <barnabas.czeman at mainlining.org>
> ---
>   drivers/net/wireless/ath/wcn36xx/main.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c
> index 408776562a7e56da3017aa074396bcd241d62f8c..cd36cab6db75d300f4f6617a6a9e1550f62921c7 100644
> --- a/drivers/net/wireless/ath/wcn36xx/main.c
> +++ b/drivers/net/wireless/ath/wcn36xx/main.c
> @@ -1590,7 +1590,10 @@ static int wcn36xx_probe(struct platform_device *pdev)
>   	}
>   
>   	n_channels = wcn_band_2ghz.n_channels + wcn_band_5ghz.n_channels;
> -	wcn->chan_survey = devm_kmalloc(wcn->dev, n_channels, GFP_KERNEL);
> +	wcn->chan_survey = devm_kcalloc(wcn->dev,
> +					n_channels,
> +					sizeof(struct wcn36xx_chan_survey),
> +					GFP_KERNEL);
>   	if (!wcn->chan_survey) {
>   		ret = -ENOMEM;
>   		goto out_wq;
> 
> ---
> base-commit: 1ffec08567f426a1c593e038cadc61bdc38cb467
> change-id: 20241104-wcn36xx-memory-allocation-803e4e3de9a6
> 
> Best regards,
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue at linaro.org>


From u.kleine-koenig at baylibre.com  Wed Nov  6 09:07:06 2024
From: u.kleine-koenig at baylibre.com (=?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?=)
Date: Wed,  6 Nov 2024 18:07:06 +0100
Subject: [PATCH] wifi: Switch back to struct platform_driver::remove()
Message-ID: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>

After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
return void") .remove() is (again) the right callback to implement for
platform drivers.

Convert all platform drivers below drivers/net/wireless to use
.remove(), with the eventual goal to drop struct
platform_driver::remove_new(). As .remove() and .remove_new() have the
same prototypes, conversion is done by just changing the structure
member name in the driver initializer.

En passant several whitespace changes are done to make indentation
consistent in the struct initializers.

Signed-off-by: Uwe Kleine-K?nig <u.kleine-koenig at baylibre.com>
---
Hello,

I did a single patch for all of drivers/net/wireless. While I usually
prefer to do one logical change per patch, this seems to be
overengineering here as the individual changes are really trivial and
shouldn't be much in the way for stable backports. But I'll happily
split the patch if you prefer it split. Also if you object the
indentation stuff, I can rework that.

This is based on today's next, if conflicts arise when you apply it at
some later time and don't want to resolve them, feel free to just drop
the changes to the conflicting files. I'll notice and followup at a
later time then. Or ask me for a fixed resend. (Having said that, I
recommend b4 am -3 + git am -3 which should resolve most conflicts just
fine.)

Best regards
Uwe

 drivers/net/wireless/ath/ath10k/ahb.c                  |  8 ++++----
 drivers/net/wireless/ath/ath10k/snoc.c                 |  6 +++---
 drivers/net/wireless/ath/ath11k/ahb.c                  |  8 ++++----
 drivers/net/wireless/ath/ath5k/ahb.c                   |  8 ++++----
 drivers/net/wireless/ath/ath9k/ahb.c                   | 10 +++++-----
 drivers/net/wireless/ath/wcn36xx/main.c                |  8 ++++----
 .../net/wireless/broadcom/brcm80211/brcmfmac/common.c  |  2 +-
 drivers/net/wireless/mediatek/mt76/mt7603/soc.c        |  2 +-
 drivers/net/wireless/mediatek/mt76/mt7615/soc.c        |  2 +-
 drivers/net/wireless/mediatek/mt76/mt7915/soc.c        |  2 +-
 drivers/net/wireless/ti/wl12xx/main.c                  |  2 +-
 drivers/net/wireless/ti/wl18xx/main.c                  |  4 ++--
 12 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/drivers/net/wireless/ath/ath10k/ahb.c b/drivers/net/wireless/ath/ath10k/ahb.c
index f0441b3d7dcb..db9f9ebcb62d 100644
--- a/drivers/net/wireless/ath/ath10k/ahb.c
+++ b/drivers/net/wireless/ath/ath10k/ahb.c
@@ -837,12 +837,12 @@ static void ath10k_ahb_remove(struct platform_device *pdev)
 }
 
 static struct platform_driver ath10k_ahb_driver = {
-	.driver         = {
-		.name   = "ath10k_ahb",
+	.driver = {
+		.name = "ath10k_ahb",
 		.of_match_table = ath10k_ahb_of_match,
 	},
-	.probe  = ath10k_ahb_probe,
-	.remove_new = ath10k_ahb_remove,
+	.probe = ath10k_ahb_probe,
+	.remove = ath10k_ahb_remove,
 };
 
 int ath10k_ahb_init(void)
diff --git a/drivers/net/wireless/ath/ath10k/snoc.c b/drivers/net/wireless/ath/ath10k/snoc.c
index 0fe47d51013c..d436a874cd5a 100644
--- a/drivers/net/wireless/ath/ath10k/snoc.c
+++ b/drivers/net/wireless/ath/ath10k/snoc.c
@@ -1885,11 +1885,11 @@ static void ath10k_snoc_shutdown(struct platform_device *pdev)
 }
 
 static struct platform_driver ath10k_snoc_driver = {
-	.probe  = ath10k_snoc_probe,
-	.remove_new = ath10k_snoc_remove,
+	.probe = ath10k_snoc_probe,
+	.remove = ath10k_snoc_remove,
 	.shutdown = ath10k_snoc_shutdown,
 	.driver = {
-		.name   = "ath10k_snoc",
+		.name = "ath10k_snoc",
 		.of_match_table = ath10k_snoc_dt_match,
 	},
 };
diff --git a/drivers/net/wireless/ath/ath11k/ahb.c b/drivers/net/wireless/ath/ath11k/ahb.c
index 916402ad06b8..f2fc04596d48 100644
--- a/drivers/net/wireless/ath/ath11k/ahb.c
+++ b/drivers/net/wireless/ath/ath11k/ahb.c
@@ -1313,12 +1313,12 @@ static void ath11k_ahb_shutdown(struct platform_device *pdev)
 }
 
 static struct platform_driver ath11k_ahb_driver = {
-	.driver         = {
-		.name   = "ath11k",
+	.driver = {
+		.name = "ath11k",
 		.of_match_table = ath11k_ahb_of_match,
 	},
-	.probe  = ath11k_ahb_probe,
-	.remove_new = ath11k_ahb_remove,
+	.probe = ath11k_ahb_probe,
+	.remove = ath11k_ahb_remove,
 	.shutdown = ath11k_ahb_shutdown,
 };
 
diff --git a/drivers/net/wireless/ath/ath5k/ahb.c b/drivers/net/wireless/ath/ath5k/ahb.c
index f27308ccb2f1..cb3e891ee1bd 100644
--- a/drivers/net/wireless/ath/ath5k/ahb.c
+++ b/drivers/net/wireless/ath/ath5k/ahb.c
@@ -218,10 +218,10 @@ static void ath_ahb_remove(struct platform_device *pdev)
 }
 
 static struct platform_driver ath_ahb_driver = {
-	.probe      = ath_ahb_probe,
-	.remove_new = ath_ahb_remove,
-	.driver		= {
-		.name	= "ar231x-wmac",
+	.probe = ath_ahb_probe,
+	.remove = ath_ahb_remove,
+	.driver = {
+		.name = "ar231x-wmac",
 	},
 };
 
diff --git a/drivers/net/wireless/ath/ath9k/ahb.c b/drivers/net/wireless/ath/ath9k/ahb.c
index 1a6697b6e3b4..d4805e02b927 100644
--- a/drivers/net/wireless/ath/ath9k/ahb.c
+++ b/drivers/net/wireless/ath/ath9k/ahb.c
@@ -158,12 +158,12 @@ static void ath_ahb_remove(struct platform_device *pdev)
 }
 
 static struct platform_driver ath_ahb_driver = {
-	.probe      = ath_ahb_probe,
-	.remove_new = ath_ahb_remove,
-	.driver		= {
-		.name	= "ath9k",
+	.probe = ath_ahb_probe,
+	.remove = ath_ahb_remove,
+	.driver = {
+		.name = "ath9k",
 	},
-	.id_table    = ath9k_platform_id_table,
+	.id_table = ath9k_platform_id_table,
 };
 
 MODULE_DEVICE_TABLE(platform, ath9k_platform_id_table);
diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c
index 408776562a7e..8557d4826a46 100644
--- a/drivers/net/wireless/ath/wcn36xx/main.c
+++ b/drivers/net/wireless/ath/wcn36xx/main.c
@@ -1679,10 +1679,10 @@ static const struct of_device_id wcn36xx_of_match[] = {
 MODULE_DEVICE_TABLE(of, wcn36xx_of_match);
 
 static struct platform_driver wcn36xx_driver = {
-	.probe      = wcn36xx_probe,
-	.remove_new = wcn36xx_remove,
-	.driver         = {
-		.name   = "wcn36xx",
+	.probe = wcn36xx_probe,
+	.remove = wcn36xx_remove,
+	.driver = {
+		.name = "wcn36xx",
 		.of_match_table = wcn36xx_of_match,
 	},
 };
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
index 58d50918dd17..cfcf01eb0daa 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/common.c
@@ -594,7 +594,7 @@ static void brcmf_common_pd_remove(struct platform_device *pdev)
 }
 
 static struct platform_driver brcmf_pd = {
-	.remove_new	= brcmf_common_pd_remove,
+	.remove		= brcmf_common_pd_remove,
 	.driver		= {
 		.name	= BRCMFMAC_PDATA_NAME,
 	}
diff --git a/drivers/net/wireless/mediatek/mt76/mt7603/soc.c b/drivers/net/wireless/mediatek/mt76/mt7603/soc.c
index ec02148a7f1f..08590aa68356 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7603/soc.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7603/soc.c
@@ -71,7 +71,7 @@ MODULE_FIRMWARE(MT7628_FIRMWARE_E2);
 
 struct platform_driver mt76_wmac_driver = {
 	.probe		= mt76_wmac_probe,
-	.remove_new	= mt76_wmac_remove,
+	.remove		= mt76_wmac_remove,
 	.driver = {
 		.name = "mt76_wmac",
 		.of_match_table = of_wmac_match,
diff --git a/drivers/net/wireless/mediatek/mt76/mt7615/soc.c b/drivers/net/wireless/mediatek/mt76/mt7615/soc.c
index 12e3e4a91d27..06a0f2a141e8 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7615/soc.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7615/soc.c
@@ -63,7 +63,7 @@ struct platform_driver mt7622_wmac_driver = {
 		.of_match_table = mt7622_wmac_of_match,
 	},
 	.probe = mt7622_wmac_probe,
-	.remove_new = mt7622_wmac_remove,
+	.remove = mt7622_wmac_remove,
 };
 
 MODULE_FIRMWARE(MT7622_FIRMWARE_N9);
diff --git a/drivers/net/wireless/mediatek/mt76/mt7915/soc.c b/drivers/net/wireless/mediatek/mt76/mt7915/soc.c
index 90a6f61d1089..c823a7554a3a 100644
--- a/drivers/net/wireless/mediatek/mt76/mt7915/soc.c
+++ b/drivers/net/wireless/mediatek/mt76/mt7915/soc.c
@@ -1303,7 +1303,7 @@ struct platform_driver mt798x_wmac_driver = {
 		.of_match_table = mt798x_wmac_of_match,
 	},
 	.probe = mt798x_wmac_probe,
-	.remove_new = mt798x_wmac_remove,
+	.remove = mt798x_wmac_remove,
 };
 
 MODULE_FIRMWARE(MT7986_FIRMWARE_WA);
diff --git a/drivers/net/wireless/ti/wl12xx/main.c b/drivers/net/wireless/ti/wl12xx/main.c
index b26d42b4e3cc..ffbf54776330 100644
--- a/drivers/net/wireless/ti/wl12xx/main.c
+++ b/drivers/net/wireless/ti/wl12xx/main.c
@@ -1939,7 +1939,7 @@ MODULE_DEVICE_TABLE(platform, wl12xx_id_table);
 
 static struct platform_driver wl12xx_driver = {
 	.probe		= wl12xx_probe,
-	.remove_new	= wl12xx_remove,
+	.remove		= wl12xx_remove,
 	.id_table	= wl12xx_id_table,
 	.driver = {
 		.name	= "wl12xx_driver",
diff --git a/drivers/net/wireless/ti/wl18xx/main.c b/drivers/net/wireless/ti/wl18xx/main.c
index 39d8eebb9b6e..4be1110bac88 100644
--- a/drivers/net/wireless/ti/wl18xx/main.c
+++ b/drivers/net/wireless/ti/wl18xx/main.c
@@ -2097,9 +2097,9 @@ MODULE_DEVICE_TABLE(platform, wl18xx_id_table);
 
 static struct platform_driver wl18xx_driver = {
 	.probe		= wl18xx_probe,
-	.remove_new	= wlcore_remove,
+	.remove		= wlcore_remove,
 	.id_table	= wl18xx_id_table,
-	.driver = {
+	.driver		= {
 		.name	= "wl18xx_driver",
 	}
 };

base-commit: 5b913f5d7d7fe0f567dea8605f21da6eaa1735fb
-- 
2.45.2



From quic_jjohnson at quicinc.com  Wed Nov  6 10:22:56 2024
From: quic_jjohnson at quicinc.com (Jeff Johnson)
Date: Wed, 6 Nov 2024 10:22:56 -0800
Subject: [PATCH] wifi: Switch back to struct platform_driver::remove()
In-Reply-To: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
References: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
Message-ID: <5c6a887c-db74-44b4-9d8c-2e412f938b47@quicinc.com>

On 11/6/2024 9:07 AM, Uwe Kleine-K?nig wrote:
> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
> 
> Convert all platform drivers below drivers/net/wireless to use
> .remove(), with the eventual goal to drop struct
> platform_driver::remove_new(). As .remove() and .remove_new() have the
> same prototypes, conversion is done by just changing the structure
> member name in the driver initializer.
> 
> En passant several whitespace changes are done to make indentation
> consistent in the struct initializers.
> 
> Signed-off-by: Uwe Kleine-K?nig <u.kleine-koenig at baylibre.com>

I'll let Kalle make the decision if he wants to take this patch into wireless
or if he'd prefer that the parts that have driver-specific projects be split
out separately. If he'll take it as-is, for drivers/net/wireless/ath/*:

Acked-by: Jeff Johnson <quic_jjohnson at quicinc.com>



From toke at toke.dk  Thu Nov  7 01:35:13 2024
From: toke at toke.dk (Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?=)
Date: Thu, 07 Nov 2024 10:35:13 +0100
Subject: [PATCH] wifi: Switch back to struct platform_driver::remove()
In-Reply-To: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
References: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
Message-ID: <87fro375f2.fsf@toke.dk>

Uwe Kleine-K?nig <u.kleine-koenig at baylibre.com> writes:

> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
>
> Convert all platform drivers below drivers/net/wireless to use
> .remove(), with the eventual goal to drop struct
> platform_driver::remove_new(). As .remove() and .remove_new() have the
> same prototypes, conversion is done by just changing the structure
> member name in the driver initializer.
>
> En passant several whitespace changes are done to make indentation
> consistent in the struct initializers.
>
> Signed-off-by: Uwe Kleine-K?nig <u.kleine-koenig at baylibre.com>

For ath9k:

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


From arend.vanspriel at broadcom.com  Thu Nov  7 01:38:11 2024
From: arend.vanspriel at broadcom.com (Arend van Spriel)
Date: Thu, 7 Nov 2024 10:38:11 +0100
Subject: [PATCH] wifi: Switch back to struct platform_driver::remove()
In-Reply-To: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
References: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
Message-ID: <8f8c6ac8-159a-425e-9d9b-b31c956f700d@broadcom.com>

On 11/6/2024 6:07 PM, Uwe Kleine-K?nig wrote:
> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
> 
> Convert all platform drivers below drivers/net/wireless to use
> .remove(), with the eventual goal to drop struct
> platform_driver::remove_new(). As .remove() and .remove_new() have the
> same prototypes, conversion is done by just changing the structure
> member name in the driver initializer.
> 
> En passant several whitespace changes are done to make indentation
> consistent in the struct initializers.

I am fine with the patch as is, but it is really up to Kalle. For 
brcmfmac here is my...

Acked-by: Arend van Spriel <arend.vanspriel at broadcom.com>


From jirislaby at kernel.org  Thu Nov  7 02:01:14 2024
From: jirislaby at kernel.org (Jiri Slaby)
Date: Thu, 7 Nov 2024 11:01:14 +0100
Subject: [PATCH] wifi: Switch back to struct platform_driver::remove()
In-Reply-To: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
References: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
Message-ID: <d3890b5e-26e1-46ef-81ba-5427892517bf@kernel.org>

On 06. 11. 24, 18:07, Uwe Kleine-K?nig wrote:
> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
> 
> Convert all platform drivers below drivers/net/wireless to use
> .remove(), with the eventual goal to drop struct
> platform_driver::remove_new(). As .remove() and .remove_new() have the
> same prototypes, conversion is done by just changing the structure
> member name in the driver initializer.
> 
> En passant several whitespace changes are done to make indentation
> consistent in the struct initializers.
...
>   drivers/net/wireless/ath/ath5k/ahb.c                   |  8 ++++----

For the above:

Acked-by: Jiri Slaby <jirislaby at kernel.org>

-- 
js
suse labs


From loic.poulain at linaro.org  Thu Nov  7 02:27:46 2024
From: loic.poulain at linaro.org (Loic Poulain)
Date: Thu, 7 Nov 2024 11:27:46 +0100
Subject: [PATCH] wifi: wcn36xx: fix channel survey memory allocation size
In-Reply-To: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>
References: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>
Message-ID: <CAMZdPi_-uH6dk8KcrPRzNGJjX6YjkoNsYM1EJKJu5BYzrhd8gg@mail.gmail.com>

On Mon, 4 Nov 2024 at 21:00, Barnab?s Cz?m?n
<barnabas.czeman at mainlining.org> wrote:
>
> KASAN reported a memory allocation issue in wcn->chan_survey
> due to incorrect size calculation.
> This commit uses kcalloc to allocate memory for wcn->chan_survey,
> ensuring proper initialization and preventing the use of uninitialized
> values when there are no frames on the channel.
>
> Fixes: 29696e0aa413 ("wcn36xx: Track SNR and RSSI for each RX frame")
> Signed-off-by: Barnab?s Cz?m?n <barnabas.czeman at mainlining.org>

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


From kvalo at kernel.org  Mon Nov 11 04:10:58 2024
From: kvalo at kernel.org (Kalle Valo)
Date: Mon, 11 Nov 2024 12:10:58 +0000 (UTC)
Subject: [PATCH] wifi: Switch back to struct platform_driver::remove()
In-Reply-To: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
References: <20241106170706.38922-2-u.kleine-koenig@baylibre.com>
Message-ID: <173132705627.852485.14271331752921398347.kvalo@kernel.org>

Uwe Kleine-K?nig <u.kleine-koenig at baylibre.com> wrote:

> After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> return void") .remove() is (again) the right callback to implement for
> platform drivers.
> 
> Convert all platform drivers below drivers/net/wireless to use
> .remove(), with the eventual goal to drop struct
> platform_driver::remove_new(). As .remove() and .remove_new() have the
> same prototypes, conversion is done by just changing the structure
> member name in the driver initializer.
> 
> En passant several whitespace changes are done to make indentation
> consistent in the struct initializers.
> 
> Signed-off-by: Uwe Kleine-K?nig <u.kleine-koenig at baylibre.com>
> Acked-by: Jeff Johnson <quic_jjohnson at quicinc.com>
> Acked-by: Toke H?iland-J?rgensen <toke at toke.dk>
> Acked-by: Arend van Spriel <arend.vanspriel at broadcom.com>
> Acked-by: Jiri Slaby <jirislaby at kernel.org>

Patch applied to wireless-next.git, thanks.

b41f96ecb9b7 wifi: Switch back to struct platform_driver::remove()

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20241106170706.38922-2-u.kleine-koenig at baylibre.com/

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



From mieszko.gorecki at swiftloop.pl  Tue Nov 19 01:05:41 2024
From: mieszko.gorecki at swiftloop.pl (=?UTF-8?Q? Mieszko_G=C3=B3recki ?=)
Date: Tue, 19 Nov 2024 09:05:41 GMT
Subject: Zapytanie ofertowe 
Message-ID: <20241119084500-0.1.5b.f65j.0.8bourotoz0@swiftloop.pl>

Dzie? dobry,

Pozwoli?em sobie na kontakt, poniewa? jestem zainteresowany weryfikacj? mo?liwo?ci nawi?zania wsp??pracy.

Wspieramy firmy w pozyskiwaniu nowych klient?w biznesowych.

Czy mo?emy porozmawia? w celu przedstawienia szczeg??owych informacji? 


Pozdrawiam
Mieszko G?recki


From quic_jjohnson at quicinc.com  Thu Nov 21 08:12:39 2024
From: quic_jjohnson at quicinc.com (Jeff Johnson)
Date: Thu, 21 Nov 2024 08:12:39 -0800
Subject: [PATCH] wifi: wcn36xx: fix channel survey memory allocation
 size
In-Reply-To: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>
References: <20241104-wcn36xx-memory-allocation-v1-1-5ec901cf37b6@mainlining.org>
Message-ID: <173220555907.3016923.16789393852476367069.b4-ty@quicinc.com>


On Mon, 04 Nov 2024 21:00:35 +0100, Barnab?s Cz?m?n wrote:
> KASAN reported a memory allocation issue in wcn->chan_survey
> due to incorrect size calculation.
> This commit uses kcalloc to allocate memory for wcn->chan_survey,
> ensuring proper initialization and preventing the use of uninitialized
> values when there are no frames on the channel.
> 
> 
> [...]

Applied, thanks!

[1/1] wifi: wcn36xx: fix channel survey memory allocation size
      commit: 6200d947f050efdba4090dfefd8a01981363d954

Best regards,
-- 
Jeff Johnson <quic_jjohnson at quicinc.com>



