linux-clk.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource
@ 2019-12-09 19:57 Yangtao Li
  2019-12-09 19:57 ` [PATCH 02/17] clk: qcom: " Yangtao Li
                   ` (16 more replies)
  0 siblings, 17 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/sunxi-ng/ccu-sun50i-a64.c    | 4 +---
 drivers/clk/sunxi-ng/ccu-sun50i-h6.c     | 4 +---
 drivers/clk/sunxi-ng/ccu-sun8i-a83t.c    | 4 +---
 drivers/clk/sunxi-ng/ccu-sun8i-de2.c     | 4 +---
 drivers/clk/sunxi-ng/ccu-sun8i-r40.c     | 4 +---
 drivers/clk/sunxi-ng/ccu-sun9i-a80-de.c  | 4 +---
 drivers/clk/sunxi-ng/ccu-sun9i-a80-usb.c | 4 +---
 drivers/clk/sunxi-ng/ccu-sun9i-a80.c     | 4 +---
 drivers/clk/sunxi/clk-mod0.c             | 4 +---
 drivers/clk/sunxi/clk-sun6i-apb0-gates.c | 4 +---
 drivers/clk/sunxi/clk-sun6i-apb0.c       | 4 +---
 drivers/clk/sunxi/clk-sun6i-ar100.c      | 4 +---
 drivers/clk/sunxi/clk-sun8i-apb0.c       | 4 +---
 13 files changed, 13 insertions(+), 39 deletions(-)

