LinuxPPC-Dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH -next 12/34] ASoC: spear: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/spear/spdif_in.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/sound/soc/spear/spdif_in.c b/sound/soc/spear/spdif_in.c
index 78a6a36..4b68d6e 100644
--- a/sound/soc/spear/spdif_in.c
+++ b/sound/soc/spear/spdif_in.c
@@ -202,12 +202,11 @@ static int spdif_in_probe(struct platform_device *pdev)
 {
 	struct spdif_in_dev *host;
 	struct spear_spdif_platform_data *pdata;
-	struct resource *res, *res_fifo;
+	struct resource *res_fifo;
 	void __iomem *io_base;
 	int ret;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	io_base = devm_ioremap_resource(&pdev->dev, res);
+	io_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(io_base))
 		return PTR_ERR(io_base);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 13/34] ASoC: kirkwood-i2s: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/kirkwood/kirkwood-i2s.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/kirkwood/kirkwood-i2s.c b/sound/soc/kirkwood/kirkwood-i2s.c
index 3446a113..5076ec4 100644
--- a/sound/soc/kirkwood/kirkwood-i2s.c
+++ b/sound/soc/kirkwood/kirkwood-i2s.c
@@ -523,7 +523,6 @@ static int kirkwood_i2s_dev_probe(struct platform_device *pdev)
 	struct kirkwood_asoc_platform_data *data = pdev->dev.platform_data;
 	struct snd_soc_dai_driver *soc_dai = kirkwood_i2s_dai;
 	struct kirkwood_dma_data *priv;
-	struct resource *mem;
 	struct device_node *np = pdev->dev.of_node;
 	int err;
 
@@ -533,8 +532,7 @@ static int kirkwood_i2s_dev_probe(struct platform_device *pdev)
 
 	dev_set_drvdata(&pdev->dev, priv);
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	priv->io = devm_ioremap_resource(&pdev->dev, mem);
+	priv->io = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(priv->io))
 		return PTR_ERR(priv->io);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 15/34] ASoC: stm32: sai: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/stm/stm32_sai.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/stm/stm32_sai.c b/sound/soc/stm/stm32_sai.c
index 63f68e6..1ac5103c 100644
--- a/sound/soc/stm/stm32_sai.c
+++ b/sound/soc/stm/stm32_sai.c
@@ -152,7 +152,6 @@ static int stm32_sai_probe(struct platform_device *pdev)
 {
 	struct stm32_sai_data *sai;
 	struct reset_control *rst;
-	struct resource *res;
 	const struct of_device_id *of_id;
 	u32 val;
 	int ret;
@@ -161,8 +160,7 @@ static int stm32_sai_probe(struct platform_device *pdev)
 	if (!sai)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	sai->base = devm_ioremap_resource(&pdev->dev, res);
+	sai->base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(sai->base))
 		return PTR_ERR(sai->base);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 14/34] ASoC: xtfpga-i2s: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/xtensa/xtfpga-i2s.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/xtensa/xtfpga-i2s.c b/sound/soc/xtensa/xtfpga-i2s.c
index 9ce2c75..9da395d 100644
--- a/sound/soc/xtensa/xtfpga-i2s.c
+++ b/sound/soc/xtensa/xtfpga-i2s.c
@@ -531,7 +531,6 @@ static int xtfpga_i2s_runtime_resume(struct device *dev)
 static int xtfpga_i2s_probe(struct platform_device *pdev)
 {
 	struct xtfpga_i2s *i2s;
-	struct resource *mem;
 	int err, irq;
 
 	i2s = devm_kzalloc(&pdev->dev, sizeof(*i2s), GFP_KERNEL);
@@ -543,8 +542,7 @@ static int xtfpga_i2s_probe(struct platform_device *pdev)
 	i2s->dev = &pdev->dev;
 	dev_dbg(&pdev->dev, "dev: %p, i2s: %p\n", &pdev->dev, i2s);
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	i2s->regs = devm_ioremap_resource(&pdev->dev, mem);
+	i2s->regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(i2s->regs)) {
 		err = PTR_ERR(i2s->regs);
 		goto err;
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 16/34] ASoC: codecs: msm8916-wcd: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/codecs/msm8916-wcd-digital.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/codecs/msm8916-wcd-digital.c b/sound/soc/codecs/msm8916-wcd-digital.c
index 1db7e43..9fa5d44 100644
--- a/sound/soc/codecs/msm8916-wcd-digital.c
+++ b/sound/soc/codecs/msm8916-wcd-digital.c
@@ -1143,7 +1143,6 @@ static int msm8916_wcd_digital_probe(struct platform_device *pdev)
 	struct msm8916_wcd_digital_priv *priv;
 	struct device *dev = &pdev->dev;
 	void __iomem *base;
