platform_get_resource_byname() and devm_ioremap_resource() can be replaced by devm_platform_ioremap_resource_byname(), which can simplify the code logic a bit, No functional change here. Signed-off-by: Zhang Zekun <zhangzekun11@xxxxxxxxxx> --- drivers/bus/qcom-ssc-block-bus.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/bus/qcom-ssc-block-bus.c b/drivers/bus/qcom-ssc-block-bus.c index 5931974a21fa..11fc77c31ce8 100644 --- a/drivers/bus/qcom-ssc-block-bus.c +++ b/drivers/bus/qcom-ssc-block-bus.c @@ -255,7 +255,6 @@ static int qcom_ssc_block_bus_probe(struct platform_device *pdev) struct qcom_ssc_block_bus_data *data; struct device_node *np = pdev->dev.of_node; struct of_phandle_args halt_args; - struct resource *res; int ret; data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL); @@ -277,14 +276,14 @@ static int qcom_ssc_block_bus_probe(struct platform_device *pdev) return dev_err_probe(&pdev->dev, ret, "error when enabling power domains\n"); /* low level overrides for when the HW logic doesn't "just work" */ - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "mpm_sscaon_config0"); - data->reg_mpm_sscaon_config0 = devm_ioremap_resource(&pdev->dev, res); + data->reg_mpm_sscaon_config0 = devm_platform_ioremap_resource_byname(pdev, + "mpm_sscaon_config0"); if (IS_ERR(data->reg_mpm_sscaon_config0)) return dev_err_probe(&pdev->dev, PTR_ERR(data->reg_mpm_sscaon_config0), "Failed to ioremap mpm_sscaon_config0\n"); - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "mpm_sscaon_config1"); - data->reg_mpm_sscaon_config1 = devm_ioremap_resource(&pdev->dev, res); + data->reg_mpm_sscaon_config1 = devm_platform_ioremap_resource_byname(pdev, + "mpm_sscaon_config1"); if (IS_ERR(data->reg_mpm_sscaon_config1)) return dev_err_probe(&pdev->dev, PTR_ERR(data->reg_mpm_sscaon_config1), "Failed to ioremap mpm_sscaon_config1\n"); -- 2.17.1
