| Message ID | 20240329-module-owner-sdio-v1-5-e4010b11ccaa@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | mmc/wifi/bluetooth: store owner from modules with sdio_register_driver() | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | success | Success |
| tedd_an/CheckPatch | success | CheckPatch PASS |
| tedd_an/GitLint | success | Gitlint PASS |
| tedd_an/SubjectPrefix | fail | "Bluetooth: " prefix is not specified in the subject |
| tedd_an/IncrementalBuild | success | Incremental Build PASS |
On March 29, 2024 6:24:57 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > Core in sdio_register_driver() already sets the .owner, so driver does > not need to. Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Depends on the first patch. > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c > index 00679a990e3d..13391c2d82aa 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c > @@ -1238,7 +1238,6 @@ static struct sdio_driver brcmf_sdmmc_driver = { > .name = KBUILD_MODNAME, > .id_table = brcmf_sdmmc_ids, > .drv = { > - .owner = THIS_MODULE, > .pm = pm_sleep_ptr(&brcmf_sdio_pm_ops), > .coredump = brcmf_dev_coredump, > }, > > -- > 2.34.1
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c index 00679a990e3d..13391c2d82aa 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c @@ -1238,7 +1238,6 @@ static struct sdio_driver brcmf_sdmmc_driver = { .name = KBUILD_MODNAME, .id_table = brcmf_sdmmc_ids, .drv = { - .owner = THIS_MODULE, .pm = pm_sleep_ptr(&brcmf_sdio_pm_ops), .coredump = brcmf_dev_coredump, },
Core in sdio_register_driver() already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Depends on the first patch. --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c | 1 - 1 file changed, 1 deletion(-)