2008-12-13 20:16:16

by Roel Kluin

[permalink] [raw]

Subject: [PATCH] mmc: balanc pci_iomap with pci_iounmap

---

balance pci_iomap with pci_iounmap, not iounmap

Signed-off-by: Roel Kluin <[email protected]>
---
diff --git a/drivers/mmc/host/sdricoh_cs.c b/drivers/mmc/host/sdricoh_cs.c
index 1df44d9..d4d86be 100644
--- a/drivers/mmc/host/sdricoh_cs.c
+++ b/drivers/mmc/host/sdricoh_cs.c
@@ -463,7 +463,7 @@ static int sdricoh_init_mmc(struct pci_dev *pci_dev,

err:
if (iobase)
- iounmap(iobase);
+ pci_iounmap(iobase);
if (mmc)
mmc_free_host(mmc);

2008-12-13 20:21:43

by Roel Kluin

[permalink] [raw]

Subject: [PATCH V2] mmc: balanc pci_iomap with pci_iounmap

---

balance pci_iomap with pci_iounmap, not iounmap

Signed-off-by: Roel Kluin <[email protected]>
---
Sorry, this is what it should be...

diff --git a/drivers/mmc/host/sdricoh_cs.c b/drivers/mmc/host/sdricoh_cs.c
index 1df44d9..d4748a8 100644
--- a/drivers/mmc/host/sdricoh_cs.c
+++ b/drivers/mmc/host/sdricoh_cs.c
@@ -463,7 +463,7 @@ static int sdricoh_init_mmc(struct pci_dev *pci_dev,

err:
if (iobase)
- iounmap(iobase);
+ pci_iounmap(pci_dev, iobase);
if (mmc)
mmc_free_host(mmc);

2008-12-21 15:00:50

by Pierre Ossman

[permalink] [raw]

Subject: Re: [PATCH V2] mmc: balanc pci_iomap with pci_iounmap

---

On Sat, 13 Dec 2008 21:21:33 +0100
Roel Kluin <[email protected]> wrote:

> balance pci_iomap with pci_iounmap, not iounmap
>
> Signed-off-by: Roel Kluin <[email protected]>
> ---
> Sorry, this is what it should be...
>

Applied, thanks.

--
-- Pierre Ossman

WARNING: This correspondence is being monitored by the
Swedish government. Make sure your server uses encryption
for SMTP traffic and consider using PGP for end-to-end
encryption.

---

Attachments:

signature.asc (197.00 B)