-	struct resource *mem_res;
 	struct regmap *digital_map;
 	int ret;
 
@@ -1151,8 +1150,7 @@ static int msm8916_wcd_digital_probe(struct platform_device *pdev)
 	if (!priv)
 		return -ENOMEM;
 
-	mem_res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(&pdev->dev, mem_res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 17/34] ASoC: codecs: jz4725b: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/codecs/jz4725b.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/codecs/jz4725b.c b/sound/soc/codecs/jz4725b.c
index 766354c..2567a5d 100644
--- a/sound/soc/codecs/jz4725b.c
+++ b/sound/soc/codecs/jz4725b.c
@@ -545,15 +545,13 @@ static int jz4725b_codec_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
 	struct jz_icdc *icdc;
-	struct resource *mem;
 	int ret;
 
 	icdc = devm_kzalloc(dev, sizeof(*icdc), GFP_KERNEL);
 	if (!icdc)
 		return -ENOMEM;
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	icdc->base = devm_ioremap_resource(dev, mem);
+	icdc->base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(icdc->base))
 		return PTR_ERR(icdc->base);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 18/34] ASoC: mmp-sspa: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/pxa/mmp-sspa.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/pxa/mmp-sspa.c b/sound/soc/pxa/mmp-sspa.c
index 72f4364..e3e5425 100644
--- a/sound/soc/pxa/mmp-sspa.c
+++ b/sound/soc/pxa/mmp-sspa.c
@@ -399,7 +399,6 @@ static const struct snd_soc_component_driver mmp_sspa_component = {
 static int asoc_mmp_sspa_probe(struct platform_device *pdev)
 {
 	struct sspa_priv *priv;
-	struct resource *res;
 
 	priv = devm_kzalloc(&pdev->dev,
 				sizeof(struct sspa_priv), GFP_KERNEL);
@@ -417,8 +416,7 @@ static int asoc_mmp_sspa_probe(struct platform_device *pdev)
 	if (priv->dma_params == NULL)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	priv->sspa->mmio_base = devm_ioremap_resource(&pdev->dev, res);
+	priv->sspa->mmio_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(priv->sspa->mmio_base))
 		return PTR_ERR(priv->sspa->mmio_base);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 19/34] ASoC: jz4740: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/codecs/jz4740.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/codecs/jz4740.c b/sound/soc/codecs/jz4740.c
index 974e17f..460aa1f 100644
--- a/sound/soc/codecs/jz4740.c
+++ b/sound/soc/codecs/jz4740.c
@@ -318,7 +318,6 @@ static int jz4740_codec_probe(struct platform_device *pdev)
 {
 	int ret;
 	struct jz4740_codec *jz4740_codec;
-	struct resource *mem;
 	void __iomem *base;
 
 	jz4740_codec = devm_kzalloc(&pdev->dev, sizeof(*jz4740_codec),
@@ -326,8 +325,7 @@ static int jz4740_codec_probe(struct platform_device *pdev)
 	if (!jz4740_codec)
 		return -ENOMEM;
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(&pdev->dev, mem);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 20/34] ASoC: inno_rk3036: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/codecs/inno_rk3036.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/codecs/inno_rk3036.c b/sound/soc/codecs/inno_rk3036.c
index 7feedbb..14d8fe1 100644
--- a/sound/soc/codecs/inno_rk3036.c
+++ b/sound/soc/codecs/inno_rk3036.c
@@ -405,7 +405,6 @@ static int rk3036_codec_platform_probe(struct platform_device *pdev)
 {
 	struct rk3036_codec_priv *priv;
 	struct device_node *of_node = pdev->dev.of_node;
-	struct resource *res;
 	void __iomem *base;
 	struct regmap *grf;
 	int ret;
@@ -414,8 +413,7 @@ static int rk3036_codec_platform_probe(struct platform_device *pdev)
 	if (!priv)
 		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);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 21/34] ASoC: uniphier: evea: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/uniphier/evea.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/uniphier/evea.c b/sound/soc/uniphier/evea.c
index f9c1016..d27e9ca 100644
--- a/sound/soc/uniphier/evea.c
+++ b/sound/soc/uniphier/evea.c
@@ -451,7 +451,6 @@ static const struct regmap_config evea_regmap_config = {
 static int evea_probe(struct platform_device *pdev)
 {
 	struct evea_priv *evea;
-	struct resource *res;
 	void __iomem *preg;
 	int ret;
 
@@ -475,8 +474,7 @@ static int evea_probe(struct platform_device *pdev)
 	if (IS_ERR(evea->rst_exiv))
 		return PTR_ERR(evea->rst_exiv);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	preg = devm_ioremap_resource(&pdev->dev, res);
+	preg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(preg))
 		return PTR_ERR(preg);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 22/34] ASoC: uniphier: aio-dma: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/uniphier/aio-dma.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/uniphier/aio-dma.c b/sound/soc/uniphier/aio-dma.c
