ath10k: get rid of pci_assign_resource() call from pci_probe
authorBartosz Markowski <bartosz.markowski@tieto.com>
Wed, 14 May 2014 13:56:17 +0000 (16:56 +0300)
committerKalle Valo <kvalo@qca.qualcomm.com>
Fri, 16 May 2014 13:50:26 +0000 (16:50 +0300)
On ARM-based (MSM mach), the pci_assign_resource() is passing
some invalid pointers and leading to L2 cache errors,
what prevents the PCI communication completly.

So far I have not found this funtion to be directly called by
any other wifi driver and did not found this assigning needed
on any other platform. So removing it completely.

Signed-off-by: Bartosz Markowski <bartosz.markowski@tieto.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath10k/pci.c

index 33e87557a7dd7ed010578efed440d03d20ca84c2..7d72b9cfe0bed2b3fbdc2609343d241c1b5ba219 100644 (file)
@@ -2647,18 +2647,6 @@ static int ath10k_pci_probe(struct pci_dev *pdev,
 
        pci_set_drvdata(pdev, ar);
 
-       /*
-        * Without any knowledge of the Host, the Target may have been reset or
-        * power cycled and its Config Space may no longer reflect the PCI
-        * address space that was assigned earlier by the PCI infrastructure.
-        * Refresh it now.
-        */
-       ret = pci_assign_resource(pdev, BAR_NUM);
-       if (ret) {
-               ath10k_err("failed to assign PCI space: %d\n", ret);
-               goto err_ar;
-       }
-
        ret = pci_enable_device(pdev);
        if (ret) {
                ath10k_err("failed to enable PCI device: %d\n", ret);