diff --git a/drivers/clk/sunxi-ng/ccu-sun50i-a64.c b/drivers/clk/sunxi-ng/ccu-sun50i-a64.c
index 49bd7a4c015c..087880b20c15 100644
--- a/drivers/clk/sunxi-ng/ccu-sun50i-a64.c
+++ b/drivers/clk/sunxi-ng/ccu-sun50i-a64.c
@@ -923,12 +923,10 @@ static const struct sunxi_ccu_desc sun50i_a64_ccu_desc = {
 
 static int sun50i_a64_ccu_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	void __iomem *reg;
 	u32 val;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun50i-h6.c b/drivers/clk/sunxi-ng/ccu-sun50i-h6.c
index f2497d0a4683..00fa8ff2f51b 100644
--- a/drivers/clk/sunxi-ng/ccu-sun50i-h6.c
+++ b/drivers/clk/sunxi-ng/ccu-sun50i-h6.c
@@ -1183,13 +1183,11 @@ static const u32 usb2_clk_regs[] = {
 
 static int sun50i_h6_ccu_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	void __iomem *reg;
 	u32 val;
 	int i;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-a83t.c b/drivers/clk/sunxi-ng/ccu-sun8i-a83t.c
index 2b434521c5cc..b2aaa99ad23f 100644
--- a/drivers/clk/sunxi-ng/ccu-sun8i-a83t.c
+++ b/drivers/clk/sunxi-ng/ccu-sun8i-a83t.c
@@ -887,12 +887,10 @@ static void sun8i_a83t_cpu_pll_fixup(void __iomem *reg)
 
 static int sun8i_a83t_ccu_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	void __iomem *reg;
 	u32 val;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c
index d9668493c3f9..f48f61297551 100644
--- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c
+++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c
@@ -251,7 +251,6 @@ static const struct sunxi_ccu_desc sun8i_v3s_de2_clk_desc = {
 
 static int sunxi_de2_clk_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct clk *bus_clk, *mod_clk;
 	struct reset_control *rstc;
 	void __iomem *reg;
@@ -262,8 +261,7 @@ static int sunxi_de2_clk_probe(struct platform_device *pdev)
 	if (!ccu_desc)
 		return -EINVAL;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-r40.c b/drivers/clk/sunxi-ng/ccu-sun8i-r40.c
index 897490800102..b456f85fd8da 100644
--- a/drivers/clk/sunxi-ng/ccu-sun8i-r40.c
+++ b/drivers/clk/sunxi-ng/ccu-sun8i-r40.c
@@ -1294,14 +1294,12 @@ static struct regmap_config sun8i_r40_ccu_regmap_config = {
 
 static int sun8i_r40_ccu_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct regmap *regmap;
 	void __iomem *reg;
 	u32 val;
 	int ret;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun9i-a80-de.c b/drivers/clk/sunxi-ng/ccu-sun9i-a80-de.c
index 6616e8114f62..94552199d54d 100644
--- a/drivers/clk/sunxi-ng/ccu-sun9i-a80-de.c
+++ b/drivers/clk/sunxi-ng/ccu-sun9i-a80-de.c
@@ -203,14 +203,12 @@ static const struct sunxi_ccu_desc sun9i_a80_de_clk_desc = {
 
 static int sun9i_a80_de_clk_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct clk *bus_clk;
 	struct reset_control *rstc;
 	void __iomem *reg;
 	int ret;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun9i-a80-usb.c b/drivers/clk/sunxi-ng/ccu-sun9i-a80-usb.c
index 4b4a507d04ed..adac159a5f63 100644
--- a/drivers/clk/sunxi-ng/ccu-sun9i-a80-usb.c
+++ b/drivers/clk/sunxi-ng/ccu-sun9i-a80-usb.c
@@ -92,13 +92,11 @@ static const struct sunxi_ccu_desc sun9i_a80_usb_clk_desc = {
 
 static int sun9i_a80_usb_clk_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct clk *bus_clk;
 	void __iomem *reg;
 	int ret;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi-ng/ccu-sun9i-a80.c b/drivers/clk/sunxi-ng/ccu-sun9i-a80.c
index dcac1391767f..8295667e2252 100644
--- a/drivers/clk/sunxi-ng/ccu-sun9i-a80.c
+++ b/drivers/clk/sunxi-ng/ccu-sun9i-a80.c
@@ -1213,12 +1213,10 @@ static void sun9i_a80_cpu_pll_fixup(void __iomem *reg)
 
 static int sun9i_a80_ccu_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	void __iomem *reg;
 	u32 val;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi/clk-mod0.c b/drivers/clk/sunxi/clk-mod0.c
index 0cca91e075a5..cdb98937f584 100644
--- a/drivers/clk/sunxi/clk-mod0.c
+++ b/drivers/clk/sunxi/clk-mod0.c
@@ -88,14 +88,12 @@ CLK_OF_DECLARE_DRIVER(sun4i_a10_mod0, "allwinner,sun4i-a10-mod0-clk",
 static int sun4i_a10_mod0_clk_probe(struct platform_device *pdev)
 {
 	struct device_node *np = pdev->dev.of_node;
-	struct resource *r;
 	void __iomem *reg;
 
 	if (!np)
 		return -ENODEV;
 
-	r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, r);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi/clk-sun6i-apb0-gates.c b/drivers/clk/sunxi/clk-sun6i-apb0-gates.c
index a165e7172346..abae2813c59e 100644
--- a/drivers/clk/sunxi/clk-sun6i-apb0-gates.c
+++ b/drivers/clk/sunxi/clk-sun6i-apb0-gates.c
@@ -41,7 +41,6 @@ static int sun6i_a31_apb0_gates_clk_probe(struct platform_device *pdev)
 	const struct gates_data *data;
 	const char *clk_parent;
 	const char *clk_name;
-	struct resource *r;
 	void __iomem *reg;
 	int ngates;
 	int i;
@@ -55,8 +54,7 @@ static int sun6i_a31_apb0_gates_clk_probe(struct platform_device *pdev)
 		return -ENODEV;
 	data = device->data;
 
-	r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, r);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi/clk-sun6i-apb0.c b/drivers/clk/sunxi/clk-sun6i-apb0.c
index 10f70c35c265..f80c67bafe38 100644
--- a/drivers/clk/sunxi/clk-sun6i-apb0.c
+++ b/drivers/clk/sunxi/clk-sun6i-apb0.c
@@ -32,12 +32,10 @@ static int sun6i_a31_apb0_clk_probe(struct platform_device *pdev)
 	struct device_node *np = pdev->dev.of_node;
 	const char *clk_name = np->name;
 	const char *clk_parent;
-	struct resource *r;
 	void __iomem *reg;
 	struct clk *clk;
 
-	r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, r);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi/clk-sun6i-ar100.c b/drivers/clk/sunxi/clk-sun6i-ar100.c
index e1b7d0929cf7..ffc297732f8f 100644
--- a/drivers/clk/sunxi/clk-sun6i-ar100.c
+++ b/drivers/clk/sunxi/clk-sun6i-ar100.c
@@ -71,12 +71,10 @@ static DEFINE_SPINLOCK(sun6i_ar100_lock);
 static int sun6i_a31_ar100_clk_probe(struct platform_device *pdev)
 {
 	struct device_node *np = pdev->dev.of_node;
-	struct resource *r;
 	void __iomem *reg;
 	struct clk *clk;
 
-	r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, r);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
diff --git a/drivers/clk/sunxi/clk-sun8i-apb0.c b/drivers/clk/sunxi/clk-sun8i-apb0.c
index fc5d6e3b77d1..f605ecca879f 100644
--- a/drivers/clk/sunxi/clk-sun8i-apb0.c
+++ b/drivers/clk/sunxi/clk-sun8i-apb0.c
@@ -87,12 +87,10 @@ CLK_OF_DECLARE_DRIVER(sun8i_a23_apb0, "allwinner,sun8i-a23-apb0-clk",
 static int sun8i_a23_apb0_clk_probe(struct platform_device *pdev)
 {
 	struct device_node *np = pdev->dev.of_node;
-	struct resource *r;
 	void __iomem *reg;
 	struct clk *clk;
 
-	r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(&pdev->dev, r);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 02/17] clk: qcom: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 03/17] clk: samsung: " Yangtao Li
                   ` (15 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/qcom/a53-pll.c  | 4 +---
 drivers/clk/qcom/common.c   | 8 ++------
 drivers/clk/qcom/hfpll.c    | 4 +---
 drivers/clk/qcom/kpss-xcc.c | 4 +---
 4 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/drivers/clk/qcom/a53-pll.c b/drivers/clk/qcom/a53-pll.c
index 45cfc57bff92..42d14b6861d4 100644
--- a/drivers/clk/qcom/a53-pll.c
+++ b/drivers/clk/qcom/a53-pll.c
@@ -38,7 +38,6 @@ static int qcom_a53pll_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
 	struct regmap *regmap;
-	struct resource *res;
 	struct clk_pll *pll;
 	void __iomem *base;
 	struct clk_init_data init = { };
@@ -48,8 +47,7 @@ static int qcom_a53pll_probe(struct platform_device *pdev)
 	if (!pll)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
diff --git a/drivers/clk/qcom/common.c b/drivers/clk/qcom/common.c
index 60d2a78d1395..0932e019dd12 100644
--- a/drivers/clk/qcom/common.c
+++ b/drivers/clk/qcom/common.c
@@ -73,11 +73,9 @@ struct regmap *
 qcom_cc_map(struct platform_device *pdev, const struct qcom_cc_desc *desc)
 {
 	void __iomem *base;
-	struct resource *res;
 	struct device *dev = &pdev->dev;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return ERR_CAST(base);
 
@@ -313,11 +311,9 @@ int qcom_cc_probe_by_index(struct platform_device *pdev, int index,
 			   const struct qcom_cc_desc *desc)
 {
 	struct regmap *regmap;
-	struct resource *res;
 	void __iomem *base;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, index);
-	base = devm_ioremap_resource(&pdev->dev, res);
+	base = devm_platform_ioremap_resource(pdev, index);
 	if (IS_ERR(base))
 		return -ENOMEM;
 
diff --git a/drivers/clk/qcom/hfpll.c b/drivers/clk/qcom/hfpll.c
index a6de7101430c..de7df848dcfb 100644
--- a/drivers/clk/qcom/hfpll.c
+++ b/drivers/clk/qcom/hfpll.c
@@ -47,7 +47,6 @@ static const struct regmap_config hfpll_regmap_config = {
 
 static int qcom_hfpll_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct device *dev = &pdev->dev;
 	void __iomem *base;
 	struct regmap *regmap;
@@ -62,8 +61,7 @@ static int qcom_hfpll_probe(struct platform_device *pdev)
 	if (!h)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
diff --git a/drivers/clk/qcom/kpss-xcc.c b/drivers/clk/qcom/kpss-xcc.c
index 8590b5edd19d..4fec1f9142b8 100644
--- a/drivers/clk/qcom/kpss-xcc.c
+++ b/drivers/clk/qcom/kpss-xcc.c
@@ -33,7 +33,6 @@ static int kpss_xcc_driver_probe(struct platform_device *pdev)
 {
 	const struct of_device_id *id;
 	struct clk *clk;
-	struct resource *res;
 	void __iomem *base;
 	const char *name;
 
@@ -41,8 +40,7 @@ static int kpss_xcc_driver_probe(struct platform_device *pdev)
 	if (!id)
 		return -ENODEV;
 
-	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.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 03/17] clk: samsung: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
  2019-12-09 19:57 ` [PATCH 02/17] clk: qcom: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-10  2:10   ` Chanwoo Choi
  2019-12-09 19:57 ` [PATCH 04/17] clk: mediatek: " Yangtao Li
                   ` (14 subsequent siblings)
  16 siblings, 1 reply; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/samsung/clk-exynos-audss.c   | 4 +---
 drivers/clk/samsung/clk-exynos4412-isp.c | 4 +---
 drivers/clk/samsung/clk-exynos5433.c     | 4 +---
 drivers/clk/samsung/clk-s5pv210-audss.c  | 4 +---
 4 files changed, 4 insertions(+), 12 deletions(-)

diff --git a/drivers/clk/samsung/clk-exynos-audss.c b/drivers/clk/samsung/clk-exynos-audss.c
index 42b5d32c6cc7..9cc127a162ad 100644
--- a/drivers/clk/samsung/clk-exynos-audss.c
+++ b/drivers/clk/samsung/clk-exynos-audss.c
@@ -129,7 +129,6 @@ static int exynos_audss_clk_probe(struct platform_device *pdev)
 	struct clk *pll_ref, *pll_in, *cdclk, *sclk_audio, *sclk_pcm_in;
 	const struct exynos_audss_clk_drvdata *variant;
 	struct clk_hw **clk_table;
-	struct resource *res;
 	struct device *dev = &pdev->dev;
 	int i, ret = 0;
 
@@ -137,8 +136,7 @@ static int exynos_audss_clk_probe(struct platform_device *pdev)
 	if (!variant)
 		return -EINVAL;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg_base = devm_ioremap_resource(dev, res);
+	reg_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg_base))
 		return PTR_ERR(reg_base);
 
diff --git a/drivers/clk/samsung/clk-exynos4412-isp.c b/drivers/clk/samsung/clk-exynos4412-isp.c
index 4b9e73608c21..20f5129f7212 100644
--- a/drivers/clk/samsung/clk-exynos4412-isp.c
+++ b/drivers/clk/samsung/clk-exynos4412-isp.c
@@ -110,11 +110,9 @@ static int __init exynos4x12_isp_clk_probe(struct platform_device *pdev)
 	struct samsung_clk_provider *ctx;
 	struct device *dev = &pdev->dev;
 	struct device_node *np = dev->of_node;
-	struct resource *res;
 	void __iomem *reg_base;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg_base = devm_ioremap_resource(dev, res);
+	reg_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg_base)) {
 		dev_err(dev, "failed to map registers\n");
 		return PTR_ERR(reg_base);
diff --git a/drivers/clk/samsung/clk-exynos5433.c b/drivers/clk/samsung/clk-exynos5433.c
index 4b1aa9382ad2..b0d48cddfd6e 100644
--- a/drivers/clk/samsung/clk-exynos5433.c
+++ b/drivers/clk/samsung/clk-exynos5433.c
@@ -5557,7 +5557,6 @@ static int __init exynos5433_cmu_probe(struct platform_device *pdev)
 	struct exynos5433_cmu_data *data;
 	struct samsung_clk_provider *ctx;
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	void __iomem *reg_base;
 	int i;
 
@@ -5570,8 +5569,7 @@ static int __init exynos5433_cmu_probe(struct platform_device *pdev)
 		return -ENOMEM;
 	ctx = &data->ctx;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg_base = devm_ioremap_resource(dev, res);
+	reg_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg_base))
 		return PTR_ERR(reg_base);
 
diff --git a/drivers/clk/samsung/clk-s5pv210-audss.c b/drivers/clk/samsung/clk-s5pv210-audss.c
index 14985ebd043b..503bd8a10d8f 100644
--- a/drivers/clk/samsung/clk-s5pv210-audss.c
+++ b/drivers/clk/samsung/clk-s5pv210-audss.c
@@ -63,15 +63,13 @@ static struct syscore_ops s5pv210_audss_clk_syscore_ops = {
 static int s5pv210_audss_clk_probe(struct platform_device *pdev)
 {
 	int i, ret = 0;
-	struct resource *res;
 	const char *mout_audss_p[2];
 	const char *mout_i2s_p[3];
 	const char *hclk_p;
 	struct clk_hw **clk_table;
 	struct clk *hclk, *pll_ref, *pll_in, *cdclk, *sclk_audio;
 
-	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)) {
 		dev_err(&pdev->dev, "failed to map audss registers\n");
 		return PTR_ERR(reg_base);
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 04/17] clk: mediatek: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
  2019-12-09 19:57 ` [PATCH 02/17] clk: qcom: " Yangtao Li
  2019-12-09 19:57 ` [PATCH 03/17] clk: samsung: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 05/17] clk: hisilicon: " Yangtao Li
                   ` (13 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/mediatek/clk-mt2701.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/clk/mediatek/clk-mt2701.c b/drivers/clk/mediatek/clk-mt2701.c
index 695be0f77427..a6aea6646118 100644
--- a/drivers/clk/mediatek/clk-mt2701.c
+++ b/drivers/clk/mediatek/clk-mt2701.c
@@ -668,9 +668,8 @@ static int mtk_topckgen_init(struct platform_device *pdev)
 	struct clk_onecell_data *clk_data;
 	void __iomem *base;
 	struct device_node *node = pdev->dev.of_node;
-	struct resource *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);
 
@@ -889,9 +888,8 @@ static int mtk_pericfg_init(struct platform_device *pdev)
 	void __iomem *base;
 	int r;
 	struct device_node *node = pdev->dev.of_node;
-	struct resource *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.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 05/17] clk: hisilicon: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (2 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 04/17] clk: mediatek: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 06/17] clk: tegra: " Yangtao Li
                   ` (12 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/hisilicon/clk-hi3660-stub.c | 6 +-----
 drivers/clk/hisilicon/clk.c             | 7 +------
 2 files changed, 2 insertions(+), 11 deletions(-)

diff --git a/drivers/clk/hisilicon/clk-hi3660-stub.c b/drivers/clk/hisilicon/clk-hi3660-stub.c
index 3a653d54bee0..7e44d6e2a307 100644
--- a/drivers/clk/hisilicon/clk-hi3660-stub.c
+++ b/drivers/clk/hisilicon/clk-hi3660-stub.c
@@ -124,7 +124,6 @@ static struct clk_hw *hi3660_stub_clk_hw_get(struct of_phandle_args *clkspec,
 static int hi3660_stub_clk_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	unsigned int i;
 	int ret;
 
@@ -139,10 +138,7 @@ static int hi3660_stub_clk_probe(struct platform_device *pdev)
 	if (IS_ERR(stub_clk_chan.mbox))
 		return PTR_ERR(stub_clk_chan.mbox);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	if (!res)
-		return -EINVAL;
-	freq_reg = devm_ioremap(dev, res->start, resource_size(res));
+	freq_reg = devm_platform_ioremap_resource(pdev, 0);
 	if (!freq_reg)
 		return -ENOMEM;
 
diff --git a/drivers/clk/hisilicon/clk.c b/drivers/clk/hisilicon/clk.c
index 54d9fdc93599..8c521495d9e5 100644
--- a/drivers/clk/hisilicon/clk.c
+++ b/drivers/clk/hisilicon/clk.c
@@ -27,18 +27,13 @@ struct hisi_clock_data *hisi_clk_alloc(struct platform_device *pdev,
 						int nr_clks)
 {
 	struct hisi_clock_data *clk_data;
-	struct resource *res;
 	struct clk **clk_table;
 
 	clk_data = devm_kmalloc(&pdev->dev, sizeof(*clk_data), GFP_KERNEL);
 	if (!clk_data)
 		return NULL;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	if (!res)
-		return NULL;
-	clk_data->base = devm_ioremap(&pdev->dev,
-				res->start, resource_size(res));
+	clk_data->base = devm_platform_ioremap_resource(pdev, 0);
 	if (!clk_data->base)
 		return NULL;
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 06/17] clk: tegra: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (3 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 05/17] clk: hisilicon: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-11  9:49   ` Peter De Schrijver
  2019-12-09 19:57 ` [PATCH 07/17] clk: mvebu: " Yangtao Li
                   ` (11 subsequent siblings)
  16 siblings, 1 reply; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/tegra/clk-dfll.c | 34 ++++------------------------------
 1 file changed, 4 insertions(+), 30 deletions(-)

diff --git a/drivers/clk/tegra/clk-dfll.c b/drivers/clk/tegra/clk-dfll.c
index c051d92c2bbf..070a2957e119 100644
--- a/drivers/clk/tegra/clk-dfll.c
+++ b/drivers/clk/tegra/clk-dfll.c
@@ -1935,7 +1935,6 @@ static int dfll_fetch_common_params(struct tegra_dfll *td)
 int tegra_dfll_register(struct platform_device *pdev,
 			struct tegra_dfll_soc_data *soc)
 {
-	struct resource *mem;
 	struct tegra_dfll *td;
 	int ret;
 
@@ -1985,51 +1984,26 @@ int tegra_dfll_register(struct platform_device *pdev,
 		return ret;
 	}
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	if (!mem) {
-		dev_err(td->dev, "no control register resource\n");
-		return -ENODEV;
-	}
-
-	td->base = devm_ioremap(td->dev, mem->start, resource_size(mem));
+	td->base = devm_platform_ioremap_resource(pdev, 0);
 	if (!td->base) {
 		dev_err(td->dev, "couldn't ioremap DFLL control registers\n");
 		return -ENODEV;
 	}
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 1);
-	if (!mem) {
-		dev_err(td->dev, "no i2c_base resource\n");
-		return -ENODEV;
-	}
-
-	td->i2c_base = devm_ioremap(td->dev, mem->start, resource_size(mem));
+	td->i2c_base = devm_platform_ioremap_resource(pdev, 1);
 	if (!td->i2c_base) {
 		dev_err(td->dev, "couldn't ioremap i2c_base resource\n");
 		return -ENODEV;
 	}
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 2);
-	if (!mem) {
-		dev_err(td->dev, "no i2c_controller_base resource\n");
-		return -ENODEV;
-	}
-
-	td->i2c_controller_base = devm_ioremap(td->dev, mem->start,
-					       resource_size(mem));
+	td->i2c_controller_base = devm_platform_ioremap_resource(pdev, 2);
 	if (!td->i2c_controller_base) {
 		dev_err(td->dev,
 			"couldn't ioremap i2c_controller_base resource\n");
 		return -ENODEV;
 	}
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 3);
-	if (!mem) {
-		dev_err(td->dev, "no lut_base resource\n");
-		return -ENODEV;
-	}
-
-	td->lut_base = devm_ioremap(td->dev, mem->start, resource_size(mem));
+	td->lut_base = devm_platform_ioremap_resource(pdev, 3);
 	if (!td->lut_base) {
 		dev_err(td->dev,
 			"couldn't ioremap lut_base resource\n");
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 07/17] clk: mvebu: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (4 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 06/17] clk: tegra: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 08/17] clk: imx: " Yangtao Li
                   ` (10 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/mvebu/armada-37xx-periph.c | 4 +---
 drivers/clk/mvebu/armada-37xx-tbg.c    | 4 +---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/clk/mvebu/armada-37xx-periph.c b/drivers/clk/mvebu/armada-37xx-periph.c
index f5746f9ea929..0d03878920ac 100644
--- a/drivers/clk/mvebu/armada-37xx-periph.c
+++ b/drivers/clk/mvebu/armada-37xx-periph.c
@@ -725,7 +725,6 @@ static int armada_3700_periph_clock_probe(struct platform_device *pdev)
 	const struct clk_periph_data *data;
 	struct device *dev = &pdev->dev;
 	int num_periph = 0, i, ret;
-	struct resource *res;
 
 	data = of_device_get_match_data(dev);
 	if (!data)
@@ -746,8 +745,7 @@ static int armada_3700_periph_clock_probe(struct platform_device *pdev)
 		return -ENOMEM;
 	driver_data->hw_data->num = num_periph;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	driver_data->reg = devm_ioremap_resource(dev, res);
+	driver_data->reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(driver_data->reg))
 		return PTR_ERR(driver_data->reg);
 
diff --git a/drivers/clk/mvebu/armada-37xx-tbg.c b/drivers/clk/mvebu/armada-37xx-tbg.c
index 585a02e0b330..5d86912fbb6e 100644
--- a/drivers/clk/mvebu/armada-37xx-tbg.c
+++ b/drivers/clk/mvebu/armada-37xx-tbg.c
@@ -84,7 +84,6 @@ static int armada_3700_tbg_clock_probe(struct platform_device *pdev)
 	struct clk_hw_onecell_data *hw_tbg_data;
 	struct device *dev = &pdev->dev;
 	const char *parent_name;
-	struct resource *res;
 	struct clk *parent;
 	void __iomem *reg;
 	int i, ret;
@@ -105,8 +104,7 @@ static int armada_3700_tbg_clock_probe(struct platform_device *pdev)
 	parent_name = __clk_get_name(parent);
 	clk_put(parent);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	reg = devm_ioremap_resource(dev, res);
+	reg = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(reg))
 		return PTR_ERR(reg);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 08/17] clk: imx: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (5 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 07/17] clk: mvebu: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 09/17] clk: sifive: " Yangtao Li
                   ` (9 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/imx/clk-imx8qxp-lpcg.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/drivers/clk/imx/clk-imx8qxp-lpcg.c b/drivers/clk/imx/clk-imx8qxp-lpcg.c
index c0aff7ca6374..10ae712447c6 100644
--- a/drivers/clk/imx/clk-imx8qxp-lpcg.c
+++ b/drivers/clk/imx/clk-imx8qxp-lpcg.c
@@ -164,7 +164,6 @@ static int imx8qxp_lpcg_clk_probe(struct platform_device *pdev)
 	struct clk_hw_onecell_data *clk_data;
 	const struct imx8qxp_ss_lpcg *ss_lpcg;
 	const struct imx8qxp_lpcg_data *lpcg;
-	struct resource *res;
 	struct clk_hw **clks;
 	void __iomem *base;
 	int i;
@@ -173,10 +172,7 @@ static int imx8qxp_lpcg_clk_probe(struct platform_device *pdev)
 	if (!ss_lpcg)
 		return -ENODEV;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	if (!res)
-		return -EINVAL;
-	base = devm_ioremap(dev, res->start, resource_size(res));
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (!base)
 		return -ENOMEM;
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 09/17] clk: sifive: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (6 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 08/17] clk: imx: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 10/17] clk: axi-clkgen: " Yangtao Li
                   ` (8 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/sifive/fu540-prci.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/sifive/fu540-prci.c b/drivers/clk/sifive/fu540-prci.c
index 6282ee2f361c..ececee273d32 100644
--- a/drivers/clk/sifive/fu540-prci.c
+++ b/drivers/clk/sifive/fu540-prci.c
@@ -582,7 +582,6 @@ static int __prci_register_clocks(struct device *dev, struct __prci_data *pd)
 static int sifive_fu540_prci_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	struct __prci_data *pd;
 	int r;
 
@@ -590,8 +589,7 @@ static int sifive_fu540_prci_probe(struct platform_device *pdev)
 	if (!pd)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	pd->va = devm_ioremap_resource(dev, res);
+	pd->va = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(pd->va))
 		return PTR_ERR(pd->va);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 10/17] clk: axi-clkgen: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (7 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 09/17] clk: sifive: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 11/17] clk: milbeaut: " Yangtao Li
                   ` (7 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/clk-axi-clkgen.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/clk-axi-clkgen.c b/drivers/clk/clk-axi-clkgen.c
index 96f351785b41..8d0c32e81e78 100644
--- a/drivers/clk/clk-axi-clkgen.c
+++ b/drivers/clk/clk-axi-clkgen.c
@@ -412,7 +412,6 @@ static int axi_clkgen_probe(struct platform_device *pdev)
 	struct clk_init_data init;
 	const char *parent_names[2];
 	const char *clk_name;
-	struct resource *mem;
 	unsigned int i;
 	int ret;
 
@@ -427,8 +426,7 @@ static int axi_clkgen_probe(struct platform_device *pdev)
 	if (!axi_clkgen)
 		return -ENOMEM;
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	axi_clkgen->base = devm_ioremap_resource(&pdev->dev, mem);
+	axi_clkgen->base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(axi_clkgen->base))
 		return PTR_ERR(axi_clkgen->base);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 11/17] clk: milbeaut: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (8 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 10/17] clk: axi-clkgen: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 12/17] clk: socfpga: " Yangtao Li
                   ` (6 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/clk-milbeaut.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/clk-milbeaut.c b/drivers/clk/clk-milbeaut.c
index 80b9d78493bc..05f09fbfb80a 100644
--- a/drivers/clk/clk-milbeaut.c
+++ b/drivers/clk/clk-milbeaut.c
@@ -560,14 +560,12 @@ static void m10v_reg_mux_pre(const struct m10v_clk_mux_factors *factors,
 static int m10v_clk_probe(struct platform_device *pdev)
 {
 	int id;
-	struct resource *res;
 	struct device *dev = &pdev->dev;
 	struct device_node *np = dev->of_node;
 	void __iomem *base;
 	const char *parent_name;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 12/17] clk: socfpga: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (9 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 11/17] clk: milbeaut: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-16 20:20   ` Dinh Nguyen
  2019-12-09 19:57 ` [PATCH 13/17] clk: gemini: " Yangtao Li
                   ` (5 subsequent siblings)
  16 siblings, 1 reply; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/socfpga/clk-s10.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/socfpga/clk-s10.c b/drivers/clk/socfpga/clk-s10.c