index fa001d3..862346d 100644
--- a/sound/soc/uniphier/aio-dma.c
+++ b/sound/soc/uniphier/aio-dma.c
@@ -276,12 +276,10 @@ int uniphier_aiodma_soc_register_platform(struct platform_device *pdev)
 {
 	struct uniphier_aio_chip *chip = platform_get_drvdata(pdev);
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	void __iomem *preg;
 	int irq, ret;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	preg = devm_ioremap_resource(dev, res);
+	preg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(preg))
 		return PTR_ERR(preg);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 23/34] ASoC: psc-ac97: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/au1x/psc-ac97.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/sound/soc/au1x/psc-ac97.c b/sound/soc/au1x/psc-ac97.c
index 21e5f6a..08bc04e 100644
--- a/sound/soc/au1x/psc-ac97.c
+++ b/sound/soc/au1x/psc-ac97.c
@@ -363,7 +363,7 @@ static const struct snd_soc_component_driver au1xpsc_ac97_component = {
 static int au1xpsc_ac97_drvprobe(struct platform_device *pdev)
 {
 	int ret;
-	struct resource *iores, *dmares;
+	struct resource *dmares;
 	unsigned long sel;
 	struct au1xpsc_audio_data *wd;
 
@@ -374,8 +374,7 @@ static int au1xpsc_ac97_drvprobe(struct platform_device *pdev)
 
 	mutex_init(&wd->lock);
 
-	iores = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	wd->mmio = devm_ioremap_resource(&pdev->dev, iores);
+	wd->mmio = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(wd->mmio))
 		return PTR_ERR(wd->mmio);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 24/34] ASoC: au1x: psc-i2s: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/au1x/psc-i2s.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/sound/soc/au1x/psc-i2s.c b/sound/soc/au1x/psc-i2s.c
index 076303f..767ce95 100644
--- a/sound/soc/au1x/psc-i2s.c
+++ b/sound/soc/au1x/psc-i2s.c
@@ -291,7 +291,7 @@ static const struct snd_soc_component_driver au1xpsc_i2s_component = {
 
 static int au1xpsc_i2s_drvprobe(struct platform_device *pdev)
 {
-	struct resource *iores, *dmares;
+	struct resource *dmares;
 	unsigned long sel;
 	struct au1xpsc_audio_data *wd;
 
@@ -300,8 +300,7 @@ static int au1xpsc_i2s_drvprobe(struct platform_device *pdev)
 	if (!wd)
 		return -ENOMEM;
 
-	iores = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	wd->mmio = devm_ioremap_resource(&pdev->dev, iores);
+	wd->mmio = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(wd->mmio))
 		return PTR_ERR(wd->mmio);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 25/34] ASoC: meson: g12a-tohdmitx: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/meson/g12a-tohdmitx.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/meson/g12a-tohdmitx.c b/sound/soc/meson/g12a-tohdmitx.c
index 707ccb1..6ae0c4f 100644
--- a/sound/soc/meson/g12a-tohdmitx.c
+++ b/sound/soc/meson/g12a-tohdmitx.c
@@ -378,12 +378,10 @@ MODULE_DEVICE_TABLE(of, g12a_tohdmitx_of_match);
 static int g12a_tohdmitx_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	void __iomem *regs;
 	struct regmap *map;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	regs = devm_ioremap_resource(dev, res);
+	regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(regs))
 		return PTR_ERR(regs);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 26/34] ASoC: meson: axg-tdm-formatter: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/meson/axg-tdm-formatter.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/meson/axg-tdm-formatter.c b/sound/soc/meson/axg-tdm-formatter.c
index 2e49820..21c735a 100644
--- a/sound/soc/meson/axg-tdm-formatter.c
+++ b/sound/soc/meson/axg-tdm-formatter.c
@@ -253,7 +253,6 @@ int axg_tdm_formatter_probe(struct platform_device *pdev)
 	struct device *dev = &pdev->dev;
 	const struct axg_tdm_formatter_driver *drv;
 	struct axg_tdm_formatter *formatter;
-	struct resource *res;
 	void __iomem *regs;
 	int ret;
 
