Commit dd369800 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Chris Ball

mmc: dw_mmc: eliminate useless usage of ret

In few places usage of ret variable is not needed.
This patch simplifies those pieces of code.
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: default avatarSeungwon Jeon <tgih.jun@samsung.com>
Signed-off-by: default avatarChris Ball <cjb@laptop.org>
parent bcc87666
...@@ -100,22 +100,18 @@ static void dw_mci_pci_remove(struct pci_dev *pdev) ...@@ -100,22 +100,18 @@ static void dw_mci_pci_remove(struct pci_dev *pdev)
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
static int dw_mci_pci_suspend(struct device *dev) static int dw_mci_pci_suspend(struct device *dev)
{ {
int ret;
struct pci_dev *pdev = to_pci_dev(dev); struct pci_dev *pdev = to_pci_dev(dev);
struct dw_mci *host = pci_get_drvdata(pdev); struct dw_mci *host = pci_get_drvdata(pdev);
ret = dw_mci_suspend(host); return dw_mci_suspend(host);
return ret;
} }
static int dw_mci_pci_resume(struct device *dev) static int dw_mci_pci_resume(struct device *dev)
{ {
int ret;
struct pci_dev *pdev = to_pci_dev(dev); struct pci_dev *pdev = to_pci_dev(dev);
struct dw_mci *host = pci_get_drvdata(pdev); struct dw_mci *host = pci_get_drvdata(pdev);
ret = dw_mci_resume(host); return dw_mci_resume(host);
return ret;
} }
#else #else
#define dw_mci_pci_suspend NULL #define dw_mci_pci_suspend NULL
......
...@@ -56,8 +56,7 @@ int dw_mci_pltfm_register(struct platform_device *pdev, ...@@ -56,8 +56,7 @@ int dw_mci_pltfm_register(struct platform_device *pdev,
} }
platform_set_drvdata(pdev, host); platform_set_drvdata(pdev, host);
ret = dw_mci_probe(host); return dw_mci_probe(host);
return ret;
} }
EXPORT_SYMBOL_GPL(dw_mci_pltfm_register); EXPORT_SYMBOL_GPL(dw_mci_pltfm_register);
...@@ -81,26 +80,16 @@ EXPORT_SYMBOL_GPL(dw_mci_pltfm_remove); ...@@ -81,26 +80,16 @@ EXPORT_SYMBOL_GPL(dw_mci_pltfm_remove);
*/ */
static int dw_mci_pltfm_suspend(struct device *dev) static int dw_mci_pltfm_suspend(struct device *dev)
{ {
int ret;
struct dw_mci *host = dev_get_drvdata(dev); struct dw_mci *host = dev_get_drvdata(dev);
ret = dw_mci_suspend(host); return dw_mci_suspend(host);
if (ret)
return ret;
return 0;
} }
static int dw_mci_pltfm_resume(struct device *dev) static int dw_mci_pltfm_resume(struct device *dev)
{ {
int ret;
struct dw_mci *host = dev_get_drvdata(dev); struct dw_mci *host = dev_get_drvdata(dev);
ret = dw_mci_resume(host); return dw_mci_resume(host);
if (ret)
return ret;
return 0;
} }
#else #else
#define dw_mci_pltfm_suspend NULL #define dw_mci_pltfm_suspend NULL
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment