| Message ID | 20190905034932.12587-1-yamada.masahiro@socionext.com |
|---|---|
| State | Accepted |
| Commit | e89327f659dd517f30b6232b1fabd6f6c6777c3e |
| Headers | show |
| Series | irqchip: uniphier-aidet: use devm_platform_ioremap_resource() | expand |
On Thu, 05 Sep 2019 04:49:32 +0100, Masahiro Yamada <yamada.masahiro@socionext.com> wrote: > > Replace the chain of platform_get_resource() and devm_ioremap_resource() > with devm_platform_ioremap_resource(). > > This allows to remove the local variable for (struct resource *), and > have one function call less. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > drivers/irqchip/irq-uniphier-aidet.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) Applied to irqchip-next. Thanks, M. -- Jazz is not dead, it just smells funny.
diff --git a/drivers/irqchip/irq-uniphier-aidet.c b/drivers/irqchip/irq-uniphier-aidet.c index ed7b4f47ff3f..89121b39be26 100644 --- a/drivers/irqchip/irq-uniphier-aidet.c +++ b/drivers/irqchip/irq-uniphier-aidet.c @@ -166,7 +166,6 @@ static int uniphier_aidet_probe(struct platform_device *pdev) struct device_node *parent_np; struct irq_domain *parent_domain; struct uniphier_aidet_priv *priv; - struct resource *res; parent_np = of_irq_find_parent(dev->of_node); if (!parent_np) @@ -181,8 +180,7 @@ static int uniphier_aidet_probe(struct platform_device *pdev) if (!priv) return -ENOMEM; - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); - priv->reg_base = devm_ioremap_resource(dev, res); + priv->reg_base = devm_platform_ioremap_resource(pdev, 0); if (IS_ERR(priv->reg_base)) return PTR_ERR(priv->reg_base);
Replace the chain of platform_get_resource() and devm_ioremap_resource() with devm_platform_ioremap_resource(). This allows to remove the local variable for (struct resource *), and have one function call less. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- drivers/irqchip/irq-uniphier-aidet.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) -- 2.17.1