@@ -269,8 +268,7 @@ int axg_tdm_formatter_probe(struct platform_device *pdev)
 	platform_set_drvdata(pdev, formatter);
 	formatter->drv = drv;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	regs = devm_ioremap_resource(dev, res);
+	regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(regs))
 		return PTR_ERR(regs);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 27/34] ASoC: meson: axg-pdm: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/meson/axg-pdm.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/meson/axg-pdm.c b/sound/soc/meson/axg-pdm.c
index 9d56844..bfd37d4 100644
--- a/sound/soc/meson/axg-pdm.c
+++ b/sound/soc/meson/axg-pdm.c
@@ -585,7 +585,6 @@ static int axg_pdm_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
 	struct axg_pdm *priv;
-	struct resource *res;
 	void __iomem *regs;
 	int ret;
 
@@ -600,8 +599,7 @@ static int axg_pdm_probe(struct platform_device *pdev)
 		return -ENODEV;
 	}
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	regs = devm_ioremap_resource(dev, res);
+	regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(regs))
 		return PTR_ERR(regs);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 29/34] ASoC: meson: axg-spdifout: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/meson/axg-spdifout.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/meson/axg-spdifout.c b/sound/soc/meson/axg-spdifout.c
index 9dea528..7ce6aa97 100644
--- a/sound/soc/meson/axg-spdifout.c
+++ b/sound/soc/meson/axg-spdifout.c
@@ -401,7 +401,6 @@ static int axg_spdifout_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
 	struct axg_spdifout *priv;
-	struct resource *res;
 	void __iomem *regs;
 	int ret;
 
@@ -410,8 +409,7 @@ static int axg_spdifout_probe(struct platform_device *pdev)
 		return -ENOMEM;
 	platform_set_drvdata(pdev, priv);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	regs = devm_ioremap_resource(dev, res);
+	regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(regs))
 		return PTR_ERR(regs);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 28/34] ASoC: meson: axg-spdifin: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/meson/axg-spdifin.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/meson/axg-spdifin.c b/sound/soc/meson/axg-spdifin.c
index 01b2035..d0d09f9 100644
--- a/sound/soc/meson/axg-spdifin.c
+++ b/sound/soc/meson/axg-spdifin.c
@@ -453,7 +453,6 @@ static int axg_spdifin_probe(struct platform_device *pdev)
 	struct device *dev = &pdev->dev;
 	struct axg_spdifin *priv;
 	struct snd_soc_dai_driver *dai_drv;
-	struct resource *res;
 	void __iomem *regs;
 	int ret;
 
@@ -468,8 +467,7 @@ static int axg_spdifin_probe(struct platform_device *pdev)
 		return -ENODEV;
 	}
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	regs = devm_ioremap_resource(dev, res);
+	regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(regs))
 		return PTR_ERR(regs);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 30/34] ASoC: meson: axg-fifo: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/meson/axg-fifo.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/meson/axg-fifo.c b/sound/soc/meson/axg-fifo.c
index 01c1c7d..80a3dde 100644
--- a/sound/soc/meson/axg-fifo.c
+++ b/sound/soc/meson/axg-fifo.c
@@ -314,7 +314,6 @@ int axg_fifo_probe(struct platform_device *pdev)
 	struct device *dev = &pdev->dev;
 	const struct axg_fifo_match_data *data;
 	struct axg_fifo *fifo;
-	struct resource *res;
 	void __iomem *regs;
 
 	data = of_device_get_match_data(dev);
@@ -328,8 +327,7 @@ int axg_fifo_probe(struct platform_device *pdev)
 		return -ENOMEM;
 	platform_set_drvdata(pdev, fifo);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	regs = devm_ioremap_resource(dev, res);
+	regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(regs))
 		return PTR_ERR(regs);
 
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 31/34] ASoC: xlnx: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/xilinx/xlnx_i2s.c   | 4 +---
 sound/soc/xilinx/xlnx_spdif.c | 3 +--
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/sound/soc/xilinx/xlnx_i2s.c b/sound/soc/xilinx/xlnx_i2s.c
index 8b35316..cc641e5 100644
--- a/sound/soc/xilinx/xlnx_i2s.c
+++ b/sound/soc/xilinx/xlnx_i2s.c
@@ -95,7 +95,6 @@ MODULE_DEVICE_TABLE(of, xlnx_i2s_of_match);
 
 static int xlnx_i2s_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	void __iomem *base;
 	struct snd_soc_dai_driver *dai_drv;
 	int ret;
@@ -107,8 +106,7 @@ static int xlnx_i2s_probe(struct platform_device *pdev)
 	if (!dai_drv)
 		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/sound/soc/xilinx/xlnx_spdif.c b/sound/soc/xilinx/xlnx_spdif.c
