Commit c43fd774 authored by Adrian Hunter's avatar Adrian Hunter Committed by Chris Ball

mmc: sdhci-pci: add flag for devices that can support runtime PM

Only allow runtime PM for devices that specifically indicate
that they support it.
Signed-off-by: default avatarAdrian Hunter <adrian.hunter@intel.com>
Signed-off-by: default avatarChris Ball <cjb@laptop.org>
parent e840ce13
...@@ -47,6 +47,7 @@ struct sdhci_pci_slot; ...@@ -47,6 +47,7 @@ struct sdhci_pci_slot;
struct sdhci_pci_fixes { struct sdhci_pci_fixes {
unsigned int quirks; unsigned int quirks;
bool allow_runtime_pm;
int (*probe) (struct sdhci_pci_chip *); int (*probe) (struct sdhci_pci_chip *);
...@@ -72,6 +73,7 @@ struct sdhci_pci_chip { ...@@ -72,6 +73,7 @@ struct sdhci_pci_chip {
struct pci_dev *pdev; struct pci_dev *pdev;
unsigned int quirks; unsigned int quirks;
bool allow_runtime_pm;
const struct sdhci_pci_fixes *fixes; const struct sdhci_pci_fixes *fixes;
int num_slots; /* Slots on controller */ int num_slots; /* Slots on controller */
...@@ -305,16 +307,19 @@ static const struct sdhci_pci_fixes sdhci_intel_mrst_hc1_hc2 = { ...@@ -305,16 +307,19 @@ static const struct sdhci_pci_fixes sdhci_intel_mrst_hc1_hc2 = {
static const struct sdhci_pci_fixes sdhci_intel_mfd_sd = { static const struct sdhci_pci_fixes sdhci_intel_mfd_sd = {
.quirks = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC, .quirks = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
.allow_runtime_pm = true,
.probe_slot = mfd_sd_probe_slot, .probe_slot = mfd_sd_probe_slot,
.remove_slot = mfd_sd_remove_slot, .remove_slot = mfd_sd_remove_slot,
}; };
static const struct sdhci_pci_fixes sdhci_intel_mfd_sdio = { static const struct sdhci_pci_fixes sdhci_intel_mfd_sdio = {
.quirks = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC, .quirks = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
.allow_runtime_pm = true,
}; };
static const struct sdhci_pci_fixes sdhci_intel_mfd_emmc = { static const struct sdhci_pci_fixes sdhci_intel_mfd_emmc = {
.quirks = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC, .quirks = SDHCI_QUIRK_NO_ENDATTR_IN_NOPDESC,
.allow_runtime_pm = true,
.probe_slot = mfd_emmc_probe_slot, .probe_slot = mfd_emmc_probe_slot,
.remove_slot = mfd_emmc_remove_slot, .remove_slot = mfd_emmc_remove_slot,
}; };
...@@ -1355,8 +1360,10 @@ static int __devinit sdhci_pci_probe(struct pci_dev *pdev, ...@@ -1355,8 +1360,10 @@ static int __devinit sdhci_pci_probe(struct pci_dev *pdev,
chip->pdev = pdev; chip->pdev = pdev;
chip->fixes = (const struct sdhci_pci_fixes *)ent->driver_data; chip->fixes = (const struct sdhci_pci_fixes *)ent->driver_data;
if (chip->fixes) if (chip->fixes) {
chip->quirks = chip->fixes->quirks; chip->quirks = chip->fixes->quirks;
chip->allow_runtime_pm = chip->fixes->allow_runtime_pm;
}
chip->num_slots = slots; chip->num_slots = slots;
pci_set_drvdata(pdev, chip); pci_set_drvdata(pdev, chip);
...@@ -1381,7 +1388,8 @@ static int __devinit sdhci_pci_probe(struct pci_dev *pdev, ...@@ -1381,7 +1388,8 @@ static int __devinit sdhci_pci_probe(struct pci_dev *pdev,
chip->slots[i] = slot; chip->slots[i] = slot;
} }
sdhci_pci_runtime_pm_allow(&pdev->dev); if (chip->allow_runtime_pm)
sdhci_pci_runtime_pm_allow(&pdev->dev);
return 0; return 0;
...@@ -1399,11 +1407,12 @@ static void __devexit sdhci_pci_remove(struct pci_dev *pdev) ...@@ -1399,11 +1407,12 @@ static void __devexit sdhci_pci_remove(struct pci_dev *pdev)
int i; int i;
struct sdhci_pci_chip *chip; struct sdhci_pci_chip *chip;
sdhci_pci_runtime_pm_forbid(&pdev->dev);
chip = pci_get_drvdata(pdev); chip = pci_get_drvdata(pdev);
if (chip) { if (chip) {
if (chip->allow_runtime_pm)
sdhci_pci_runtime_pm_forbid(&pdev->dev);
for (i = 0; i < chip->num_slots; i++) for (i = 0; i < chip->num_slots; i++)
sdhci_pci_remove_slot(chip->slots[i]); sdhci_pci_remove_slot(chip->slots[i]);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment