Re: [PATCH 0/7] mmc/wifi/bluetooth: store owner from modules with sdio_register_driver()

From: Ulf Hansson
Date: Tue Apr 02 2024 - 06:59:13 EST


On Fri, 29 Mar 2024 at 18:24, Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxx> wrote:
>
> Merging
> =======
> All further patches depend on the first patch. Everything could go via
> one tree, e.g. MMC, or the cleanup patches removing owner would wait a
> cycle.

Patch 1 applied for next, thanks!

I can certainly pick the remaining in the series through my mmc tree,
but waiting a bit to queue them up to allow some more acks to be
received.

Kind regards
Uffe


>
> Description
> ===========
> Modules registering driver with sdio_register_driver() might
> forget to set .owner field.
>
> Solve the problem by moving this task away from the drivers to the core
> code, just like we did for platform_driver in commit 9447057eaff8
> ("platform_device: use a macro instead of platform_driver_register").
>
> Best regards,
> Krzysztof
>
> ---
> Krzysztof Kozlowski (7):
> mmc: sdio: store owner from modules with sdio_register_driver()
> bluetooth: btmrvl_sdio: drop driver owner initialization
> bluetooth: btmtksdio: drop driver owner initialization
> wifi: ath10k: sdio: drop driver owner initialization
> wifi: brcm80211: drop driver owner initialization
> wifi: marvell: mwifiex: drop driver owner initialization
> wifi: silabs: wfx: drop driver owner initialization
>
> drivers/bluetooth/btmrvl_sdio.c | 1 -
> drivers/bluetooth/btmtksdio.c | 1 -
> drivers/mmc/core/sdio_bus.c | 9 ++++++---
> drivers/net/wireless/ath/ath10k/sdio.c | 1 -
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c | 1 -
> drivers/net/wireless/marvell/mwifiex/sdio.c | 1 -
> drivers/net/wireless/silabs/wfx/bus_sdio.c | 1 -
> include/linux/mmc/sdio_func.h | 5 ++++-
> 8 files changed, 10 insertions(+), 10 deletions(-)
> ---
> base-commit: 087c142b2b04898c897aa77938d05a93907150e5
> change-id: 20240329-module-owner-sdio-abd5de3f1d74
>
> Best regards,
> --
> Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>