[lkml]   [2007]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: [Sdhci-devel] [PATCH 2.6.20-rc2] Add a quirk to allow ENE PCI SD card readers to work again
    Hi Darren,

    It has come to my attention that the current routine for setting power
    is not compliant with the specification. As such, I'd like you to try
    the following and see if removes the need for your patch:

    diff --git a/drivers/mmc/sdhci.c b/drivers/mmc/sdhci.c
    index dc1b04a..13ac2fe 100644
    --- a/drivers/mmc/sdhci.c
    +++ b/drivers/mmc/sdhci.c
    @@ -685,23 +690,21 @@ static void sdhci_set_power(struct sdhci_host
    *host, unsigned short power)
    if (power == (unsigned short)-1)
    goto out;

    - pwr = SDHCI_POWER_ON;
    switch (power) {
    case MMC_VDD_170:
    case MMC_VDD_180:
    case MMC_VDD_190:
    - pwr |= SDHCI_POWER_180;
    + pwr = SDHCI_POWER_180;
    case MMC_VDD_290:
    case MMC_VDD_300:
    case MMC_VDD_310:
    - pwr |= SDHCI_POWER_300;
    + pwr = SDHCI_POWER_300;
    case MMC_VDD_320:
    case MMC_VDD_330:
    case MMC_VDD_340:
    - pwr |= SDHCI_POWER_330;
    + pwr = SDHCI_POWER_330;
    @@ -709,6 +712,10 @@ static void sdhci_set_power(struct sdhci_host
    *host, unsigned short power)

    writeb(pwr, host->ioaddr + SDHCI_POWER_CONTROL);

    + pwr |= SDHCI_POWER_ON;
    + writeb(pwr, host->ioaddr + SDHCI_POWER_CONTROL);
    host->power = power;

    I'd appreciate if you could test this sooner rather than later as the
    merge window is just around the corner.

    -- Pierre Ossman

    Linux kernel, MMC maintainer
    PulseAudio, core developer
    rdesktop, core developer
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-02-02 08:57    [W:0.542 / U:16.424 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site