mirror of
https://github.com/torvalds/linux.git
synced 2024-11-20 02:51:44 +00:00
mmc: sdhci-pltfm: move devm_ioremap_resource() up
Call devm_ioremap_resource() right after platform_get_resource(). This saves the error check of platform_get_resource() because devm_ioremap_resource() checks if the given resource is NULL. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> Acked-by: Adrian Hunter <adrian.hunter@intel.com> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
This commit is contained in:
parent
e30568d43f
commit
378382b8d0
@ -119,11 +119,13 @@ struct sdhci_host *sdhci_pltfm_init(struct platform_device *pdev,
|
||||
{
|
||||
struct sdhci_host *host;
|
||||
struct resource *iomem;
|
||||
void __iomem *ioaddr;
|
||||
int ret;
|
||||
|
||||
iomem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
|
||||
if (!iomem) {
|
||||
ret = -ENOMEM;
|
||||
ioaddr = devm_ioremap_resource(&pdev->dev, iomem);
|
||||
if (IS_ERR(ioaddr)) {
|
||||
ret = PTR_ERR(ioaddr);
|
||||
goto err;
|
||||
}
|
||||
|
||||
@ -135,6 +137,7 @@ struct sdhci_host *sdhci_pltfm_init(struct platform_device *pdev,
|
||||
goto err;
|
||||
}
|
||||
|
||||
host->ioaddr = ioaddr;
|
||||
host->hw_name = dev_name(&pdev->dev);
|
||||
if (pdata && pdata->ops)
|
||||
host->ops = pdata->ops;
|
||||
@ -152,12 +155,6 @@ struct sdhci_host *sdhci_pltfm_init(struct platform_device *pdev,
|
||||
goto err_request;
|
||||
}
|
||||
|
||||
host->ioaddr = devm_ioremap_resource(&pdev->dev, iomem);
|
||||
if (IS_ERR(host->ioaddr)) {
|
||||
ret = PTR_ERR(host->ioaddr);
|
||||
goto err_request;
|
||||
}
|
||||
|
||||
/*
|
||||
* Some platforms need to probe the controller to be able to
|
||||
* determine which caps should be used.
|
||||
|
Loading…
Reference in New Issue
Block a user