index 3b9000f..e2ca087 100644
--- a/sound/soc/xilinx/xlnx_spdif.c
+++ b/sound/soc/xilinx/xlnx_spdif.c
@@ -260,8 +260,7 @@ static int xlnx_spdif_probe(struct platform_device *pdev)
 		return ret;
 	}
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	ctx->base = devm_ioremap_resource(dev, res);
+	ctx->base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(ctx->base)) {
 		ret = PTR_ERR(ctx->base);
 		goto clk_err;
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 32/34] ASoC: sun8i-codec-analog: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/sunxi/sun8i-codec-analog.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/sunxi/sun8i-codec-analog.c b/sound/soc/sunxi/sun8i-codec-analog.c
index e92aeed..be872ee 100644
--- a/sound/soc/sunxi/sun8i-codec-analog.c
+++ b/sound/soc/sunxi/sun8i-codec-analog.c
@@ -819,12 +819,10 @@ MODULE_DEVICE_TABLE(of, sun8i_codec_analog_of_match);
 
 static int sun8i_codec_analog_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct regmap *regmap;
 	void __iomem *base;
 
-	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)) {
 		dev_err(&pdev->dev, "Failed to map the registers\n");
 		return PTR_ERR(base);
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 33/34] ASoC: sunxi: sun50i-codec-analog: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/sunxi/sun50i-codec-analog.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/sunxi/sun50i-codec-analog.c b/sound/soc/sunxi/sun50i-codec-analog.c
index 6d1de56..f5b7069 100644
--- a/sound/soc/sunxi/sun50i-codec-analog.c
+++ b/sound/soc/sunxi/sun50i-codec-analog.c
@@ -459,12 +459,10 @@ MODULE_DEVICE_TABLE(of, sun50i_codec_analog_of_match);
 
 static int sun50i_codec_analog_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct regmap *regmap;
 	void __iomem *base;
 
-	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)) {
 		dev_err(&pdev->dev, "Failed to map the registers\n");
 		return PTR_ERR(base);
-- 
2.7.4



^ permalink raw reply related

* [PATCH -next 34/34] ASoC: sun8i-codec: use devm_platform_ioremap_resource() to simplify code
From: YueHaibing @ 2019-07-27 15:07 UTC (permalink / raw)
  To: lgirdwood, broonie, perex, tiwai, eric, wahrenst, f.fainelli,
	rjui, sbranden, bcm-kernel-feedback-list, paul, timur,
	nicoleotsuka, Xiubo.Lee, festevam, shawnguo, s.hauer, kernel,
	linux-imx, matthias.bgg, jbrunet, khilman, daniel, haojian.zhuang,
	robert.jarzmik, baohua, olivier.moysan, arnaud.pouliquen,
	mcoquelin.stm32, alexandre.torgue, mripard, wens, thierry.reding,
	jonathanh, yamada.masahiro, michal.simek, jcmvbkbc
  Cc: linux-xtensa, alsa-devel, gregkh, YueHaibing, linux-kernel,
	linux-mediatek, linux-rpi-kernel, linux-tegra, linux-amlogic,
	linuxppc-dev, linux-stm32, linux-arm-kernel
In-Reply-To: <20190727150738.54764-1-yuehaibing@huawei.com>

Use devm_platform_ioremap_resource() to simplify the code a bit.
This is detected by coccinelle.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
---
 sound/soc/sunxi/sun8i-codec.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/sound/soc/sunxi/sun8i-codec.c b/sound/soc/sunxi/sun8i-codec.c
index 0e0e8eb..55798bc 100644
--- a/sound/soc/sunxi/sun8i-codec.c
+++ b/sound/soc/sunxi/sun8i-codec.c
@@ -533,7 +533,6 @@ static const struct regmap_config sun8i_codec_regmap_config = {
 
 static int sun8i_codec_probe(struct platform_device *pdev)
 {
-	struct resource *res_base;
 	struct sun8i_codec *scodec;
 	void __iomem *base;
 	int ret;
@@ -556,8 +555,7 @@ static int sun8i_codec_probe(struct platform_device *pdev)
 		return PTR_ERR(scodec->clk_bus);
 	}
 
-	res_base = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(&pdev->dev, res_base);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base)) {
 		dev_err(&pdev->dev, "Failed to map the registers\n");
 		return PTR_ERR(base);
-- 
2.7.4



^ permalink raw reply related

* [Bug 200055] WARNING: CPU: 0 PID: 1 at kernel/locking/lockdep.c:3214 .__lockdep_init_map+0x260/0x270
From: bugzilla-daemon @ 2019-07-27 22:47 UTC (permalink / raw)
  To: linuxppc-dev
In-Reply-To: <bug-200055-206035@https.bugzilla.kernel.org/>

https://bugzilla.kernel.org/show_bug.cgi?id=200055

--- Comment #19 from Erhard F. (erhard_f@mailbox.org) ---
Created attachment 284001
  --> https://bugzilla.kernel.org/attachment.cgi?id=284001&action=edit
bisect.log

At last... I got the offending commit. Funnily enough it is about silencing
lockdep warnings. ;)

