On Thu 09 Apr 06:52 PDT 2020, Dejin Zheng wrote: > use devm_platform_ioremap_resource() to simplify code, it > contains platform_get_resource and devm_ioremap_resource. > > Reviewed-by: Barry Song <baohua@xxxxxxxxxx> > Acked-by: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx> > Acked-by: Robert Richter <rrichter@xxxxxxxxxxx> > Acked-by: Thor Thayer <thor.thayer@xxxxxxxxxxxxxxx> > Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> > Suggested-by: Wolfram Sang <wsa@xxxxxxxxxxxxx> Didn't Wolfram "only" suggest that you should squash your series? > Signed-off-by: Dejin Zheng <zhengdejin5@xxxxxxxxx> Change looks good though. Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx> Regards, Bjorn > --- > > v2 -> v3: > - squash all 24 patches into one by Wolfram's suggestion. > v1 -> v2: > - delete 4 patches of can not be sent by gmail server. > then, has 24 patches. > > drivers/i2c/busses/i2c-altera.c | 4 +--- > drivers/i2c/busses/i2c-axxia.c | 4 +--- > drivers/i2c/busses/i2c-bcm-kona.c | 4 +--- > drivers/i2c/busses/i2c-davinci.c | 4 +--- > drivers/i2c/busses/i2c-digicolor.c | 4 +--- > drivers/i2c/busses/i2c-emev2.c | 4 +--- > drivers/i2c/busses/i2c-exynos5.c | 4 +--- > drivers/i2c/busses/i2c-hix5hd2.c | 4 +--- > drivers/i2c/busses/i2c-img-scb.c | 4 +--- > drivers/i2c/busses/i2c-jz4780.c | 4 +--- > drivers/i2c/busses/i2c-lpc2k.c | 4 +--- > drivers/i2c/busses/i2c-meson.c | 4 +--- > drivers/i2c/busses/i2c-mv64xxx.c | 4 +--- > drivers/i2c/busses/i2c-octeon-platdrv.c | 4 +--- > drivers/i2c/busses/i2c-owl.c | 4 +--- > drivers/i2c/busses/i2c-qup.c | 4 +--- > drivers/i2c/busses/i2c-rk3x.c | 4 +--- > drivers/i2c/busses/i2c-sirf.c | 4 +--- > drivers/i2c/busses/i2c-stu300.c | 4 +--- > drivers/i2c/busses/i2c-sun6i-p2wi.c | 4 +--- > drivers/i2c/busses/i2c-synquacer.c | 4 +--- > drivers/i2c/busses/i2c-xlp9xx.c | 4 +--- > drivers/i2c/busses/i2c-xlr.c | 4 +--- > drivers/i2c/busses/i2c-zx2967.c | 4 +--- > 24 files changed, 24 insertions(+), 72 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-altera.c b/drivers/i2c/busses/i2c-altera.c > index 1de23b4f3809..7c4c836a3228 100644 > --- a/drivers/i2c/busses/i2c-altera.c > +++ b/drivers/i2c/busses/i2c-altera.c > @@ -382,7 +382,6 @@ static const struct i2c_algorithm altr_i2c_algo = { > static int altr_i2c_probe(struct platform_device *pdev) > { > struct altr_i2c_dev *idev = NULL; > - struct resource *res; > int irq, ret; > u32 val; > > @@ -390,8 +389,7 @@ static int altr_i2c_probe(struct platform_device *pdev) > if (!idev) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - idev->base = devm_ioremap_resource(&pdev->dev, res); > + idev->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(idev->base)) > return PTR_ERR(idev->base); > > diff --git a/drivers/i2c/busses/i2c-axxia.c b/drivers/i2c/busses/i2c-axxia.c > index 0214daa913ff..0aa0753e2430 100644 > --- a/drivers/i2c/busses/i2c-axxia.c > +++ b/drivers/i2c/busses/i2c-axxia.c > @@ -734,7 +734,6 @@ static int axxia_i2c_probe(struct platform_device *pdev) > { > struct device_node *np = pdev->dev.of_node; > struct axxia_i2c_dev *idev = NULL; > - struct resource *res; > void __iomem *base; > int ret = 0; > > @@ -742,8 +741,7 @@ static int axxia_i2c_probe(struct platform_device *pdev) > if (!idev) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - base = devm_ioremap_resource(&pdev->dev, res); > + base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(base)) > return PTR_ERR(base); > > diff --git a/drivers/i2c/busses/i2c-bcm-kona.c b/drivers/i2c/busses/i2c-bcm-kona.c > index 4e489a9d16fb..2bde3b5bad04 100644 > --- a/drivers/i2c/busses/i2c-bcm-kona.c > +++ b/drivers/i2c/busses/i2c-bcm-kona.c > @@ -750,7 +750,6 @@ static int bcm_kona_i2c_probe(struct platform_device *pdev) > int rc = 0; > struct bcm_kona_i2c_dev *dev; > struct i2c_adapter *adap; > - struct resource *iomem; > > /* Allocate memory for private data structure */ > dev = devm_kzalloc(&pdev->dev, sizeof(*dev), GFP_KERNEL); > @@ -762,8 +761,7 @@ static int bcm_kona_i2c_probe(struct platform_device *pdev) > init_completion(&dev->done); > > /* Map hardware registers */ > - iomem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - dev->base = devm_ioremap_resource(dev->device, iomem); > + dev->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(dev->base)) > return -ENOMEM; > > diff --git a/drivers/i2c/busses/i2c-davinci.c b/drivers/i2c/busses/i2c-davinci.c > index e3ceb256a380..232a7679b69b 100644 > --- a/drivers/i2c/busses/i2c-davinci.c > +++ b/drivers/i2c/busses/i2c-davinci.c > @@ -761,7 +761,6 @@ static int davinci_i2c_probe(struct platform_device *pdev) > { > struct davinci_i2c_dev *dev; > struct i2c_adapter *adap; > - struct resource *mem; > struct i2c_bus_recovery_info *rinfo; > int r, irq; > > @@ -814,8 +813,7 @@ static int davinci_i2c_probe(struct platform_device *pdev) > if (IS_ERR(dev->clk)) > return PTR_ERR(dev->clk); > > - mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - dev->base = devm_ioremap_resource(&pdev->dev, mem); > + dev->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(dev->base)) { > return PTR_ERR(dev->base); > } > diff --git a/drivers/i2c/busses/i2c-digicolor.c b/drivers/i2c/busses/i2c-digicolor.c > index 3adf72540db1..c199ed3334f9 100644 > --- a/drivers/i2c/busses/i2c-digicolor.c > +++ b/drivers/i2c/busses/i2c-digicolor.c > @@ -291,7 +291,6 @@ static int dc_i2c_probe(struct platform_device *pdev) > { > struct device_node *np = pdev->dev.of_node; > struct dc_i2c *i2c; > - struct resource *r; > int ret = 0, irq; > > i2c = devm_kzalloc(&pdev->dev, sizeof(struct dc_i2c), GFP_KERNEL); > @@ -312,8 +311,7 @@ static int dc_i2c_probe(struct platform_device *pdev) > if (IS_ERR(i2c->clk)) > return PTR_ERR(i2c->clk); > > - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->regs = devm_ioremap_resource(&pdev->dev, r); > + i2c->regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->regs)) > return PTR_ERR(i2c->regs); > > diff --git a/drivers/i2c/busses/i2c-emev2.c b/drivers/i2c/busses/i2c-emev2.c > index 959d4912ec0d..1a319352e51b 100644 > --- a/drivers/i2c/busses/i2c-emev2.c > +++ b/drivers/i2c/busses/i2c-emev2.c > @@ -361,15 +361,13 @@ static const struct i2c_algorithm em_i2c_algo = { > static int em_i2c_probe(struct platform_device *pdev) > { > struct em_i2c_device *priv; > - struct resource *r; > int ret; > > priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); > if (!priv) > return -ENOMEM; > > - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - priv->base = devm_ioremap_resource(&pdev->dev, r); > + priv->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(priv->base)) > return PTR_ERR(priv->base); > > diff --git a/drivers/i2c/busses/i2c-exynos5.c b/drivers/i2c/busses/i2c-exynos5.c > index e7514c16b756..5bf716df88f4 100644 > --- a/drivers/i2c/busses/i2c-exynos5.c > +++ b/drivers/i2c/busses/i2c-exynos5.c > @@ -740,7 +740,6 @@ static int exynos5_i2c_probe(struct platform_device *pdev) > { > struct device_node *np = pdev->dev.of_node; > struct exynos5_i2c *i2c; > - struct resource *mem; > int ret; > > i2c = devm_kzalloc(&pdev->dev, sizeof(struct exynos5_i2c), GFP_KERNEL); > @@ -766,8 +765,7 @@ static int exynos5_i2c_probe(struct platform_device *pdev) > if (ret) > return ret; > > - mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->regs = devm_ioremap_resource(&pdev->dev, mem); > + i2c->regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->regs)) { > ret = PTR_ERR(i2c->regs); > goto err_clk; > diff --git a/drivers/i2c/busses/i2c-hix5hd2.c b/drivers/i2c/busses/i2c-hix5hd2.c > index 224f830f77f9..4b72cb3b4231 100644 > --- a/drivers/i2c/busses/i2c-hix5hd2.c > +++ b/drivers/i2c/busses/i2c-hix5hd2.c > @@ -390,7 +390,6 @@ static int hix5hd2_i2c_probe(struct platform_device *pdev) > { > struct device_node *np = pdev->dev.of_node; > struct hix5hd2_i2c_priv *priv; > - struct resource *mem; > unsigned int freq; > int irq, ret; > > @@ -411,8 +410,7 @@ static int hix5hd2_i2c_probe(struct platform_device *pdev) > } > } > > - mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - priv->regs = devm_ioremap_resource(&pdev->dev, mem); > + priv->regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(priv->regs)) > return PTR_ERR(priv->regs); > > diff --git a/drivers/i2c/busses/i2c-img-scb.c b/drivers/i2c/busses/i2c-img-scb.c > index 20a4fbc53007..5e0022e55315 100644 > --- a/drivers/i2c/busses/i2c-img-scb.c > +++ b/drivers/i2c/busses/i2c-img-scb.c > @@ -1330,7 +1330,6 @@ static int img_i2c_probe(struct platform_device *pdev) > { > struct device_node *node = pdev->dev.of_node; > struct img_i2c *i2c; > - struct resource *res; > int irq, ret; > u32 val; > > @@ -1338,8 +1337,7 @@ static int img_i2c_probe(struct platform_device *pdev) > if (!i2c) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->base = devm_ioremap_resource(&pdev->dev, res); > + i2c->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->base)) > return PTR_ERR(i2c->base); > > diff --git a/drivers/i2c/busses/i2c-jz4780.c b/drivers/i2c/busses/i2c-jz4780.c > index b426fc956938..ba831df6661e 100644 > --- a/drivers/i2c/busses/i2c-jz4780.c > +++ b/drivers/i2c/busses/i2c-jz4780.c > @@ -763,7 +763,6 @@ static int jz4780_i2c_probe(struct platform_device *pdev) > int ret = 0; > unsigned int clk_freq = 0; > unsigned short tmp; > - struct resource *r; > struct jz4780_i2c *i2c; > > i2c = devm_kzalloc(&pdev->dev, sizeof(struct jz4780_i2c), GFP_KERNEL); > @@ -787,8 +786,7 @@ static int jz4780_i2c_probe(struct platform_device *pdev) > init_completion(&i2c->trans_waitq); > spin_lock_init(&i2c->lock); > > - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->iomem = devm_ioremap_resource(&pdev->dev, r); > + i2c->iomem = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->iomem)) > return PTR_ERR(i2c->iomem); > > diff --git a/drivers/i2c/busses/i2c-lpc2k.c b/drivers/i2c/busses/i2c-lpc2k.c > index deea18b14add..30f6dd1d37c7 100644 > --- a/drivers/i2c/busses/i2c-lpc2k.c > +++ b/drivers/i2c/busses/i2c-lpc2k.c > @@ -346,7 +346,6 @@ static const struct i2c_algorithm i2c_lpc2k_algorithm = { > static int i2c_lpc2k_probe(struct platform_device *pdev) > { > struct lpc2k_i2c *i2c; > - struct resource *res; > u32 bus_clk_rate; > u32 scl_high; > u32 clkrate; > @@ -356,8 +355,7 @@ static int i2c_lpc2k_probe(struct platform_device *pdev) > if (!i2c) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->base = devm_ioremap_resource(&pdev->dev, res); > + i2c->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->base)) > return PTR_ERR(i2c->base); > > diff --git a/drivers/i2c/busses/i2c-meson.c b/drivers/i2c/busses/i2c-meson.c > index 06b3bed78421..b3bd869281d3 100644 > --- a/drivers/i2c/busses/i2c-meson.c > +++ b/drivers/i2c/busses/i2c-meson.c > @@ -397,7 +397,6 @@ static int meson_i2c_probe(struct platform_device *pdev) > { > struct device_node *np = pdev->dev.of_node; > struct meson_i2c *i2c; > - struct resource *mem; > struct i2c_timings timings; > int irq, ret = 0; > > @@ -422,8 +421,7 @@ static int meson_i2c_probe(struct platform_device *pdev) > return PTR_ERR(i2c->clk); > } > > - mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->regs = devm_ioremap_resource(&pdev->dev, mem); > + i2c->regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->regs)) > return PTR_ERR(i2c->regs); > > diff --git a/drivers/i2c/busses/i2c-mv64xxx.c b/drivers/i2c/busses/i2c-mv64xxx.c > index febb7c7ea72b..62fb1ecc84a2 100644 > --- a/drivers/i2c/busses/i2c-mv64xxx.c > +++ b/drivers/i2c/busses/i2c-mv64xxx.c > @@ -877,7 +877,6 @@ mv64xxx_i2c_probe(struct platform_device *pd) > { > struct mv64xxx_i2c_data *drv_data; > struct mv64xxx_i2c_pdata *pdata = dev_get_platdata(&pd->dev); > - struct resource *r; > int rc; > > if ((!pdata && !pd->dev.of_node)) > @@ -888,8 +887,7 @@ mv64xxx_i2c_probe(struct platform_device *pd) > if (!drv_data) > return -ENOMEM; > > - r = platform_get_resource(pd, IORESOURCE_MEM, 0); > - drv_data->reg_base = devm_ioremap_resource(&pd->dev, r); > + drv_data->reg_base = devm_platform_ioremap_resource(pd, 0); > if (IS_ERR(drv_data->reg_base)) > return PTR_ERR(drv_data->reg_base); > > diff --git a/drivers/i2c/busses/i2c-octeon-platdrv.c b/drivers/i2c/busses/i2c-octeon-platdrv.c > index 64bda83e65ac..0c227963c8d6 100644 > --- a/drivers/i2c/busses/i2c-octeon-platdrv.c > +++ b/drivers/i2c/busses/i2c-octeon-platdrv.c > @@ -136,7 +136,6 @@ static int octeon_i2c_probe(struct platform_device *pdev) > { > struct device_node *node = pdev->dev.of_node; > int irq, result = 0, hlc_irq = 0; > - struct resource *res_mem; > struct octeon_i2c *i2c; > bool cn78xx_style; > > @@ -167,8 +166,7 @@ static int octeon_i2c_probe(struct platform_device *pdev) > i2c->roff.twsi_int = 0x10; > i2c->roff.sw_twsi_ext = 0x18; > > - res_mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->twsi_base = devm_ioremap_resource(&pdev->dev, res_mem); > + i2c->twsi_base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->twsi_base)) { > result = PTR_ERR(i2c->twsi_base); > goto out; > diff --git a/drivers/i2c/busses/i2c-owl.c b/drivers/i2c/busses/i2c-owl.c > index b6b5a495118b..c9e7fcac12ae 100644 > --- a/drivers/i2c/busses/i2c-owl.c > +++ b/drivers/i2c/busses/i2c-owl.c > @@ -399,15 +399,13 @@ static int owl_i2c_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct owl_i2c_dev *i2c_dev; > - struct resource *res; > int ret, irq; > > i2c_dev = devm_kzalloc(dev, sizeof(*i2c_dev), GFP_KERNEL); > if (!i2c_dev) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c_dev->base = devm_ioremap_resource(dev, res); > + i2c_dev->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c_dev->base)) > return PTR_ERR(i2c_dev->base); > > diff --git a/drivers/i2c/busses/i2c-qup.c b/drivers/i2c/busses/i2c-qup.c > index 2d7dabe12723..940e9fcca8ec 100644 > --- a/drivers/i2c/busses/i2c-qup.c > +++ b/drivers/i2c/busses/i2c-qup.c > @@ -1665,7 +1665,6 @@ static int qup_i2c_probe(struct platform_device *pdev) > static const int blk_sizes[] = {4, 16, 32}; > struct qup_i2c_dev *qup; > unsigned long one_bit_t; > - struct resource *res; > u32 io_mode, hw_ver, size; > int ret, fs_div, hs_div; > u32 src_clk_freq = DEFAULT_SRC_CLK; > @@ -1762,8 +1761,7 @@ static int qup_i2c_probe(struct platform_device *pdev) > return -EINVAL; > } > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - qup->base = devm_ioremap_resource(qup->dev, res); > + qup->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(qup->base)) > return PTR_ERR(qup->base); > > diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c > index 1a33007b03e9..862a26f66b76 100644 > --- a/drivers/i2c/busses/i2c-rk3x.c > +++ b/drivers/i2c/busses/i2c-rk3x.c > @@ -1193,7 +1193,6 @@ static int rk3x_i2c_probe(struct platform_device *pdev) > struct device_node *np = pdev->dev.of_node; > const struct of_device_id *match; > struct rk3x_i2c *i2c; > - struct resource *mem; > int ret = 0; > int bus_nr; > u32 value; > @@ -1223,8 +1222,7 @@ static int rk3x_i2c_probe(struct platform_device *pdev) > spin_lock_init(&i2c->lock); > init_waitqueue_head(&i2c->wait); > > - mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->regs = devm_ioremap_resource(&pdev->dev, mem); > + i2c->regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->regs)) > return PTR_ERR(i2c->regs); > > diff --git a/drivers/i2c/busses/i2c-sirf.c b/drivers/i2c/busses/i2c-sirf.c > index fb7a046b3226..269a71b0ee7f 100644 > --- a/drivers/i2c/busses/i2c-sirf.c > +++ b/drivers/i2c/busses/i2c-sirf.c > @@ -272,7 +272,6 @@ static int i2c_sirfsoc_probe(struct platform_device *pdev) > { > struct sirfsoc_i2c *siic; > struct i2c_adapter *adap; > - struct resource *mem_res; > struct clk *clk; > int bitrate; > int ctrl_speed; > @@ -310,8 +309,7 @@ static int i2c_sirfsoc_probe(struct platform_device *pdev) > adap = &siic->adapter; > adap->class = I2C_CLASS_DEPRECATED; > > - mem_res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - siic->base = devm_ioremap_resource(&pdev->dev, mem_res); > + siic->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(siic->base)) { > err = PTR_ERR(siic->base); > goto out; > diff --git a/drivers/i2c/busses/i2c-stu300.c b/drivers/i2c/busses/i2c-stu300.c > index 42e0a53e7fa4..9a03779c149e 100644 > --- a/drivers/i2c/busses/i2c-stu300.c > +++ b/drivers/i2c/busses/i2c-stu300.c > @@ -860,7 +860,6 @@ static int stu300_probe(struct platform_device *pdev) > { > struct stu300_dev *dev; > struct i2c_adapter *adap; > - struct resource *res; > int bus_nr; > int ret = 0; > > @@ -876,8 +875,7 @@ static int stu300_probe(struct platform_device *pdev) > } > > dev->pdev = pdev; > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - dev->virtbase = devm_ioremap_resource(&pdev->dev, res); > + dev->virtbase = devm_platform_ioremap_resource(pdev, 0); > dev_dbg(&pdev->dev, "initialize bus device I2C%d on virtual " > "base %p\n", bus_nr, dev->virtbase); > if (IS_ERR(dev->virtbase)) > diff --git a/drivers/i2c/busses/i2c-sun6i-p2wi.c b/drivers/i2c/busses/i2c-sun6i-p2wi.c > index 7c07ce116e38..cb78c38ea3a6 100644 > --- a/drivers/i2c/busses/i2c-sun6i-p2wi.c > +++ b/drivers/i2c/busses/i2c-sun6i-p2wi.c > @@ -187,7 +187,6 @@ static int p2wi_probe(struct platform_device *pdev) > struct device_node *childnp; > unsigned long parent_clk_freq; > u32 clk_freq = 100000; > - struct resource *r; > struct p2wi *p2wi; > u32 slave_addr; > int clk_div; > @@ -231,8 +230,7 @@ static int p2wi_probe(struct platform_device *pdev) > p2wi->slave_addr = slave_addr; > } > > - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - p2wi->regs = devm_ioremap_resource(dev, r); > + p2wi->regs = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(p2wi->regs)) > return PTR_ERR(p2wi->regs); > > diff --git a/drivers/i2c/busses/i2c-synquacer.c b/drivers/i2c/busses/i2c-synquacer.c > index 86026798b4f7..be389e95b57a 100644 > --- a/drivers/i2c/busses/i2c-synquacer.c > +++ b/drivers/i2c/busses/i2c-synquacer.c > @@ -536,7 +536,6 @@ static const struct i2c_adapter synquacer_i2c_ops = { > static int synquacer_i2c_probe(struct platform_device *pdev) > { > struct synquacer_i2c *i2c; > - struct resource *r; > u32 bus_speed; > int ret; > > @@ -574,8 +573,7 @@ static int synquacer_i2c_probe(struct platform_device *pdev) > return -EINVAL; > } > > - r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - i2c->base = devm_ioremap_resource(&pdev->dev, r); > + i2c->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(i2c->base)) > return PTR_ERR(i2c->base); > > diff --git a/drivers/i2c/busses/i2c-xlp9xx.c b/drivers/i2c/busses/i2c-xlp9xx.c > index 8a873975cf12..beb2bb6905d6 100644 > --- a/drivers/i2c/busses/i2c-xlp9xx.c > +++ b/drivers/i2c/busses/i2c-xlp9xx.c > @@ -504,15 +504,13 @@ static int xlp9xx_i2c_smbus_setup(struct xlp9xx_i2c_dev *priv, > static int xlp9xx_i2c_probe(struct platform_device *pdev) > { > struct xlp9xx_i2c_dev *priv; > - struct resource *res; > int err = 0; > > priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL); > if (!priv) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - priv->base = devm_ioremap_resource(&pdev->dev, res); > + priv->base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(priv->base)) > return PTR_ERR(priv->base); > > diff --git a/drivers/i2c/busses/i2c-xlr.c b/drivers/i2c/busses/i2c-xlr.c > index 34cd4b308540..544fd9459d60 100644 > --- a/drivers/i2c/busses/i2c-xlr.c > +++ b/drivers/i2c/busses/i2c-xlr.c > @@ -362,7 +362,6 @@ static int xlr_i2c_probe(struct platform_device *pdev) > { > const struct of_device_id *match; > struct xlr_i2c_private *priv; > - struct resource *res; > struct clk *clk; > unsigned long clk_rate; > unsigned long clk_div; > @@ -380,8 +379,7 @@ static int xlr_i2c_probe(struct platform_device *pdev) > else > priv->cfg = &xlr_i2c_config_default; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - priv->iobase = devm_ioremap_resource(&pdev->dev, res); > + priv->iobase = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(priv->iobase)) > return PTR_ERR(priv->iobase); > > diff --git a/drivers/i2c/busses/i2c-zx2967.c b/drivers/i2c/busses/i2c-zx2967.c > index 5f3318559b8d..8db9519695a6 100644 > --- a/drivers/i2c/busses/i2c-zx2967.c > +++ b/drivers/i2c/busses/i2c-zx2967.c > @@ -502,7 +502,6 @@ static int zx2967_i2c_probe(struct platform_device *pdev) > { > struct zx2967_i2c *i2c; > void __iomem *reg_base; > - struct resource *res; > struct clk *clk; > int ret; > > @@ -510,8 +509,7 @@ static int zx2967_i2c_probe(struct platform_device *pdev) > if (!i2c) > return -ENOMEM; > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - reg_base = devm_ioremap_resource(&pdev->dev, res); > + reg_base = devm_platform_ioremap_resource(pdev, 0); > if (IS_ERR(reg_base)) > return PTR_ERR(reg_base); > > -- > 2.25.0 >