index 993f3a73c71e..85055fc56404 100644
--- a/drivers/clk/socfpga/clk-s10.c
+++ b/drivers/clk/socfpga/clk-s10.c
@@ -276,11 +276,9 @@ static struct stratix10_clock_data *__socfpga_s10_clk_init(struct platform_devic
 	struct device *dev = &pdev->dev;
 	struct stratix10_clock_data *clk_data;
 	struct clk **clk_table;
-	struct resource *res;
 	void __iomem *base;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base)) {
 		pr_err("%s: failed to map clock registers\n", __func__);
 		return ERR_CAST(base);
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 13/17] clk: gemini: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (10 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 12/17] clk: socfpga: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 14/17] clk: axm5516: " Yangtao Li
                   ` (4 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/clk-gemini.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/clk-gemini.c b/drivers/clk/clk-gemini.c
index b51069e794ff..0184ff9bd206 100644
--- a/drivers/clk/clk-gemini.c
+++ b/drivers/clk/clk-gemini.c
@@ -276,7 +276,6 @@ static int gemini_clk_probe(struct platform_device *pdev)
 	struct device *dev = &pdev->dev;
 	struct device_node *np = dev->of_node;
 	unsigned int mult, div;
-	struct resource *res;
 	u32 val;
 	int ret;
 	int i;
@@ -286,8 +285,7 @@ static int gemini_clk_probe(struct platform_device *pdev)
 		return -ENOMEM;
 
 	/* Remap the system controller for the exclusive register */
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 14/17] clk: axm5516: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (11 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 13/17] clk: gemini: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 15/17] clk: bm1880: " Yangtao Li
                   ` (3 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/clk-axm5516.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/clk-axm5516.c b/drivers/clk/clk-axm5516.c
index 07e80fe8c310..3ab02b62b645 100644
--- a/drivers/clk/clk-axm5516.c
+++ b/drivers/clk/clk-axm5516.c
@@ -541,14 +541,12 @@ MODULE_DEVICE_TABLE(of, axmclk_match_table);
 static int axmclk_probe(struct platform_device *pdev)
 {
 	void __iomem *base;
-	struct resource *res;
 	int i, ret;
 	struct device *dev = &pdev->dev;
 	struct regmap *regmap;
 	size_t num_clks;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(dev, res);
+	base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(base))
 		return PTR_ERR(base);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 15/17] clk: bm1880: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (12 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 14/17] clk: axm5516: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 16/17] clk: actions: " Yangtao Li
                   ` (2 subsequent siblings)
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/clk-bm1880.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/clk/clk-bm1880.c b/drivers/clk/clk-bm1880.c
index 4cd175afce9b..9fa0ca1fe833 100644
--- a/drivers/clk/clk-bm1880.c
+++ b/drivers/clk/clk-bm1880.c
@@ -893,16 +893,13 @@ static int bm1880_clk_probe(struct platform_device *pdev)
 	struct bm1880_clock_data *clk_data;
 	void __iomem *pll_base, *sys_base;
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	int num_clks, i;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	pll_base = devm_ioremap_resource(&pdev->dev, res);
+	pll_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(pll_base))
 		return PTR_ERR(pll_base);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
-	sys_base = devm_ioremap_resource(&pdev->dev, res);
+	sys_base = devm_platform_ioremap_resource(pdev, 1);
 	if (IS_ERR(sys_base))
 		return PTR_ERR(sys_base);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 16/17] clk: actions: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (13 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 15/17] clk: bm1880: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2019-12-09 19:57 ` [PATCH 17/17] ARC: clk: " Yangtao Li
  2020-01-31  1:06 ` [PATCH 01/17] clk: sunxi: sunxi-ng: " Stephen Boyd
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/actions/owl-common.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/actions/owl-common.c b/drivers/clk/actions/owl-common.c
index 4de97cc7cb54..9e98e8a92ad7 100644
--- a/drivers/clk/actions/owl-common.c
+++ b/drivers/clk/actions/owl-common.c
@@ -43,10 +43,8 @@ int owl_clk_regmap_init(struct platform_device *pdev,
 {
 	void __iomem *base;
 	struct regmap *regmap;
-	struct resource *res;
 
-	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.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* [PATCH 17/17] ARC: clk: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (14 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 16/17] clk: actions: " Yangtao Li
@ 2019-12-09 19:57 ` Yangtao Li
  2020-01-31  1:06 ` [PATCH 01/17] clk: sunxi: sunxi-ng: " Stephen Boyd
  16 siblings, 0 replies; 21+ messages in thread
From: Yangtao Li @ 2019-12-09 19:57 UTC (permalink / raw)
  To: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, weiyongjun1, wangyan.wang,
	chunhui.dai, miquel.raynal, heiko, jcmvbkbc, nsekhar,
	geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
 drivers/clk/clk-hsdk-pll.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/clk/clk-hsdk-pll.c b/drivers/clk/clk-hsdk-pll.c
index 97d1e8c35b71..155f2af65361 100644
--- a/drivers/clk/clk-hsdk-pll.c
+++ b/drivers/clk/clk-hsdk-pll.c
@@ -299,7 +299,6 @@ static const struct clk_ops hsdk_pll_ops = {
 static int hsdk_pll_clk_probe(struct platform_device *pdev)
 {
 	int ret;
-	struct resource *mem;
 	const char *parent_name;
 	unsigned int num_parents;
 	struct hsdk_pll_clk *pll_clk;
@@ -310,8 +309,7 @@ static int hsdk_pll_clk_probe(struct platform_device *pdev)
 	if (!pll_clk)
 		return -ENOMEM;
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	pll_clk->regs = devm_ioremap_resource(dev, mem);
+	pll_clk->regs = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(pll_clk->regs))
 		return PTR_ERR(pll_clk->regs);
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 21+ messages in thread

* Re: [PATCH 03/17] clk: samsung: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 ` [PATCH 03/17] clk: samsung: " Yangtao Li
@ 2019-12-10  2:10   ` Chanwoo Choi
  0 siblings, 0 replies; 21+ messages in thread
From: Chanwoo Choi @ 2019-12-10  2:10 UTC (permalink / raw)
  To: Yangtao Li, afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pdeschrijver,
	pgaikwad, thierry.reding, jonathanh, matthias.bgg, rfontana,
	gregkh, t-kristo, john, tglx, allison, kstewart, swinslow,
	aisheng.dong, robh, daniel.baluta, wangyan.wang, chunhui.dai,
	miquel.raynal, heiko, jcmvbkbc, nsekhar, geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek

Hi,

On 12/10/19 4:57 AM, Yangtao Li wrote:
> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
>  drivers/clk/samsung/clk-exynos-audss.c   | 4 +---
>  drivers/clk/samsung/clk-exynos4412-isp.c | 4 +---
>  drivers/clk/samsung/clk-exynos5433.c     | 4 +---
>  drivers/clk/samsung/clk-s5pv210-audss.c  | 4 +---
>  4 files changed, 4 insertions(+), 12 deletions(-)
> 
> diff --git a/drivers/clk/samsung/clk-exynos-audss.c b/drivers/clk/samsung/clk-exynos-audss.c
> index 42b5d32c6cc7..9cc127a162ad 100644
> --- a/drivers/clk/samsung/clk-exynos-audss.c
> +++ b/drivers/clk/samsung/clk-exynos-audss.c
> @@ -129,7 +129,6 @@ static int exynos_audss_clk_probe(struct platform_device *pdev)
>  	struct clk *pll_ref, *pll_in, *cdclk, *sclk_audio, *sclk_pcm_in;
>  	const struct exynos_audss_clk_drvdata *variant;
>  	struct clk_hw **clk_table;
> -	struct resource *res;
>  	struct device *dev = &pdev->dev;
>  	int i, ret = 0;
>  
> @@ -137,8 +136,7 @@ static int exynos_audss_clk_probe(struct platform_device *pdev)
>  	if (!variant)
>  		return -EINVAL;
>  
> -	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	reg_base = devm_ioremap_resource(dev, res);
> +	reg_base = devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(reg_base))
>  		return PTR_ERR(reg_base);
>  
> diff --git a/drivers/clk/samsung/clk-exynos4412-isp.c b/drivers/clk/samsung/clk-exynos4412-isp.c
> index 4b9e73608c21..20f5129f7212 100644
> --- a/drivers/clk/samsung/clk-exynos4412-isp.c
> +++ b/drivers/clk/samsung/clk-exynos4412-isp.c
> @@ -110,11 +110,9 @@ static int __init exynos4x12_isp_clk_probe(struct platform_device *pdev)
>  	struct samsung_clk_provider *ctx;
>  	struct device *dev = &pdev->dev;
>  	struct device_node *np = dev->of_node;
> -	struct resource *res;
>  	void __iomem *reg_base;
>  
> -	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	reg_base = devm_ioremap_resource(dev, res);
> +	reg_base = devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(reg_base)) {
>  		dev_err(dev, "failed to map registers\n");
>  		return PTR_ERR(reg_base);
> diff --git a/drivers/clk/samsung/clk-exynos5433.c b/drivers/clk/samsung/clk-exynos5433.c
> index 4b1aa9382ad2..b0d48cddfd6e 100644
> --- a/drivers/clk/samsung/clk-exynos5433.c
> +++ b/drivers/clk/samsung/clk-exynos5433.c
> @@ -5557,7 +5557,6 @@ static int __init exynos5433_cmu_probe(struct platform_device *pdev)
>  	struct exynos5433_cmu_data *data;
>  	struct samsung_clk_provider *ctx;
>  	struct device *dev = &pdev->dev;
> -	struct resource *res;
>  	void __iomem *reg_base;
>  	int i;
>  
> @@ -5570,8 +5569,7 @@ static int __init exynos5433_cmu_probe(struct platform_device *pdev)
>  		return -ENOMEM;
>  	ctx = &data->ctx;
>  
> -	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	reg_base = devm_ioremap_resource(dev, res);
> +	reg_base = devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(reg_base))
>  		return PTR_ERR(reg_base);
>  
> diff --git a/drivers/clk/samsung/clk-s5pv210-audss.c b/drivers/clk/samsung/clk-s5pv210-audss.c
> index 14985ebd043b..503bd8a10d8f 100644
> --- a/drivers/clk/samsung/clk-s5pv210-audss.c
> +++ b/drivers/clk/samsung/clk-s5pv210-audss.c
> @@ -63,15 +63,13 @@ static struct syscore_ops s5pv210_audss_clk_syscore_ops = {
>  static int s5pv210_audss_clk_probe(struct platform_device *pdev)
>  {
>  	int i, ret = 0;
> -	struct resource *res;
>  	const char *mout_audss_p[2];
>  	const char *mout_i2s_p[3];
>  	const char *hclk_p;
>  	struct clk_hw **clk_table;
>  	struct clk *hclk, *pll_ref, *pll_in, *cdclk, *sclk_audio;
>  
> -	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)) {
>  		dev_err(&pdev->dev, "failed to map audss registers\n");
>  		return PTR_ERR(reg_base);
> 

Acked-by: Chanwoo Choi <cw00.choi@samsung.com>


-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 06/17] clk: tegra: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 ` [PATCH 06/17] clk: tegra: " Yangtao Li
@ 2019-12-11  9:49   ` Peter De Schrijver
  0 siblings, 0 replies; 21+ messages in thread
From: Peter De Schrijver @ 2019-12-11  9:49 UTC (permalink / raw)
  To: Yangtao Li
  Cc: afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, dinguyen, mripard, wens, emilio, pgaikwad,
	thierry.reding, jonathanh, matthias.bgg, rfontana, gregkh,
	t-kristo, john, tglx, allison, kstewart, swinslow, aisheng.dong,
	robh, daniel.baluta, weiyongjun1, wangyan.wang, chunhui.dai,
	miquel.raynal, heiko, jcmvbkbc, nsekhar, geert+renesas,
	linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek

On Mon, Dec 09, 2019 at 07:57:38PM +0000, Yangtao Li wrote:
> External email: Use caution opening links or attachments
> 
> 

Acked-by:  Peter De Schrijver <pdeschrijver@nvidia.com>

> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
>  drivers/clk/tegra/clk-dfll.c | 34 ++++------------------------------
>  1 file changed, 4 insertions(+), 30 deletions(-)
> 
> diff --git a/drivers/clk/tegra/clk-dfll.c b/drivers/clk/tegra/clk-dfll.c
> index c051d92c2bbf..070a2957e119 100644
> --- a/drivers/clk/tegra/clk-dfll.c
> +++ b/drivers/clk/tegra/clk-dfll.c
> @@ -1935,7 +1935,6 @@ static int dfll_fetch_common_params(struct tegra_dfll *td)
>  int tegra_dfll_register(struct platform_device *pdev,
>                         struct tegra_dfll_soc_data *soc)
>  {
> -       struct resource *mem;
>         struct tegra_dfll *td;
>         int ret;
> 
> @@ -1985,51 +1984,26 @@ int tegra_dfll_register(struct platform_device *pdev,
>                 return ret;
>         }
> 
> -       mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -       if (!mem) {
> -               dev_err(td->dev, "no control register resource\n");
> -               return -ENODEV;
> -       }
> -
> -       td->base = devm_ioremap(td->dev, mem->start, resource_size(mem));
> +       td->base = devm_platform_ioremap_resource(pdev, 0);
>         if (!td->base) {
>                 dev_err(td->dev, "couldn't ioremap DFLL control registers\n");
>                 return -ENODEV;
>         }
> 
> -       mem = platform_get_resource(pdev, IORESOURCE_MEM, 1);
> -       if (!mem) {
> -               dev_err(td->dev, "no i2c_base resource\n");
> -               return -ENODEV;
> -       }
> -
> -       td->i2c_base = devm_ioremap(td->dev, mem->start, resource_size(mem));
> +       td->i2c_base = devm_platform_ioremap_resource(pdev, 1);
>         if (!td->i2c_base) {
>                 dev_err(td->dev, "couldn't ioremap i2c_base resource\n");
>                 return -ENODEV;
>         }
> 
> -       mem = platform_get_resource(pdev, IORESOURCE_MEM, 2);
> -       if (!mem) {
> -               dev_err(td->dev, "no i2c_controller_base resource\n");
> -               return -ENODEV;
> -       }
> -
> -       td->i2c_controller_base = devm_ioremap(td->dev, mem->start,
> -                                              resource_size(mem));
> +       td->i2c_controller_base = devm_platform_ioremap_resource(pdev, 2);
>         if (!td->i2c_controller_base) {
>                 dev_err(td->dev,
>                         "couldn't ioremap i2c_controller_base resource\n");
>                 return -ENODEV;
>         }
> 
> -       mem = platform_get_resource(pdev, IORESOURCE_MEM, 3);
> -       if (!mem) {
> -               dev_err(td->dev, "no lut_base resource\n");
> -               return -ENODEV;
> -       }
> -
> -       td->lut_base = devm_ioremap(td->dev, mem->start, resource_size(mem));
> +       td->lut_base = devm_platform_ioremap_resource(pdev, 3);
>         if (!td->lut_base) {
>                 dev_err(td->dev,
>                         "couldn't ioremap lut_base resource\n");
> --
> 2.17.1
> 

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 12/17] clk: socfpga: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 ` [PATCH 12/17] clk: socfpga: " Yangtao Li
@ 2019-12-16 20:20   ` Dinh Nguyen
  0 siblings, 0 replies; 21+ messages in thread
From: Dinh Nguyen @ 2019-12-16 20:20 UTC (permalink / raw)
  To: Yangtao Li, afaerber, manivannan.sadhasivam, mturquette, sboyd,
	Eugeniy.Paltsev, shawnguo, s.hauer, kernel, festevam, linux-imx,
	agross, s.nawrocki, tomasz.figa, cw00.choi, kgene, krzk, palmer,
	paul.walmsley, mripard, wens, emilio, pdeschrijver, pgaikwad,
	thierry.reding, jonathanh, matthias.bgg, rfontana, gregkh,
	t-kristo, john, tglx, allison, kstewart, swinslow, aisheng.dong,
	robh, daniel.baluta, weiyongjun1, wangyan.wang, chunhui.dai,
	miquel.raynal, heiko, jcmvbkbc, nsekhar, geert+renesas
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek



On 12/9/19 1:57 PM, Yangtao Li wrote:
> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
>  drivers/clk/socfpga/clk-s10.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/clk/socfpga/clk-s10.c b/drivers/clk/socfpga/clk-s10.c
> index 993f3a73c71e..85055fc56404 100644
> --- a/drivers/clk/socfpga/clk-s10.c
> +++ b/drivers/clk/socfpga/clk-s10.c
> @@ -276,11 +276,9 @@ static struct stratix10_clock_data *__socfpga_s10_clk_init(struct platform_devic
>  	struct device *dev = &pdev->dev;
>  	struct stratix10_clock_data *clk_data;
>  	struct clk **clk_table;
> -	struct resource *res;
>  	void __iomem *base;
>  
> -	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	base = devm_ioremap_resource(dev, res);
> +	base = devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(base)) {
>  		pr_err("%s: failed to map clock registers\n", __func__);
>  		return ERR_CAST(base);
> 

Tested-by: Dinh Nguyen <dinguyen@kernel.org>

^ permalink raw reply	[flat|nested] 21+ messages in thread

* Re: [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource
  2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
                   ` (15 preceding siblings ...)
  2019-12-09 19:57 ` [PATCH 17/17] ARC: clk: " Yangtao Li
@ 2020-01-31  1:06 ` Stephen Boyd
  16 siblings, 0 replies; 21+ messages in thread
