diff options
| author | Pierre Ossman <drzeus@drzeus.cx> | 2005-11-05 11:16:50 +0100 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2005-11-05 11:16:50 +0100 |
| commit | 25cc5e5bd20e9274dddb9d1e55b83798363528dc (patch) | |
| tree | fd065b54bbce7e04387670e163d2aba1ddf43ace /drivers/mmc/wbsd.c | |
| parent | [MMC] Response to write commands is R1 nor R1b (diff) | |
| download | linux-25cc5e5bd20e9274dddb9d1e55b83798363528dc.tar.xz linux-25cc5e5bd20e9274dddb9d1e55b83798363528dc.zip | |
[MMC] Fix chip config in wbsd
There is a broken if clause in the wbsd driver that can cause the
driver to try and configure the chip even though none is found. This
results in i/o on invalid ports.
Signed-off-by: Pierre Ossman <drzeus@drzeus.cx>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'drivers/mmc/wbsd.c')
| -rw-r--r-- | drivers/mmc/wbsd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/mmc/wbsd.c b/drivers/mmc/wbsd.c index 942668e93a74..ea0c4b3361bf 100644 --- a/drivers/mmc/wbsd.c +++ b/drivers/mmc/wbsd.c @@ -1852,9 +1852,9 @@ static int __devinit wbsd_init(struct device* dev, int base, int irq, int dma, /* * See if chip needs to be configured. */ - if (pnp && (host->config != 0)) + if (pnp) { - if (!wbsd_chip_validate(host)) + if ((host->config != 0) && !wbsd_chip_validate(host)) { printk(KERN_WARNING DRIVER_NAME ": PnP active but chip not configured! " |