From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= Subject: [PATCH V2 5/5] brcmsmac: remove PCI suspend/resume from bcma driver Date: Fri, 13 Jan 2012 23:58:42 +0100 Message-ID: <1326495522-6024-5-git-send-email-zajec5@gmail.com> References: <1326495522-6024-1-git-send-email-zajec5@gmail.com> Cc: Network Development , Linus Torvalds , Arend van Spriel , Larry Finger , Alwin Beukers , Roland Vossen , "Franky (Zhenhui) Lin" To: "John W. Linville" Return-path: Received: from mail-ey0-f174.google.com ([209.85.215.174]:37946 "EHLO mail-ey0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759413Ab2AMW7D (ORCPT ); Fri, 13 Jan 2012 17:59:03 -0500 Received: by mail-ey0-f174.google.com with SMTP id l12so346245eaa.19 for ; Fri, 13 Jan 2012 14:59:03 -0800 (PST) In-Reply-To: <1326495522-6024-1-git-send-email-zajec5@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Linus Torvalds The brcmsmac driver isn't a PCI driver any more, it's a bcma one. The PCI device has been resumed by the PCI driver (the generic PCI layer, really), we should be resuming just our own driver state. Also add pr_debug() calls to show that we now actually get the suspend/resume events. Signed-off-by: Linus Torvalds --- .../net/wireless/brcm80211/brcmsmac/mac80211_if.c | 38 ++------------------ 1 files changed, 3 insertions(+), 35 deletions(-) diff --git a/drivers/net/wireless/brcm80211/brcmsmac/mac80211_if.c b/drivers/net/wireless/brcm80211/brcmsmac/mac80211_if.c index 19721db..c43b2f8 100644 --- a/drivers/net/wireless/brcm80211/brcmsmac/mac80211_if.c +++ b/drivers/net/wireless/brcm80211/brcmsmac/mac80211_if.c @@ -1128,13 +1128,6 @@ static int __devinit brcms_bcma_probe(struct bcma_device *pdev) return 0; } -static int brcms_pci_suspend(struct pci_dev *pdev) -{ - pci_save_state(pdev); - pci_disable_device(pdev); - return pci_set_power_state(pdev, PCI_D3hot); -} - static int brcms_suspend(struct bcma_device *pdev) { struct brcms_info *wl; @@ -1153,40 +1146,15 @@ static int brcms_suspend(struct bcma_device *pdev) wl->pub->hw_up = false; spin_unlock_bh(&wl->lock); - /* temporarily do suspend ourselves */ - return brcms_pci_suspend(pdev->bus->host_pci); -} - -static int brcms_pci_resume(struct pci_dev *pdev) -{ - int err = 0; - uint val; - - err = pci_set_power_state(pdev, PCI_D0); - if (err) - return err; - - pci_restore_state(pdev); - - err = pci_enable_device(pdev); - if (err) - return err; - - pci_set_master(pdev); - - pci_read_config_dword(pdev, 0x40, &val); - if ((val & 0x0000ff00) != 0) - pci_write_config_dword(pdev, 0x40, val & 0xffff00ff); + pr_debug("brcms_suspend ok\n"); return 0; } static int brcms_resume(struct bcma_device *pdev) { - /* - * just do pci resume for now until bcma supports it. - */ - return brcms_pci_resume(pdev->bus->host_pci); + pr_debug("brcms_resume ok\n"); + return 0; } static struct bcma_driver brcms_bcma_driver = { -- 1.7.7