From: Stephen Boyd @ 2020-01-31  1:06 UTC (permalink / raw)
  To: Eugeniy.Paltsev, Yangtao Li, afaerber, agross, aisheng.dong,
	allison, chunhui.dai, cw00.choi, daniel.baluta, dinguyen, emilio,
	festevam, geert+renesas, gregkh, heiko, jcmvbkbc, john,
	jonathanh, kernel, kgene, krzk, kstewart, linux-imx,
	manivannan.sadhasivam, matthias.bgg, miquel.raynal, mripard,
	mturquette, nsekhar, palmer, paul.walmsley, pdeschrijver,
	pgaikwad, rfontana, robh, s.hauer, s.nawrocki, shawnguo,
	swinslow, t-kristo, tglx, thierry.reding, tomasz.figa,
	wangyan.wang, weiyongjun1, wens
  Cc: linux-arm-kernel, linux-clk, linux-kernel, linux-arm-msm,
	linux-samsung-soc, linux-riscv, linux-tegra, linux-mediatek,
	Yangtao Li

Quoting Yangtao Li (2019-12-09 11:57:33)
> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>

Please add a cover letter, resend this series picking up any acks and
dropping any patches that were rejected by maintainers of the drivers.
There was significant discussion on one patch that was rejected, so I'll
only pick up patches for the next cycle that are acked/reviewed-by
relevant folks.


