* [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe @ 2021-08-17 6:35 Ahmad Fatoum 2021-08-17 10:01 ` Ahmad Fatoum ` (2 more replies) 0 siblings, 3 replies; 9+ messages in thread From: Ahmad Fatoum @ 2021-08-17 6:35 UTC (permalink / raw) To: Arend van Spriel, Franky Lin, Hante Meuleman, Chi-hsien Lin, Wright Feng, Chung-hsien Hsu Cc: kernel, Ahmad Fatoum, SHA-cyfmac-dev-list, brcm80211-dev-list.pdl, netdev, linux-wireless, Kalle Valo, Jakub Kicinski, David S. Miller, linux-kernel When resuming from suspend, brcmf_pcie_pm_leave_D3 will first attempt a hot resume and then fall back to removing the PCI device and then reprobing. If this probe fails, the kernel will oops, because brcmf_err, which is called to report the failure will dereference the stale bus pointer. Open code and use the default bus-less brcmf_err to avoid this. Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de> --- To: Arend van Spriel <aspriel@gmail.com> To: Franky Lin <franky.lin@broadcom.com> To: Hante Meuleman <hante.meuleman@broadcom.com> To: Chi-hsien Lin <chi-hsien.lin@infineon.com> To: Wright Feng <wright.feng@infineon.com> To: Chung-hsien Hsu <chung-hsien.hsu@infineon.com> Cc: SHA-cyfmac-dev-list@infineon.com Cc: brcm80211-dev-list.pdl@broadcom.com Cc: netdev@vger.kernel.org Cc: linux-wireless@vger.kernel.org Cc: Kalle Valo <kvalo@codeaurora.org> Cc: Jakub Kicinski <kuba@kernel.org> Cc: "David S. Miller" <davem@davemloft.net> Cc: linux-kernel@vger.kernel.org --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c index 9ef94d7a7ca7..d824bea4b79d 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c @@ -2209,7 +2209,7 @@ static int brcmf_pcie_pm_leave_D3(struct device *dev) err = brcmf_pcie_probe(pdev, NULL); if (err) - brcmf_err(bus, "probe after resume failed, err=%d\n", err); + __brcmf_err(NULL, __func__, "probe after resume failed, err=%d\n", err); return err; } -- 2.30.2 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 6:35 [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe Ahmad Fatoum @ 2021-08-17 10:01 ` Ahmad Fatoum [not found] ` <CAHp75Vfc_T04p95PgVUd+CK+ttPwX2aOC4WPD35Z01WQV1MxKw@mail.gmail.com> 2021-08-29 11:45 ` Kalle Valo 2 siblings, 0 replies; 9+ messages in thread From: Ahmad Fatoum @ 2021-08-17 10:01 UTC (permalink / raw) To: Arend van Spriel, Franky Lin, Hante Meuleman, Chi-hsien Lin, Wright Feng, Chung-hsien Hsu Cc: kernel, SHA-cyfmac-dev-list, brcm80211-dev-list.pdl, netdev, linux-wireless, Kalle Valo, Jakub Kicinski, David S. Miller, linux-kernel, rafal On 17.08.21 08:35, Ahmad Fatoum wrote: > When resuming from suspend, brcmf_pcie_pm_leave_D3 will first attempt a > hot resume and then fall back to removing the PCI device and then > reprobing. If this probe fails, the kernel will oops, because brcmf_err, > which is called to report the failure will dereference the stale bus > pointer. Open code and use the default bus-less brcmf_err to avoid this. Should've included a Fixes tag: Fixes: 8602e62441ab ("brcmfmac: pass bus to the __brcmf_err() in pcie.c") Please let me know if I should resend with the tag added. Cheers, Ahmad > Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de> > --- > To: Arend van Spriel <aspriel@gmail.com> > To: Franky Lin <franky.lin@broadcom.com> > To: Hante Meuleman <hante.meuleman@broadcom.com> > To: Chi-hsien Lin <chi-hsien.lin@infineon.com> > To: Wright Feng <wright.feng@infineon.com> > To: Chung-hsien Hsu <chung-hsien.hsu@infineon.com> > Cc: SHA-cyfmac-dev-list@infineon.com > Cc: brcm80211-dev-list.pdl@broadcom.com > Cc: netdev@vger.kernel.org > Cc: linux-wireless@vger.kernel.org > Cc: Kalle Valo <kvalo@codeaurora.org> > Cc: Jakub Kicinski <kuba@kernel.org> > Cc: "David S. Miller" <davem@davemloft.net> > Cc: linux-kernel@vger.kernel.org > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c > index 9ef94d7a7ca7..d824bea4b79d 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c > @@ -2209,7 +2209,7 @@ static int brcmf_pcie_pm_leave_D3(struct device *dev) > > err = brcmf_pcie_probe(pdev, NULL); > if (err) > - brcmf_err(bus, "probe after resume failed, err=%d\n", err); > + __brcmf_err(NULL, __func__, "probe after resume failed, err=%d\n", err); > > return err; > } > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 9+ messages in thread
[parent not found: <CAHp75Vfc_T04p95PgVUd+CK+ttPwX2aOC4WPD35Z01WQV1MxKw@mail.gmail.com>]
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe [not found] ` <CAHp75Vfc_T04p95PgVUd+CK+ttPwX2aOC4WPD35Z01WQV1MxKw@mail.gmail.com> @ 2021-08-17 11:11 ` Ahmad Fatoum 2021-08-17 11:54 ` Andy Shevchenko 0 siblings, 1 reply; 9+ messages in thread From: Ahmad Fatoum @ 2021-08-17 11:11 UTC (permalink / raw) To: Andy Shevchenko Cc: Arend van Spriel, Franky Lin, Hante Meuleman, Chi-hsien Lin, Wright Feng, Chung-hsien Hsu, kernel, SHA-cyfmac-dev-list, brcm80211-dev-list.pdl, netdev, linux-wireless, Kalle Valo, Jakub Kicinski, David S. Miller, linux-kernel On 17.08.21 13:02, Andy Shevchenko wrote: > On Tuesday, August 17, 2021, Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > >> When resuming from suspend, brcmf_pcie_pm_leave_D3 will first attempt a >> hot resume and then fall back to removing the PCI device and then >> reprobing. If this probe fails, the kernel will oops, because brcmf_err, >> which is called to report the failure will dereference the stale bus >> pointer. Open code and use the default bus-less brcmf_err to avoid this. >> >> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de> >> --- >> To: Arend van Spriel <aspriel@gmail.com> >> To: Franky Lin <franky.lin@broadcom.com> >> To: Hante Meuleman <hante.meuleman@broadcom.com> >> To: Chi-hsien Lin <chi-hsien.lin@infineon.com> >> To: Wright Feng <wright.feng@infineon.com> >> To: Chung-hsien Hsu <chung-hsien.hsu@infineon.com> >> Cc: SHA-cyfmac-dev-list@infineon.com >> Cc: brcm80211-dev-list.pdl@broadcom.com >> Cc: netdev@vger.kernel.org >> Cc: linux-wireless@vger.kernel.org >> Cc: Kalle Valo <kvalo@codeaurora.org> >> Cc: Jakub Kicinski <kuba@kernel.org> >> Cc: "David S. Miller" <davem@davemloft.net> >> Cc: linux-kernel@vger.kernel.org >> --- >> drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c >> b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c >> index 9ef94d7a7ca7..d824bea4b79d 100644 >> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c >> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c >> @@ -2209,7 +2209,7 @@ static int brcmf_pcie_pm_leave_D3(struct device *dev) >> >> err = brcmf_pcie_probe(pdev, NULL); >> if (err) >> - brcmf_err(bus, "probe after resume failed, err=%d\n", err); >> + __brcmf_err(NULL, __func__, "probe after resume failed, >> err=%d\n", > > > This is weird looking line now. Why can’t you simply use dev_err() / > netdev_err()? That's what brcmf_err normally expands to, but in this file the macro is overridden to add the extra first argument. The brcmf_ logging function write to brcmf trace buffers. This is not done with netdev_err/dev_err (and replacing the existing logging is out of scope for a regression fix anyway). Cheers, Ahmad > > >> >> return err; >> } >> -- >> 2.30.2 >> >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 11:11 ` Ahmad Fatoum @ 2021-08-17 11:54 ` Andy Shevchenko 2021-08-17 12:03 ` Ahmad Fatoum 0 siblings, 1 reply; 9+ messages in thread From: Andy Shevchenko @ 2021-08-17 11:54 UTC (permalink / raw) To: Ahmad Fatoum Cc: Arend van Spriel, Franky Lin, Hante Meuleman, Chi-hsien Lin, Wright Feng, Chung-hsien Hsu, kernel, SHA-cyfmac-dev-list, brcm80211-dev-list.pdl, netdev, linux-wireless, Kalle Valo, Jakub Kicinski, David S. Miller, linux-kernel On Tue, Aug 17, 2021 at 2:11 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > On 17.08.21 13:02, Andy Shevchenko wrote: > > On Tuesday, August 17, 2021, Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: ... > >> err = brcmf_pcie_probe(pdev, NULL); > >> if (err) > >> - brcmf_err(bus, "probe after resume failed, err=%d\n", err); > >> + __brcmf_err(NULL, __func__, "probe after resume failed, > >> err=%d\n", > > > > > > This is weird looking line now. Why can’t you simply use dev_err() / > > netdev_err()? > > That's what brcmf_err normally expands to, but in this file the macro > is overridden to add the extra first argument. So, then the problem is in macro here. You need another portion of macro(s) that will use the dev pointer directly. When you have a valid device, use it. And here it seems the case. > The brcmf_ logging function write to brcmf trace buffers. This is not > done with netdev_err/dev_err (and replacing the existing logging > is out of scope for a regression fix anyway). I see. -- With Best Regards, Andy Shevchenko ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 11:54 ` Andy Shevchenko @ 2021-08-17 12:03 ` Ahmad Fatoum 2021-08-17 12:07 ` Ahmad Fatoum 0 siblings, 1 reply; 9+ messages in thread From: Ahmad Fatoum @ 2021-08-17 12:03 UTC (permalink / raw) To: Andy Shevchenko Cc: Arend van Spriel, Franky Lin, Hante Meuleman, Chi-hsien Lin, Wright Feng, Chung-hsien Hsu, kernel, SHA-cyfmac-dev-list, brcm80211-dev-list.pdl, netdev, linux-wireless, Kalle Valo, Jakub Kicinski, David S. Miller, linux-kernel On 17.08.21 13:54, Andy Shevchenko wrote: > On Tue, Aug 17, 2021 at 2:11 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: >> On 17.08.21 13:02, Andy Shevchenko wrote: >>> On Tuesday, August 17, 2021, Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > > ... > >>>> err = brcmf_pcie_probe(pdev, NULL); >>>> if (err) >>>> - brcmf_err(bus, "probe after resume failed, err=%d\n", err); >>>> + __brcmf_err(NULL, __func__, "probe after resume failed, >>>> err=%d\n", >>> >>> >>> This is weird looking line now. Why can’t you simply use dev_err() / >>> netdev_err()? >> >> That's what brcmf_err normally expands to, but in this file the macro >> is overridden to add the extra first argument. > > So, then the problem is in macro here. You need another portion of > macro(s) that will use the dev pointer directly. When you have a valid > device, use it. And here it seems the case. Ah, you mean using pdev instead of the stale bus. Ye, I could do that. Thanks for pointing out. > >> The brcmf_ logging function write to brcmf trace buffers. This is not >> done with netdev_err/dev_err (and replacing the existing logging >> is out of scope for a regression fix anyway). > > I see. > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 12:03 ` Ahmad Fatoum @ 2021-08-17 12:07 ` Ahmad Fatoum 2021-08-17 13:06 ` Andy Shevchenko 0 siblings, 1 reply; 9+ messages in thread From: Ahmad Fatoum @ 2021-08-17 12:07 UTC (permalink / raw) To: Andy Shevchenko Cc: brcm80211-dev-list.pdl, Arend van Spriel, Chung-hsien Hsu, Chi-hsien Lin, netdev, linux-wireless, Hante Meuleman, linux-kernel, Wright Feng, SHA-cyfmac-dev-list, kernel, Jakub Kicinski, David S. Miller, Kalle Valo, Franky Lin On 17.08.21 14:03, Ahmad Fatoum wrote: > On 17.08.21 13:54, Andy Shevchenko wrote: >> On Tue, Aug 17, 2021 at 2:11 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: >>> On 17.08.21 13:02, Andy Shevchenko wrote: >>>> On Tuesday, August 17, 2021, Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: >> >> ... >> >>>>> err = brcmf_pcie_probe(pdev, NULL); >>>>> if (err) >>>>> - brcmf_err(bus, "probe after resume failed, err=%d\n", err); >>>>> + __brcmf_err(NULL, __func__, "probe after resume failed, >>>>> err=%d\n", >>>> >>>> >>>> This is weird looking line now. Why can’t you simply use dev_err() / >>>> netdev_err()? >>> >>> That's what brcmf_err normally expands to, but in this file the macro >>> is overridden to add the extra first argument. >> >> So, then the problem is in macro here. You need another portion of >> macro(s) that will use the dev pointer directly. When you have a valid >> device, use it. And here it seems the case. > > Ah, you mean using pdev instead of the stale bus. Ye, I could do that. > Thanks for pointing out. Ah, not so easy: __brcmf_err accepts a struct brcmf_bus * as first argument, but there is none I can pass along. As the whole file uses the brcm_ logging functions, I'd just leave this one without a device. > >> >>> The brcmf_ logging function write to brcmf trace buffers. This is not >>> done with netdev_err/dev_err (and replacing the existing logging >>> is out of scope for a regression fix anyway). >> >> I see. >> > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 12:07 ` Ahmad Fatoum @ 2021-08-17 13:06 ` Andy Shevchenko 2021-08-17 13:19 ` Ahmad Fatoum 0 siblings, 1 reply; 9+ messages in thread From: Andy Shevchenko @ 2021-08-17 13:06 UTC (permalink / raw) To: Ahmad Fatoum Cc: brcm80211-dev-list.pdl, Arend van Spriel, Chung-hsien Hsu, Chi-hsien Lin, netdev, linux-wireless, Hante Meuleman, linux-kernel, Wright Feng, SHA-cyfmac-dev-list, kernel, Jakub Kicinski, David S. Miller, Kalle Valo, Franky Lin On Tue, Aug 17, 2021 at 3:07 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > On 17.08.21 14:03, Ahmad Fatoum wrote: > > On 17.08.21 13:54, Andy Shevchenko wrote: > >> On Tue, Aug 17, 2021 at 2:11 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > >>> On 17.08.21 13:02, Andy Shevchenko wrote: > >>>> On Tuesday, August 17, 2021, Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: ... > >>>>> err = brcmf_pcie_probe(pdev, NULL); > >>>>> if (err) > >>>>> - brcmf_err(bus, "probe after resume failed, err=%d\n", err); > >>>>> + __brcmf_err(NULL, __func__, "probe after resume failed, > >>>>> err=%d\n", > >>>> > >>>> > >>>> This is weird looking line now. Why can’t you simply use dev_err() / > >>>> netdev_err()? > >>> > >>> That's what brcmf_err normally expands to, but in this file the macro > >>> is overridden to add the extra first argument. > >> > >> So, then the problem is in macro here. You need another portion of > >> macro(s) that will use the dev pointer directly. When you have a valid > >> device, use it. And here it seems the case. > > > > Ah, you mean using pdev instead of the stale bus. Ye, I could do that. > > Thanks for pointing out. > > Ah, not so easy: __brcmf_err accepts a struct brcmf_bus * as first argument, > but there is none I can pass along. As the whole file uses the brcm_ > logging functions, I'd just leave this one without a device. And what exactly prevents you to split that to something like __brcm_dev_err() // as current __brcm_err with dev argument { ... } __brsm_err(bus, ...) __brcm_dev_err(bus->dev, ...) ? -- With Best Regards, Andy Shevchenko ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 13:06 ` Andy Shevchenko @ 2021-08-17 13:19 ` Ahmad Fatoum 0 siblings, 0 replies; 9+ messages in thread From: Ahmad Fatoum @ 2021-08-17 13:19 UTC (permalink / raw) To: Andy Shevchenko Cc: brcm80211-dev-list.pdl, Arend van Spriel, Chung-hsien Hsu, Chi-hsien Lin, netdev, linux-wireless, Hante Meuleman, linux-kernel, Wright Feng, SHA-cyfmac-dev-list, kernel, Jakub Kicinski, David S. Miller, Kalle Valo, Franky Lin On 17.08.21 15:06, Andy Shevchenko wrote: > On Tue, Aug 17, 2021 at 3:07 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: >> On 17.08.21 14:03, Ahmad Fatoum wrote: >>> On 17.08.21 13:54, Andy Shevchenko wrote: >>>> On Tue, Aug 17, 2021 at 2:11 PM Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: >>>>> On 17.08.21 13:02, Andy Shevchenko wrote: >>>>>> On Tuesday, August 17, 2021, Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > > ... > >>>>>>> err = brcmf_pcie_probe(pdev, NULL); >>>>>>> if (err) >>>>>>> - brcmf_err(bus, "probe after resume failed, err=%d\n", err); >>>>>>> + __brcmf_err(NULL, __func__, "probe after resume failed, >>>>>>> err=%d\n", >>>>>> >>>>>> >>>>>> This is weird looking line now. Why can’t you simply use dev_err() / >>>>>> netdev_err()? >>>>> >>>>> That's what brcmf_err normally expands to, but in this file the macro >>>>> is overridden to add the extra first argument. >>>> >>>> So, then the problem is in macro here. You need another portion of >>>> macro(s) that will use the dev pointer directly. When you have a valid >>>> device, use it. And here it seems the case. >>> >>> Ah, you mean using pdev instead of the stale bus. Ye, I could do that. >>> Thanks for pointing out. >> >> Ah, not so easy: __brcmf_err accepts a struct brcmf_bus * as first argument, >> but there is none I can pass along. As the whole file uses the brcm_ >> logging functions, I'd just leave this one without a device. > > And what exactly prevents you to split that to something like > > __brcm_dev_err() // as current __brcm_err with dev argument > { > ... > } > > __brsm_err(bus, ...) __brcm_dev_err(bus->dev, ...) > > ? I like my regression fixes to be short and to the point. Cheers, Ahmad -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: brcmfmac: pcie: fix oops on failure to resume and reprobe 2021-08-17 6:35 [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe Ahmad Fatoum 2021-08-17 10:01 ` Ahmad Fatoum [not found] ` <CAHp75Vfc_T04p95PgVUd+CK+ttPwX2aOC4WPD35Z01WQV1MxKw@mail.gmail.com> @ 2021-08-29 11:45 ` Kalle Valo 2 siblings, 0 replies; 9+ messages in thread From: Kalle Valo @ 2021-08-29 11:45 UTC (permalink / raw) To: Ahmad Fatoum Cc: Arend van Spriel, Franky Lin, Hante Meuleman, Chi-hsien Lin, Wright Feng, Chung-hsien Hsu, kernel, Ahmad Fatoum, SHA-cyfmac-dev-list, brcm80211-dev-list.pdl, netdev, linux-wireless, Jakub Kicinski, David S. Miller, linux-kernel Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: > When resuming from suspend, brcmf_pcie_pm_leave_D3 will first attempt a > hot resume and then fall back to removing the PCI device and then > reprobing. If this probe fails, the kernel will oops, because brcmf_err, > which is called to report the failure will dereference the stale bus > pointer. Open code and use the default bus-less brcmf_err to avoid this. > > Fixes: 8602e62441ab ("brcmfmac: pass bus to the __brcmf_err() in pcie.c") > Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de> Patch applied to wireless-drivers-next.git, thanks. d745ca4f2c4a brcmfmac: pcie: fix oops on failure to resume and reprobe -- https://patchwork.kernel.org/project/linux-wireless/patch/20210817063521.22450-1-a.fatoum@pengutronix.de/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-08-29 11:46 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-17 6:35 [PATCH] brcmfmac: pcie: fix oops on failure to resume and reprobe Ahmad Fatoum 2021-08-17 10:01 ` Ahmad Fatoum [not found] ` <CAHp75Vfc_T04p95PgVUd+CK+ttPwX2aOC4WPD35Z01WQV1MxKw@mail.gmail.com> 2021-08-17 11:11 ` Ahmad Fatoum 2021-08-17 11:54 ` Andy Shevchenko 2021-08-17 12:03 ` Ahmad Fatoum 2021-08-17 12:07 ` Ahmad Fatoum 2021-08-17 13:06 ` Andy Shevchenko 2021-08-17 13:19 ` Ahmad Fatoum 2021-08-29 11:45 ` Kalle Valo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).