9e607f72748ddc5620aeeb8d1f32f30c79b360b9 is the first bad commit
commit 9e607f72748ddc5620aeeb8d1f32f30c79b360b9
Author: Denis Kirjanov <kda@linux-powerpc.org>
Date:   Wed Sep 21 14:34:58 2016 +0300

    i2c_powermac: shut up lockdep warning

    That's unclear why lockdep shows the following warning but adding a
    lockdep class to struct pmac_i2c_bus solves it

    [   20.507795] ======================================================
    [   20.507796] [ INFO: possible circular locking dependency detected ]
    [   20.507800] 4.8.0-rc7-00037-gd2ffb01 #21 Not tainted
    [   20.507801] -------------------------------------------------------
    [   20.507803] swapper/0/1 is trying to acquire lock:
    [   20.507818]  (&bus->mutex){+.+.+.}, at: [<c000000000052830>]
.pmac_i2c_open+0x30/0x100
    [   20.507819]
    [   20.507819] but task is already holding lock:
    [   20.507829]  (&policy->rwsem){+.+.+.}, at: [<c00000000068adcc>]
.cpufreq_online+0x1ac/0x9d0
    [   20.507830]
    [   20.507830] which lock already depends on the new lock.
    [   20.507830]
    [   20.507832]
    [   20.507832] the existing dependency chain (in reverse order) is:
    [   20.507837]
    [   20.507837] -> #4 (&policy->rwsem){+.+.+.}:
    [   20.507844]        [<c00000000082385c>] .down_write+0x6c/0x110
    [   20.507849]        [<c00000000068adcc>] .cpufreq_online+0x1ac/0x9d0
    [   20.507855]        [<c0000000004d76d8>]
.subsys_interface_register+0xb8/0x110
    [   20.507860]        [<c000000000689bb0>]
.cpufreq_register_driver+0x1d0/0x250
    [   20.507866]        [<c000000000b4f8f4>] .g5_cpufreq_init+0x9cc/0xa28
    [   20.507872]        [<c00000000000a98c>] .do_one_initcall+0x5c/0x1d0
    [   20.507878]        [<c000000000b0f86c>]
.kernel_init_freeable+0x1ac/0x28c
    [   20.507883]        [<c00000000000b3bc>] .kernel_init+0x1c/0x140
    [   20.507887]        [<c0000000000098f4>]
.ret_from_kernel_thread+0x58/0x64
    [   20.507894]
    [   20.507894] -> #3 (subsys mutex#2){+.+.+.}:
    [   20.507899]        [<c000000000820448>] .mutex_lock_nested+0xa8/0x590
    [   20.507903]        [<c0000000004d7f24>] .bus_probe_device+0x44/0xe0
    [   20.507907]        [<c0000000004d5208>] .device_add+0x508/0x730
    [   20.507911]        [<c0000000004dd528>] .register_cpu+0x118/0x190
    [   20.507916]        [<c000000000b14450>] .topology_init+0x148/0x248
    [   20.507921]        [<c00000000000a98c>] .do_one_initcall+0x5c/0x1d0
    [   20.507925]        [<c000000000b0f86c>]
.kernel_init_freeable+0x1ac/0x28c
    [   20.507929]        [<c00000000000b3bc>] .kernel_init+0x1c/0x140
    [   20.507934]        [<c0000000000098f4>]
.ret_from_kernel_thread+0x58/0x64
    [   20.507939]
    [   20.507939] -> #2 (cpu_add_remove_lock){+.+.+.}:
    [   20.507944]        [<c000000000820448>] .mutex_lock_nested+0xa8/0x590
    [   20.507950]        [<c000000000087a9c>] .register_cpu_notifier+0x2c/0x70
    [   20.507955]        [<c000000000b267e0>] .spawn_ksoftirqd+0x18/0x4c
    [   20.507959]        [<c00000000000a98c>] .do_one_initcall+0x5c/0x1d0
    [   20.507964]        [<c000000000b0f770>] .kernel_init_freeable+0xb0/0x28c
    [   20.507968]        [<c00000000000b3bc>] .kernel_init+0x1c/0x140
    [   20.507972]        [<c0000000000098f4>]