^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2020-01-31  1:06 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-09 19:57 [PATCH 01/17] clk: sunxi: sunxi-ng: convert to devm_platform_ioremap_resource Yangtao Li
2019-12-09 19:57 ` [PATCH 02/17] clk: qcom: " Yangtao Li
2019-12-09 19:57 ` [PATCH 03/17] clk: samsung: " Yangtao Li
2019-12-10  2:10   ` Chanwoo Choi
2019-12-09 19:57 ` [PATCH 04/17] clk: mediatek: " Yangtao Li
2019-12-09 19:57 ` [PATCH 05/17] clk: hisilicon: " Yangtao Li
2019-12-09 19:57 ` [PATCH 06/17] clk: tegra: " Yangtao Li
2019-12-11  9:49   ` Peter De Schrijver
2019-12-09 19:57 ` [PATCH 07/17] clk: mvebu: " Yangtao Li
2019-12-09 19:57 ` [PATCH 08/17] clk: imx: " Yangtao Li
2019-12-09 19:57 ` [PATCH 09/17] clk: sifive: " Yangtao Li
2019-12-09 19:57 ` [PATCH 10/17] clk: axi-clkgen: " Yangtao Li
2019-12-09 19:57 ` [PATCH 11/17] clk: milbeaut: " Yangtao Li
2019-12-09 19:57 ` [PATCH 12/17] clk: socfpga: " Yangtao Li
2019-12-16 20:20   ` Dinh Nguyen
2019-12-09 19:57 ` [PATCH 13/17] clk: gemini: " Yangtao Li
2019-12-09 19:57 ` [PATCH 14/17] clk: axm5516: " Yangtao Li
2019-12-09 19:57 ` [PATCH 15/17] clk: bm1880: " Yangtao Li
2019-12-09 19:57 ` [PATCH 16/17] clk: actions: " Yangtao Li
2019-12-09 19:57 ` [PATCH 17/17] ARC: clk: " Yangtao Li
2020-01-31  1:06 ` [PATCH 01/17] clk: sunxi: sunxi-ng: " Stephen Boyd

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).