Re: [PATCH -next] mmc: mxs-mmc: fix error return code in mxs_mmc_probe()

2013-05-26 Thread Chris Ball
Hi,

On Fri, Apr 19 2013, Marek Vasut wrote:
 Dear Wei Yongjun,

 From: Wei Yongjun yongjun_...@trendmicro.com.cn
 
 Fix to return -ENODEV in the request dma error case instead
 of 0, as done elsewhere in this function.
 
 Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

 Seems ok to me.

 Reviewed-by: Marek Vasut ma...@denx.de

Thanks, pushed to mmc-next for 3.11.

- Chris.
-- 
Chris Ball   c...@laptop.org   http://printf.net/
One Laptop Per Child
--
To unsubscribe from this list: send the line unsubscribe linux-mmc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH -next] mmc: mxs-mmc: fix error return code in mxs_mmc_probe()

2013-04-19 Thread Marek Vasut
Dear Wei Yongjun,

 From: Wei Yongjun yongjun_...@trendmicro.com.cn
 
 Fix to return -ENODEV in the request dma error case instead
 of 0, as done elsewhere in this function.
 
 Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

Seems ok to me.

Reviewed-by: Marek Vasut ma...@denx.de

Best regards,
Marek Vasut
--
To unsubscribe from this list: send the line unsubscribe linux-mmc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH -next] mmc: mxs-mmc: fix error return code in mxs_mmc_probe()

2013-04-18 Thread Wei Yongjun
From: Wei Yongjun yongjun_...@trendmicro.com.cn

Fix to return -ENODEV in the request dma error case instead
of 0, as done elsewhere in this function.

Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
 drivers/mmc/host/mxs-mmc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/mmc/host/mxs-mmc.c b/drivers/mmc/host/mxs-mmc.c
index 4278a17..a09ba6e 100644
--- a/drivers/mmc/host/mxs-mmc.c
+++ b/drivers/mmc/host/mxs-mmc.c
@@ -639,6 +639,7 @@ static int mxs_mmc_probe(struct platform_device *pdev)
if (!ssp-dmach) {
dev_err(mmc_dev(host-mmc),
%s: failed to request dma\n, __func__);
+   ret = -ENODEV;
goto out_clk_put;
}
 

--
To unsubscribe from this list: send the line unsubscribe linux-mmc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html