[PATCH v3 2/2] mmc: sdhci: add regulator disable/put when control error

[PATCH v3 2/2] mmc: sdhci: add regulator disable/put when control error

[Jaehoon Chung]

Missing regulator disable/put. Fixed them.

Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
---
 drivers/mmc/host/sdhci.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
index 1768ffb..35edcc1 100644
--- a/drivers/mmc/host/sdhci.c
+++ b/drivers/mmc/host/sdhci.c
@@ -2048,6 +2048,10 @@ reset:
 	sdhci_reset(host, SDHCI_RESET_ALL);
 	free_irq(host->irq, host);
 	del_timer_sync(&host->timer);
+	if (host->vmmc) {
+		regulator_disable(host->vmmc);
+		regulator_put(host->vmmc);
+	}
 
 untasklet:
 	tasklet_kill(&host->card_tasklet);

Re: [PATCH v3 2/2] mmc: sdhci: add regulator disable/put when control error

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 447 bytes --]

On Tue, Apr 05, 2011 at 05:02:20PM +0900, Jaehoon Chung wrote:
> Missing regulator disable/put. Fixed them.
> 
> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>

Looks OK but could be merged into the previous patch.

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [PATCH v3 2/2] mmc: sdhci: add regulator disable/put when control error

[Jae hoon Chung]

Hi..

2011/4/9 Wolfram Sang <w.sang@pengutronix.de>:
> On Tue, Apr 05, 2011 at 05:02:20PM +0900, Jaehoon Chung wrote:
>> Missing regulator disable/put. Fixed them.
>>
>> Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
>> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
>
> Looks OK but could be merged into the previous patch.

Thanks. :)
yes, i can be merged into the previous patch.
But i think good that  separeted. anyway..don't mind merged or not?

Regards,
Jaehoon Chung

>
> --
> Pengutronix e.K.                           | Wolfram Sang                |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
>
> iEYEARECAAYFAk2fViEACgkQD27XaX1/VRt1+wCaAxSjsZ+GqVdhnca16rMsO7LK
> jfcAnjvIvSBSFknx7nMomkT6I4Y1YTgV
> =KfuE
> -----END PGP SIGNATURE-----
>
>

Re: [PATCH v3 2/2] mmc: sdhci: add regulator disable/put when control error

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 454 bytes --]

> > Looks OK but could be merged into the previous patch.
> 
> Thanks. :)
> yes, i can be merged into the previous patch.
> But i think good that  separeted. anyway..don't mind merged or not?

I prefer it merged, because it then fixes all forgotten cleanups in the
error-path in one go.

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]