Commit 4aac24c1 authored by Li Zetao's avatar Li Zetao Committed by Jassi Brar
Browse files

mailbox: ti-msgmgr: Use devm_platform_ioremap_resource_byname() 



Convert platform_get_resource_byname() + devm_ioremap_resource() to a
single call to devm_platform_ioremap_resource_byname(), as this is
exactly what this function does.

Signed-off-by: default avatarLi Zetao <lizetao1@huawei.com>
Signed-off-by: default avatarJassi Brar <jaswinder.singh@linaro.org>
parent 65d9aa31

drivers/mailbox/ti-msgmgr.c

+6 −10
Original line number Diff line number Diff line
@@ -812,7 +812,6 @@ static int ti_msgmgr_probe(struct platform_device *pdev) 
	struct device *dev = &pdev->dev;

	const struct of_device_id *of_id;

	struct device_node *np;

	struct resource *res;

	const struct ti_msgmgr_desc *desc;

	struct ti_msgmgr_inst *inst;

	struct ti_queue_inst *qinst;
@@ -843,22 +842,19 @@ static int ti_msgmgr_probe(struct platform_device *pdev) 
	inst->dev = dev;

	inst->desc = desc;



	res = platform_get_resource_byname(pdev, IORESOURCE_MEM,

					   desc->data_region_name);

	inst->queue_proxy_region = devm_ioremap_resource(dev, res);

	inst->queue_proxy_region =

		devm_platform_ioremap_resource_byname(pdev, desc->data_region_name);

	if (IS_ERR(inst->queue_proxy_region))

		return PTR_ERR(inst->queue_proxy_region);



	res = platform_get_resource_byname(pdev, IORESOURCE_MEM,

					   desc->status_region_name);

	inst->queue_state_debug_region = devm_ioremap_resource(dev, res);

	inst->queue_state_debug_region =

		devm_platform_ioremap_resource_byname(pdev, desc->status_region_name);

	if (IS_ERR(inst->queue_state_debug_region))

		return PTR_ERR(inst->queue_state_debug_region);



	if (desc->is_sproxy) {

		res = platform_get_resource_byname(pdev, IORESOURCE_MEM,

						   desc->ctrl_region_name);

		inst->queue_ctrl_region = devm_ioremap_resource(dev, res);

		inst->queue_ctrl_region =

			devm_platform_ioremap_resource_byname(pdev, desc->ctrl_region_name);

		if (IS_ERR(inst->queue_ctrl_region))

			return PTR_ERR(inst->queue_ctrl_region);

	}