.ret_from_kernel_thread+0x58/0x64
    [   20.507978]
    [   20.507978] -> #1 (&host->mutex){+.+.+.}:
    [   20.507982]        [<c000000000820448>] .mutex_lock_nested+0xa8/0x590
    [   20.507987]        [<c0000000000527e8>] .kw_i2c_open+0x18/0x30
    [   20.507991]        [<c000000000052894>] .pmac_i2c_open+0x94/0x100
    [   20.507995]        [<c000000000b220a0>] .smp_core99_probe+0x260/0x410
    [   20.507999]        [<c000000000b185bc>] .smp_prepare_cpus+0x280/0x2ac
    [   20.508003]        [<c000000000b0f748>] .kernel_init_freeable+0x88/0x28c
    [   20.508008]        [<c00000000000b3bc>] .kernel_init+0x1c/0x140
    [   20.508012]        [<c0000000000098f4>]
.ret_from_kernel_thread+0x58/0x64
    [   20.508018]
    [   20.508018] -> #0 (&bus->mutex){+.+.+.}:
    [   20.508023]        [<c0000000000ed5b4>] .lock_acquire+0x84/0x100
    [   20.508027]        [<c000000000820448>] .mutex_lock_nested+0xa8/0x590
    [   20.508032]        [<c000000000052830>] .pmac_i2c_open+0x30/0x100
    [   20.508037]        [<c000000000052e14>] .pmac_i2c_do_begin+0x34/0x120
    [   20.508040]        [<c000000000056bc0>] .pmf_call_one+0x50/0xd0
    [   20.508045]        [<c00000000068ff1c>] .g5_pfunc_switch_volt+0x2c/0xc0
    [   20.508050]        [<c00000000068fecc>]
.g5_pfunc_switch_freq+0x1cc/0x1f0
    [   20.508054]        [<c00000000068fc2c>] .g5_cpufreq_target+0x2c/0x40
    [   20.508058]        [<c0000000006873ec>]
.__cpufreq_driver_target+0x23c/0x840
    [   20.508062]        [<c00000000068c798>]
.cpufreq_gov_performance_limits+0x18/0x30
    [   20.508067]        [<c00000000068915c>]
.cpufreq_start_governor+0xac/0x100
    [   20.508071]        [<c00000000068a788>] .cpufreq_set_policy+0x208/0x260
    [   20.508076]        [<c00000000068abdc>] .cpufreq_init_policy+0x6c/0xb0
    [   20.508081]        [<c00000000068ae70>] .cpufreq_online+0x250/0x9d0
    [   20.508085]        [<c0000000004d76d8>]
.subsys_interface_register+0xb8/0x110
    [   20.508090]        [<c000000000689bb0>]
.cpufreq_register_driver+0x1d0/0x250
    [   20.508094]        [<c000000000b4f8f4>] .g5_cpufreq_init+0x9cc/0xa28
    [   20.508099]        [<c00000000000a98c>] .do_one_initcall+0x5c/0x1d0
    [   20.508103]        [<c000000000b0f86c>]
.kernel_init_freeable+0x1ac/0x28c
    [   20.508107]        [<c00000000000b3bc>] .kernel_init+0x1c/0x140
    [   20.508112]        [<c0000000000098f4>]
