| Message ID | 20221112090437.3591380-1-yangyingliang@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | ba174481be4284d531487197ab32a4bcd17e30bc |
| Headers | show |
| Series | [-next] Bluetooth: hci_bcm4377: Fix missing pci_disable_device() on error in bcm4377_probe() | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | fail | error: drivers/bluetooth/hci_bcm4377.c: does not exist in index hint: Use 'git am --show-current-patch' to see the failed patch |
This is an automated email and please do not reply to this email. Dear Submitter, Thank you for submitting the patches to the linux bluetooth mailing list. While preparing the CI tests, the patches you submitted couldn't be applied to the current HEAD of the repository. ----- Output ----- error: drivers/bluetooth/hci_bcm4377.c: does not exist in index hint: Use 'git am --show-current-patch' to see the failed patch Please resolve the issue and submit the patches again. --- Regards, Linux Bluetooth
Hi, On Sat, Nov 12, 2022, at 10:04, Yang Yingliang wrote: > pci_disable_device() need be called while module exiting, switch to use > pcim_enable(), pci_disable_device() will be called in pcim_release() > after probe() failure. > > Fixes: ab80b2cec05f ("Bluetooth: hci_bcm4377: Add new driver for > BCM4377 PCIe boards") > Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> > --- Good catch, thanks! Reviewed-by: Sven Peter <sven@svenpeter.dev> Best, Sven
Hello: This patch was applied to bluetooth/bluetooth-next.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Sat, 12 Nov 2022 17:04:37 +0800 you wrote: > pci_disable_device() need be called while module exiting, switch to use > pcim_enable(), pci_disable_device() will be called in pcim_release() > after probe() failure. > > Fixes: ab80b2cec05f ("Bluetooth: hci_bcm4377: Add new driver for BCM4377 PCIe boards") > Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> > > [...] Here is the summary with links: - [-next] Bluetooth: hci_bcm4377: Fix missing pci_disable_device() on error in bcm4377_probe() https://git.kernel.org/bluetooth/bluetooth-next/c/ba174481be42 You are awesome, thank you!
diff --git a/drivers/bluetooth/hci_bcm4377.c b/drivers/bluetooth/hci_bcm4377.c index 74f44562ac33..19ad0e788646 100644 --- a/drivers/bluetooth/hci_bcm4377.c +++ b/drivers/bluetooth/hci_bcm4377.c @@ -2306,7 +2306,7 @@ static int bcm4377_probe(struct pci_dev *pdev, const struct pci_device_id *id) */ msleep(100); - ret = pci_enable_device(pdev); + ret = pcim_enable_device(pdev); if (ret) return ret; pci_set_master(pdev);
pci_disable_device() need be called while module exiting, switch to use pcim_enable(), pci_disable_device() will be called in pcim_release() after probe() failure. Fixes: ab80b2cec05f ("Bluetooth: hci_bcm4377: Add new driver for BCM4377 PCIe boards") Signed-off-by: Yang Yingliang <yangyingliang@huawei.com> --- drivers/bluetooth/hci_bcm4377.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)