On 2 January 2017 at 13:05, Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > cd_con_id is not used and always NULL. > Remove it to make code a bit more cleaner. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/sdhci-pci-core.c | 3 +-- > drivers/mmc/host/sdhci-pci.h | 1 - > 2 files changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-pci-core.c b/drivers/mmc/host/sdhci-pci-core.c > index 1a72d32af07f..5c275b7d0d53 100644 > --- a/drivers/mmc/host/sdhci-pci-core.c > +++ b/drivers/mmc/host/sdhci-pci-core.c > @@ -424,7 +424,6 @@ static int byt_sdio_probe_slot(struct sdhci_pci_slot *slot) > static int byt_sd_probe_slot(struct sdhci_pci_slot *slot) > { > slot->host->mmc->caps |= MMC_CAP_WAIT_WHILE_BUSY; > - slot->cd_con_id = NULL; > slot->cd_idx = 0; > slot->cd_override_level = true; > if (slot->chip->pdev->device == PCI_DEVICE_ID_INTEL_BXT_SD || > @@ -1817,7 +1816,7 @@ static struct sdhci_pci_slot *sdhci_pci_probe_slot( > host->mmc->caps2 |= MMC_CAP2_NO_PRESCAN_POWERUP; > > if (slot->cd_idx >= 0) { > - ret = mmc_gpiod_request_cd(host->mmc, slot->cd_con_id, slot->cd_idx, > + ret = mmc_gpiod_request_cd(host->mmc, NULL, slot->cd_idx, > slot->cd_override_level, 0, NULL); > if (ret == -EPROBE_DEFER) > goto remove; > diff --git a/drivers/mmc/host/sdhci-pci.h b/drivers/mmc/host/sdhci-pci.h > index 4abdaed72bd4..36f743464fcc 100644 > --- a/drivers/mmc/host/sdhci-pci.h > +++ b/drivers/mmc/host/sdhci-pci.h > @@ -81,7 +81,6 @@ struct sdhci_pci_slot { > int cd_gpio; > int cd_irq; > > - char *cd_con_id; > int cd_idx; > bool cd_override_level; > > -- > 2.11.0 > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html