.ret_from_kernel_thread+0x58/0x64
    [   20.508113]
    [   20.508113] other info that might help us debug this:
    [   20.508113]
    [   20.508121] Chain exists of:
    [   20.508121]   &bus->mutex --> subsys mutex#2 --> &policy->rwsem
    [   20.508121]
    [   20.508123]  Possible unsafe locking scenario:
    [   20.508123]
    [   20.508124]        CPU0                    CPU1
    [   20.508125]        ----                    ----
    [   20.508128]   lock(&policy->rwsem);
    [   20.508132]                                lock(subsys mutex#2);
    [   20.508135]                                lock(&policy->rwsem);
    [   20.508138]   lock(&bus->mutex);
    [   20.508139]
    [   20.508139]  *** DEADLOCK ***
    [   20.508139]
    [   20.508141] 3 locks held by swapper/0/1:
    [   20.508150]  #0:  (cpu_hotplug.lock){++++++}, at: [<c000000000087838>]
.get_online_cpus+0x48/0xc0
    [   20.508159]  #1:  (subsys mutex#2){+.+.+.}, at: [<c0000000004d7670>]
.subsys_interface_register+0x50/0x110
    [   20.508168]  #2:  (&policy->rwsem){+.+.+.}, at: [<c00000000068adcc>]
.cpufreq_online+0x1ac/0x9d0
    [   20.508169]
    [   20.508169] stack backtrace:
    [   20.508173] CPU: 0 PID: 1 Comm: swapper/0 Not tainted
4.8.0-rc7-00037-gd2ffb01 #21
    [   20.508175] Call Trace:
    [   20.508180] [c0000000790c2b90] [c00000000082cc70] .dump_stack+0xe0/0x14c
(unreliable)
    [   20.508184] [c0000000790c2c20] [c000000000828c88]
.print_circular_bug+0x350/0x388
    [   20.508188] [c0000000790c2cd0] [c0000000000ecb0c]
.__lock_acquire+0x196c/0x1d30
    [   20.508192] [c0000000790c2e50] [c0000000000ed5b4]
.lock_acquire+0x84/0x100
    [   20.508196] [c0000000790c2f20] [c000000000820448]
.mutex_lock_nested+0xa8/0x590
    [   20.508201] [c0000000790c3030] [c000000000052830]
.pmac_i2c_open+0x30/0x100
    [   20.508206] [c0000000790c30c0] [c000000000052e14]
.pmac_i2c_do_begin+0x34/0x120
    [   20.508209] [c0000000790c3150] [c000000000056bc0]
.pmf_call_one+0x50/0xd0
    [   20.508213] [c0000000790c31e0] [c00000000068ff1c]
.g5_pfunc_switch_volt+0x2c/0xc0
    [   20.508217] [c0000000790c3250] [c00000000068fecc]
.g5_pfunc_switch_freq+0x1cc/0x1f0
    [   20.508221] [c0000000790c3320] [c00000000068fc2c]
.g5_cpufreq_target+0x2c/0x40
    [   20.508226] [c0000000790c3390] [c0000000006873ec]
.__cpufreq_driver_target+0x23c/0x840
    [   20.508230] [c0000000790c3440] [c00000000068c798]
.cpufreq_gov_performance_limits+0x18/0x30
    [   20.508235] [c0000000790c34b0] [c00000000068915c]
.cpufreq_start_governor+0xac/0x100
    [   20.508239] [c0000000790c3530] [c00000000068a788]
.cpufreq_set_policy+0x208/0x260
    [   20.508244] [c0000000790c35d0] [c00000000068abdc]
.cpufreq_init_policy+0x6c/0xb0
    [   20.508249] [c0000000790c3940] [c00000000068ae70]
.cpufreq_online+0x250/0x9d0
    [   20.508253] [c0000000790c3a30] [c0000000004d76d8]
.subsys_interface_register+0xb8/0x110
    [   20.508258] [c0000000790c3ad0] [c000000000689bb0]
.cpufreq_register_driver+0x1d0/0x250
    [   20.508262] [c0000000790c3b60] [c000000000b4f8f4]
.g5_cpufreq_init+0x9cc/0xa28
    [   20.508267] [c0000000790c3c20] [c00000000000a98c]
.do_one_initcall+0x5c/0x1d0
    [   20.508271] [c0000000790c3d00] [c000000000b0f86c]
.kernel_init_freeable+0x1ac/0x28c
    [   20.508276] [c0000000790c3db0] [c00000000000b3bc]
.kernel_init+0x1c/0x140
    [   20.508280] [c0000000790c3e30] [c0000000000098f4]
.ret_from_kernel_thread+0x58/0x64

    Signed-off-by: Denis Kirjanov <kda@linux-powerpc.org>
    Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>

:040000 040000 18ec5f23dbd03c624aae71f3cbb05466dd38bf91
d4d7866e7cf3060e9ae1c76ad390194d4d789dcd M      arch

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

^ permalink raw reply

* [Bug 200055] WARNING: CPU: 0 PID: 1 at kernel/locking/lockdep.c:3214 .__lockdep_init_map+0x260/0x270
From: bugzilla-daemon @ 2019-07-27 22:48 UTC (permalink / raw)
  To: linuxppc-dev
In-Reply-To: <bug-200055-206035@https.bugzilla.kernel.org/>

https://bugzilla.kernel.org/show_bug.cgi?id=200055

Erhard F. (erhard_f@mailbox.org) changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #283679|0                           |1
        is obsolete|                            |
 Attachment #283681|0                           |1
        is obsolete|                            |

--- Comment #20 from Erhard F. (erhard_f@mailbox.org) ---
Created attachment 284003
  --> https://bugzilla.kernel.org/attachment.cgi?id=284003&action=edit
4.9.0-rc5+ kernel .config (G5 11,2, final bisect)

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

^ permalink raw reply


This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox