All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: linux-kernel@vger.kernel.org
Cc: Qiang Zhao <qiang.zhao@nxp.com>, Li Yang <leoyang.li@nxp.com>,
	Andy Gross <andy.gross@linaro.org>,
	David Brown <david.brown@linaro.org>,
	Heiko Stuebner <heiko@sntech.de>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Santosh Shilimkar <ssantosh@kernel.org>,
	linuxppc-dev@lists.ozlabs.org,
	linux-arm-kernel@lists.infradead.org, linux-soc@vger.kernel.org
Subject: [PATCH] soc: Convert to using %pOFn instead of device_node.name
Date: Mon, 27 Aug 2018 20:52:44 -0500	[thread overview]
Message-ID: <20180828015252.28511-43-robh@kernel.org> (raw)
In-Reply-To: <20180828015252.28511-1-robh@kernel.org>

In preparation to remove the node name pointer from struct device_node,
convert printf users to use the %pOFn format specifier.

Cc: Qiang Zhao <qiang.zhao@nxp.com>
Cc: Li Yang <leoyang.li@nxp.com>
Cc: Andy Gross <andy.gross@linaro.org>
Cc: David Brown <david.brown@linaro.org>
Cc: Heiko Stuebner <heiko@sntech.de>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Jonathan Hunter <jonathanh@nvidia.com>
Cc: Santosh Shilimkar <ssantosh@kernel.org>
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-soc@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 drivers/soc/dove/pmu.c            |  8 +++---
 drivers/soc/fsl/qe/qe_tdm.c       |  4 +--
 drivers/soc/qcom/apr.c            |  2 +-
 drivers/soc/rockchip/pm_domains.c | 44 +++++++++++++++----------------
 drivers/soc/tegra/pmc.c           | 12 ++++-----
 drivers/soc/ti/knav_dma.c         |  8 +++---
 drivers/soc/ti/knav_qmss_queue.c  |  8 +++---
 7 files changed, 43 insertions(+), 43 deletions(-)

diff --git a/drivers/soc/dove/pmu.c b/drivers/soc/dove/pmu.c
index 5abb08ffb74d..ffc5311c0ed8 100644
--- a/drivers/soc/dove/pmu.c
+++ b/drivers/soc/dove/pmu.c
@@ -383,7 +383,7 @@ int __init dove_init_pmu(void)
 
 	domains_node = of_get_child_by_name(np_pmu, "domains");
 	if (!domains_node) {
-		pr_err("%s: failed to find domains sub-node\n", np_pmu->name);
+		pr_err("%pOFn: failed to find domains sub-node\n", np_pmu);
 		return 0;
 	}
 
@@ -396,7 +396,7 @@ int __init dove_init_pmu(void)
 	pmu->pmc_base = of_iomap(pmu->of_node, 0);
 	pmu->pmu_base = of_iomap(pmu->of_node, 1);
 	if (!pmu->pmc_base || !pmu->pmu_base) {
-		pr_err("%s: failed to map PMU\n", np_pmu->name);
+		pr_err("%pOFn: failed to map PMU\n", np_pmu);
 		iounmap(pmu->pmu_base);
 		iounmap(pmu->pmc_base);
 		kfree(pmu);
@@ -414,7 +414,7 @@ int __init dove_init_pmu(void)
 			break;
 
 		domain->pmu = pmu;
-		domain->base.name = kstrdup(np->name, GFP_KERNEL);
+		domain->base.name = kasprintf(GFP_KERNEL, "%pOFn", np);
 		if (!domain->base.name) {
 			kfree(domain);
 			break;
@@ -444,7 +444,7 @@ int __init dove_init_pmu(void)
 	/* Loss of the interrupt controller is not a fatal error. */
 	parent_irq = irq_of_parse_and_map(pmu->of_node, 0);
 	if (!parent_irq) {
-		pr_err("%s: no interrupt specified\n", np_pmu->name);
+		pr_err("%pOFn: no interrupt specified\n", np_pmu);
 	} else {
 		ret = dove_init_pmu_irq(pmu, parent_irq);
 		if (ret)
diff --git a/drivers/soc/fsl/qe/qe_tdm.c b/drivers/soc/fsl/qe/qe_tdm.c
index f744c214f680..f78c34647ca2 100644
--- a/drivers/soc/fsl/qe/qe_tdm.c
+++ b/drivers/soc/fsl/qe/qe_tdm.c
@@ -131,7 +131,7 @@ int ucc_of_parse_tdm(struct device_node *np, struct ucc_tdm *utdm,
 
 	pdev = of_find_device_by_node(np2);
 	if (!pdev) {
-		pr_err("%s: failed to lookup pdev\n", np2->name);
+		pr_err("%pOFn: failed to lookup pdev\n", np2);
 		of_node_put(np2);
 		return -EINVAL;
 	}
@@ -153,7 +153,7 @@ int ucc_of_parse_tdm(struct device_node *np, struct ucc_tdm *utdm,
 	pdev = of_find_device_by_node(np2);
 	if (!pdev) {
 		ret = -EINVAL;
-		pr_err("%s: failed to lookup pdev\n", np2->name);
+		pr_err("%pOFn: failed to lookup pdev\n", np2);
 		of_node_put(np2);
 		goto err_miss_siram_property;
 	}
diff --git a/drivers/soc/qcom/apr.c b/drivers/soc/qcom/apr.c
index 57af8a537332..4bda793ba6ae 100644
--- a/drivers/soc/qcom/apr.c
+++ b/drivers/soc/qcom/apr.c
@@ -219,7 +219,7 @@ static int apr_add_device(struct device *dev, struct device_node *np,
 	adev->domain_id = id->domain_id;
 	adev->version = id->svc_version;
 	if (np)
-		strncpy(adev->name, np->name, APR_NAME_SIZE);
+		snprintf(adev->name, APR_NAME_SIZE, "%pOFn", np);
 	else
 		strncpy(adev->name, id->name, APR_NAME_SIZE);
 
diff --git a/drivers/soc/rockchip/pm_domains.c b/drivers/soc/rockchip/pm_domains.c
index 6dff8682155f..6f86a726bb45 100644
--- a/drivers/soc/rockchip/pm_domains.c
+++ b/drivers/soc/rockchip/pm_domains.c
@@ -392,21 +392,21 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 	error = of_property_read_u32(node, "reg", &id);
 	if (error) {
 		dev_err(pmu->dev,
-			"%s: failed to retrieve domain id (reg): %d\n",
-			node->name, error);
+			"%pOFn: failed to retrieve domain id (reg): %d\n",
+			node, error);
 		return -EINVAL;
 	}
 
 	if (id >= pmu->info->num_domains) {
-		dev_err(pmu->dev, "%s: invalid domain id %d\n",
-			node->name, id);
+		dev_err(pmu->dev, "%pOFn: invalid domain id %d\n",
+			node, id);
 		return -EINVAL;
 	}
 
 	pd_info = &pmu->info->domain_info[id];
 	if (!pd_info) {
-		dev_err(pmu->dev, "%s: undefined domain id %d\n",
-			node->name, id);
+		dev_err(pmu->dev, "%pOFn: undefined domain id %d\n",
+			node, id);
 		return -EINVAL;
 	}
 
@@ -424,8 +424,8 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 		if (!pd->clks)
 			return -ENOMEM;
 	} else {
-		dev_dbg(pmu->dev, "%s: doesn't have clocks: %d\n",
-			node->name, pd->num_clks);
+		dev_dbg(pmu->dev, "%pOFn: doesn't have clocks: %d\n",
+			node, pd->num_clks);
 		pd->num_clks = 0;
 	}
 
@@ -434,8 +434,8 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 		if (IS_ERR(pd->clks[i].clk)) {
 			error = PTR_ERR(pd->clks[i].clk);
 			dev_err(pmu->dev,
-				"%s: failed to get clk at index %d: %d\n",
-				node->name, i, error);
+				"%pOFn: failed to get clk at index %d: %d\n",
+				node, i, error);
 			return error;
 		}
 	}
@@ -486,8 +486,8 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 	error = rockchip_pd_power(pd, true);
 	if (error) {
 		dev_err(pmu->dev,
-			"failed to power on domain '%s': %d\n",
-			node->name, error);
+			"failed to power on domain '%pOFn': %d\n",
+			node, error);
 		goto err_unprepare_clocks;
 	}
 
@@ -575,24 +575,24 @@ static int rockchip_pm_add_subdomain(struct rockchip_pmu *pmu,
 		error = of_property_read_u32(parent, "reg", &idx);
 		if (error) {
 			dev_err(pmu->dev,
-				"%s: failed to retrieve domain id (reg): %d\n",
-				parent->name, error);
+				"%pOFn: failed to retrieve domain id (reg): %d\n",
+				parent, error);
 			goto err_out;
 		}
 		parent_domain = pmu->genpd_data.domains[idx];
 
 		error = rockchip_pm_add_one_domain(pmu, np);
 		if (error) {
-			dev_err(pmu->dev, "failed to handle node %s: %d\n",
-				np->name, error);
+			dev_err(pmu->dev, "failed to handle node %pOFn: %d\n",
+				np, error);
 			goto err_out;
 		}
 
 		error = of_property_read_u32(np, "reg", &idx);
 		if (error) {
 			dev_err(pmu->dev,
-				"%s: failed to retrieve domain id (reg): %d\n",
-				np->name, error);
+				"%pOFn: failed to retrieve domain id (reg): %d\n",
+				np, error);
 			goto err_out;
 		}
 		child_domain = pmu->genpd_data.domains[idx];
@@ -683,16 +683,16 @@ static int rockchip_pm_domain_probe(struct platform_device *pdev)
 	for_each_available_child_of_node(np, node) {
 		error = rockchip_pm_add_one_domain(pmu, node);
 		if (error) {
-			dev_err(dev, "failed to handle node %s: %d\n",
-				node->name, error);
+			dev_err(dev, "failed to handle node %pOFn: %d\n",
+				node, error);
 			of_node_put(node);
 			goto err_out;
 		}
 
 		error = rockchip_pm_add_subdomain(pmu, node);
 		if (error < 0) {
-			dev_err(dev, "failed to handle subdomain node %s: %d\n",
-				node->name, error);
+			dev_err(dev, "failed to handle subdomain node %pOFn: %d\n",
+				node, error);
 			of_node_put(node);
 			goto err_out;
 		}
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
index 2d6f3fcf3211..acbe63e925d5 100644
--- a/drivers/soc/tegra/pmc.c
+++ b/drivers/soc/tegra/pmc.c
@@ -796,7 +796,7 @@ static void tegra_powergate_add(struct tegra_pmc *pmc, struct device_node *np)
 
 	id = tegra_powergate_lookup(pmc, np->name);
 	if (id < 0) {
-		pr_err("powergate lookup failed for %s: %d\n", np->name, id);
+		pr_err("powergate lookup failed for %pOFn: %d\n", np, id);
 		goto free_mem;
 	}
 
@@ -816,13 +816,13 @@ static void tegra_powergate_add(struct tegra_pmc *pmc, struct device_node *np)
 
 	err = tegra_powergate_of_get_clks(pg, np);
 	if (err < 0) {
-		pr_err("failed to get clocks for %s: %d\n", np->name, err);
+		pr_err("failed to get clocks for %pOFn: %d\n", np, err);
 		goto set_available;
 	}
 
 	err = tegra_powergate_of_get_resets(pg, np, off);
 	if (err < 0) {
-		pr_err("failed to get resets for %s: %d\n", np->name, err);
+		pr_err("failed to get resets for %pOFn: %d\n", np, err);
 		goto remove_clks;
 	}
 
@@ -851,15 +851,15 @@ static void tegra_powergate_add(struct tegra_pmc *pmc, struct device_node *np)
 
 	err = pm_genpd_init(&pg->genpd, NULL, off);
 	if (err < 0) {
-		pr_err("failed to initialise PM domain %s: %d\n", np->name,
+		pr_err("failed to initialise PM domain %pOFn: %d\n", np,
 		       err);
 		goto remove_resets;
 	}
 
 	err = of_genpd_add_provider_simple(np, &pg->genpd);
 	if (err < 0) {
-		pr_err("failed to add PM domain provider for %s: %d\n",
-		       np->name, err);
+		pr_err("failed to add PM domain provider for %pOFn: %d\n",
+		       np, err);
 		goto remove_genpd;
 	}
 
diff --git a/drivers/soc/ti/knav_dma.c b/drivers/soc/ti/knav_dma.c
index 224d7ddeeb76..bbd4e5bc8707 100644
--- a/drivers/soc/ti/knav_dma.c
+++ b/drivers/soc/ti/knav_dma.c
@@ -544,15 +544,15 @@ static void __iomem *pktdma_get_regs(struct knav_dma_device *dma,
 
 	ret = of_address_to_resource(node, index, &res);
 	if (ret) {
-		dev_err(dev, "Can't translate of node(%s) address for index(%d)\n",
-			node->name, index);
+		dev_err(dev, "Can't translate of node(%pOFn) address for index(%d)\n",
+			node, index);
 		return ERR_PTR(ret);
 	}
 
 	regs = devm_ioremap_resource(kdev->dev, &res);
 	if (IS_ERR(regs))
-		dev_err(dev, "Failed to map register base for index(%d) node(%s)\n",
-			index, node->name);
+		dev_err(dev, "Failed to map register base for index(%d) node(%pOFn)\n",
+			index, node);
 	if (_size)
 		*_size = resource_size(&res);
 
diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c
index 6755f2af5619..b5d5673c255c 100644
--- a/drivers/soc/ti/knav_qmss_queue.c
+++ b/drivers/soc/ti/knav_qmss_queue.c
@@ -1382,15 +1382,15 @@ static void __iomem *knav_queue_map_reg(struct knav_device *kdev,
 
 	ret = of_address_to_resource(node, index, &res);
 	if (ret) {
-		dev_err(kdev->dev, "Can't translate of node(%s) address for index(%d)\n",
-			node->name, index);
+		dev_err(kdev->dev, "Can't translate of node(%pOFn) address for index(%d)\n",
+			node, index);
 		return ERR_PTR(ret);
 	}
 
 	regs = devm_ioremap_resource(kdev->dev, &res);
 	if (IS_ERR(regs))
-		dev_err(kdev->dev, "Failed to map register base for index(%d) node(%s)\n",
-			index, node->name);
+		dev_err(kdev->dev, "Failed to map register base for index(%d) node(%pOFn)\n",
+			index, node);
 	return regs;
 }
 
-- 
2.17.1


WARNING: multiple messages have this Message-ID (diff)
From: robh@kernel.org (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] soc: Convert to using %pOFn instead of device_node.name
Date: Mon, 27 Aug 2018 20:52:44 -0500	[thread overview]
Message-ID: <20180828015252.28511-43-robh@kernel.org> (raw)
In-Reply-To: <20180828015252.28511-1-robh@kernel.org>

In preparation to remove the node name pointer from struct device_node,
convert printf users to use the %pOFn format specifier.

Cc: Qiang Zhao <qiang.zhao@nxp.com>
Cc: Li Yang <leoyang.li@nxp.com>
Cc: Andy Gross <andy.gross@linaro.org>
Cc: David Brown <david.brown@linaro.org>
Cc: Heiko Stuebner <heiko@sntech.de>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Jonathan Hunter <jonathanh@nvidia.com>
Cc: Santosh Shilimkar <ssantosh@kernel.org>
Cc: linuxppc-dev at lists.ozlabs.org
Cc: linux-arm-kernel at lists.infradead.org
Cc: linux-soc at vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 drivers/soc/dove/pmu.c            |  8 +++---
 drivers/soc/fsl/qe/qe_tdm.c       |  4 +--
 drivers/soc/qcom/apr.c            |  2 +-
 drivers/soc/rockchip/pm_domains.c | 44 +++++++++++++++----------------
 drivers/soc/tegra/pmc.c           | 12 ++++-----
 drivers/soc/ti/knav_dma.c         |  8 +++---
 drivers/soc/ti/knav_qmss_queue.c  |  8 +++---
 7 files changed, 43 insertions(+), 43 deletions(-)

diff --git a/drivers/soc/dove/pmu.c b/drivers/soc/dove/pmu.c
index 5abb08ffb74d..ffc5311c0ed8 100644
--- a/drivers/soc/dove/pmu.c
+++ b/drivers/soc/dove/pmu.c
@@ -383,7 +383,7 @@ int __init dove_init_pmu(void)
 
 	domains_node = of_get_child_by_name(np_pmu, "domains");
 	if (!domains_node) {
-		pr_err("%s: failed to find domains sub-node\n", np_pmu->name);
+		pr_err("%pOFn: failed to find domains sub-node\n", np_pmu);
 		return 0;
 	}
 
@@ -396,7 +396,7 @@ int __init dove_init_pmu(void)
 	pmu->pmc_base = of_iomap(pmu->of_node, 0);
 	pmu->pmu_base = of_iomap(pmu->of_node, 1);
 	if (!pmu->pmc_base || !pmu->pmu_base) {
-		pr_err("%s: failed to map PMU\n", np_pmu->name);
+		pr_err("%pOFn: failed to map PMU\n", np_pmu);
 		iounmap(pmu->pmu_base);
 		iounmap(pmu->pmc_base);
 		kfree(pmu);
@@ -414,7 +414,7 @@ int __init dove_init_pmu(void)
 			break;
 
 		domain->pmu = pmu;
-		domain->base.name = kstrdup(np->name, GFP_KERNEL);
+		domain->base.name = kasprintf(GFP_KERNEL, "%pOFn", np);
 		if (!domain->base.name) {
 			kfree(domain);
 			break;
@@ -444,7 +444,7 @@ int __init dove_init_pmu(void)
 	/* Loss of the interrupt controller is not a fatal error. */
 	parent_irq = irq_of_parse_and_map(pmu->of_node, 0);
 	if (!parent_irq) {
-		pr_err("%s: no interrupt specified\n", np_pmu->name);
+		pr_err("%pOFn: no interrupt specified\n", np_pmu);
 	} else {
 		ret = dove_init_pmu_irq(pmu, parent_irq);
 		if (ret)
diff --git a/drivers/soc/fsl/qe/qe_tdm.c b/drivers/soc/fsl/qe/qe_tdm.c
index f744c214f680..f78c34647ca2 100644
--- a/drivers/soc/fsl/qe/qe_tdm.c
+++ b/drivers/soc/fsl/qe/qe_tdm.c
@@ -131,7 +131,7 @@ int ucc_of_parse_tdm(struct device_node *np, struct ucc_tdm *utdm,
 
 	pdev = of_find_device_by_node(np2);
 	if (!pdev) {
-		pr_err("%s: failed to lookup pdev\n", np2->name);
+		pr_err("%pOFn: failed to lookup pdev\n", np2);
 		of_node_put(np2);
 		return -EINVAL;
 	}
@@ -153,7 +153,7 @@ int ucc_of_parse_tdm(struct device_node *np, struct ucc_tdm *utdm,
 	pdev = of_find_device_by_node(np2);
 	if (!pdev) {
 		ret = -EINVAL;
-		pr_err("%s: failed to lookup pdev\n", np2->name);
+		pr_err("%pOFn: failed to lookup pdev\n", np2);
 		of_node_put(np2);
 		goto err_miss_siram_property;
 	}
diff --git a/drivers/soc/qcom/apr.c b/drivers/soc/qcom/apr.c
index 57af8a537332..4bda793ba6ae 100644
--- a/drivers/soc/qcom/apr.c
+++ b/drivers/soc/qcom/apr.c
@@ -219,7 +219,7 @@ static int apr_add_device(struct device *dev, struct device_node *np,
 	adev->domain_id = id->domain_id;
 	adev->version = id->svc_version;
 	if (np)
-		strncpy(adev->name, np->name, APR_NAME_SIZE);
+		snprintf(adev->name, APR_NAME_SIZE, "%pOFn", np);
 	else
 		strncpy(adev->name, id->name, APR_NAME_SIZE);
 
diff --git a/drivers/soc/rockchip/pm_domains.c b/drivers/soc/rockchip/pm_domains.c
index 6dff8682155f..6f86a726bb45 100644
--- a/drivers/soc/rockchip/pm_domains.c
+++ b/drivers/soc/rockchip/pm_domains.c
@@ -392,21 +392,21 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 	error = of_property_read_u32(node, "reg", &id);
 	if (error) {
 		dev_err(pmu->dev,
-			"%s: failed to retrieve domain id (reg): %d\n",
-			node->name, error);
+			"%pOFn: failed to retrieve domain id (reg): %d\n",
+			node, error);
 		return -EINVAL;
 	}
 
 	if (id >= pmu->info->num_domains) {
-		dev_err(pmu->dev, "%s: invalid domain id %d\n",
-			node->name, id);
+		dev_err(pmu->dev, "%pOFn: invalid domain id %d\n",
+			node, id);
 		return -EINVAL;
 	}
 
 	pd_info = &pmu->info->domain_info[id];
 	if (!pd_info) {
-		dev_err(pmu->dev, "%s: undefined domain id %d\n",
-			node->name, id);
+		dev_err(pmu->dev, "%pOFn: undefined domain id %d\n",
+			node, id);
 		return -EINVAL;
 	}
 
@@ -424,8 +424,8 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 		if (!pd->clks)
 			return -ENOMEM;
 	} else {
-		dev_dbg(pmu->dev, "%s: doesn't have clocks: %d\n",
-			node->name, pd->num_clks);
+		dev_dbg(pmu->dev, "%pOFn: doesn't have clocks: %d\n",
+			node, pd->num_clks);
 		pd->num_clks = 0;
 	}
 
@@ -434,8 +434,8 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 		if (IS_ERR(pd->clks[i].clk)) {
 			error = PTR_ERR(pd->clks[i].clk);
 			dev_err(pmu->dev,
-				"%s: failed to get clk at index %d: %d\n",
-				node->name, i, error);
+				"%pOFn: failed to get clk at index %d: %d\n",
+				node, i, error);
 			return error;
 		}
 	}
@@ -486,8 +486,8 @@ static int rockchip_pm_add_one_domain(struct rockchip_pmu *pmu,
 	error = rockchip_pd_power(pd, true);
 	if (error) {
 		dev_err(pmu->dev,
-			"failed to power on domain '%s': %d\n",
-			node->name, error);
+			"failed to power on domain '%pOFn': %d\n",
+			node, error);
 		goto err_unprepare_clocks;
 	}
 
@@ -575,24 +575,24 @@ static int rockchip_pm_add_subdomain(struct rockchip_pmu *pmu,
 		error = of_property_read_u32(parent, "reg", &idx);
 		if (error) {
 			dev_err(pmu->dev,
-				"%s: failed to retrieve domain id (reg): %d\n",
-				parent->name, error);
+				"%pOFn: failed to retrieve domain id (reg): %d\n",
+				parent, error);
 			goto err_out;
 		}
 		parent_domain = pmu->genpd_data.domains[idx];
 
 		error = rockchip_pm_add_one_domain(pmu, np);
 		if (error) {
-			dev_err(pmu->dev, "failed to handle node %s: %d\n",
-				np->name, error);
+			dev_err(pmu->dev, "failed to handle node %pOFn: %d\n",
+				np, error);
 			goto err_out;
 		}
 
 		error = of_property_read_u32(np, "reg", &idx);
 		if (error) {
 			dev_err(pmu->dev,
-				"%s: failed to retrieve domain id (reg): %d\n",
-				np->name, error);
+				"%pOFn: failed to retrieve domain id (reg): %d\n",
+				np, error);
 			goto err_out;
 		}
 		child_domain = pmu->genpd_data.domains[idx];
@@ -683,16 +683,16 @@ static int rockchip_pm_domain_probe(struct platform_device *pdev)
 	for_each_available_child_of_node(np, node) {
 		error = rockchip_pm_add_one_domain(pmu, node);
 		if (error) {
-			dev_err(dev, "failed to handle node %s: %d\n",
-				node->name, error);
+			dev_err(dev, "failed to handle node %pOFn: %d\n",
+				node, error);
 			of_node_put(node);
 			goto err_out;
 		}
 
 		error = rockchip_pm_add_subdomain(pmu, node);
 		if (error < 0) {
-			dev_err(dev, "failed to handle subdomain node %s: %d\n",
-				node->name, error);
+			dev_err(dev, "failed to handle subdomain node %pOFn: %d\n",
+				node, error);
 			of_node_put(node);
 			goto err_out;
 		}
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
index 2d6f3fcf3211..acbe63e925d5 100644
--- a/drivers/soc/tegra/pmc.c
+++ b/drivers/soc/tegra/pmc.c
@@ -796,7 +796,7 @@ static void tegra_powergate_add(struct tegra_pmc *pmc, struct device_node *np)
 
 	id = tegra_powergate_lookup(pmc, np->name);
 	if (id < 0) {
-		pr_err("powergate lookup failed for %s: %d\n", np->name, id);
+		pr_err("powergate lookup failed for %pOFn: %d\n", np, id);
 		goto free_mem;
 	}
 
@@ -816,13 +816,13 @@ static void tegra_powergate_add(struct tegra_pmc *pmc, struct device_node *np)
 
 	err = tegra_powergate_of_get_clks(pg, np);
 	if (err < 0) {
-		pr_err("failed to get clocks for %s: %d\n", np->name, err);
+		pr_err("failed to get clocks for %pOFn: %d\n", np, err);
 		goto set_available;
 	}
 
 	err = tegra_powergate_of_get_resets(pg, np, off);
 	if (err < 0) {
-		pr_err("failed to get resets for %s: %d\n", np->name, err);
+		pr_err("failed to get resets for %pOFn: %d\n", np, err);
 		goto remove_clks;
 	}
 
@@ -851,15 +851,15 @@ static void tegra_powergate_add(struct tegra_pmc *pmc, struct device_node *np)
 
 	err = pm_genpd_init(&pg->genpd, NULL, off);
 	if (err < 0) {
-		pr_err("failed to initialise PM domain %s: %d\n", np->name,
+		pr_err("failed to initialise PM domain %pOFn: %d\n", np,
 		       err);
 		goto remove_resets;
 	}
 
 	err = of_genpd_add_provider_simple(np, &pg->genpd);
 	if (err < 0) {
-		pr_err("failed to add PM domain provider for %s: %d\n",
-		       np->name, err);
+		pr_err("failed to add PM domain provider for %pOFn: %d\n",
+		       np, err);
 		goto remove_genpd;
 	}
 
diff --git a/drivers/soc/ti/knav_dma.c b/drivers/soc/ti/knav_dma.c
index 224d7ddeeb76..bbd4e5bc8707 100644
--- a/drivers/soc/ti/knav_dma.c
+++ b/drivers/soc/ti/knav_dma.c
@@ -544,15 +544,15 @@ static void __iomem *pktdma_get_regs(struct knav_dma_device *dma,
 
 	ret = of_address_to_resource(node, index, &res);
 	if (ret) {
-		dev_err(dev, "Can't translate of node(%s) address for index(%d)\n",
-			node->name, index);
+		dev_err(dev, "Can't translate of node(%pOFn) address for index(%d)\n",
+			node, index);
 		return ERR_PTR(ret);
 	}
 
 	regs = devm_ioremap_resource(kdev->dev, &res);
 	if (IS_ERR(regs))
-		dev_err(dev, "Failed to map register base for index(%d) node(%s)\n",
-			index, node->name);
+		dev_err(dev, "Failed to map register base for index(%d) node(%pOFn)\n",
+			index, node);
 	if (_size)
 		*_size = resource_size(&res);
 
diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c
index 6755f2af5619..b5d5673c255c 100644
--- a/drivers/soc/ti/knav_qmss_queue.c
+++ b/drivers/soc/ti/knav_qmss_queue.c
@@ -1382,15 +1382,15 @@ static void __iomem *knav_queue_map_reg(struct knav_device *kdev,
 
 	ret = of_address_to_resource(node, index, &res);
 	if (ret) {
-		dev_err(kdev->dev, "Can't translate of node(%s) address for index(%d)\n",
-			node->name, index);
+		dev_err(kdev->dev, "Can't translate of node(%pOFn) address for index(%d)\n",
+			node, index);
 		return ERR_PTR(ret);
 	}
 
 	regs = devm_ioremap_resource(kdev->dev, &res);
 	if (IS_ERR(regs))
-		dev_err(kdev->dev, "Failed to map register base for index(%d) node(%s)\n",
-			index, node->name);
+		dev_err(kdev->dev, "Failed to map register base for index(%d) node(%pOFn)\n",
+			index, node);
 	return regs;
 }
 
-- 
2.17.1

  parent reply	other threads:[~2018-08-28  1:55 UTC|newest]

Thread overview: 282+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-28  1:52 [PATCH] ARM: omap: Convert to using %pOFn instead of device_node.name Rob Herring
2018-08-28  1:52 ` Rob Herring
2018-08-28  1:52 ` [PATCH] ARM: shmobile: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  7:24   ` Geert Uytterhoeven
2018-08-28  7:24     ` Geert Uytterhoeven
2018-08-30 14:37     ` Simon Horman
2018-08-30 14:37       ` Simon Horman
2018-08-28  1:52 ` [PATCH] ARM: zynq: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-09-05 13:27   ` Michal Simek
2018-09-05 13:27     ` Michal Simek
2018-08-28  1:52 ` [PATCH] MIPS: " Rob Herring
2018-08-28 16:54   ` Paul Burton
2018-08-28  1:52 ` [PATCH] nios2: " Rob Herring
2018-08-28 15:47   ` Ley Foon Tan
2018-08-28  1:52 ` [PATCH] powerpc: " Rob Herring
2018-10-04  6:14   ` Michael Ellerman
2018-10-04  6:14     ` Michael Ellerman
2018-08-28  1:52 ` [PATCH] sparc: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28 14:13   ` Rob Herring
2018-08-28 14:13     ` Rob Herring
2018-08-28 15:44   ` [PATCH v2] " Rob Herring
2018-08-28 15:44     ` Rob Herring
2018-10-08  5:57     ` David Miller
2018-10-08  5:57       ` David Miller
2018-08-28  1:52 ` [PATCH] xtensa: " Rob Herring
2018-08-28 15:45   ` Max Filippov
2018-08-28  1:52 ` [PATCH] ata: ahci: " Rob Herring
2018-08-28 14:44   ` Jens Axboe
2018-08-28  1:52 ` [PATCH] bus: fsl-mc: " Rob Herring
2018-08-28  1:52 ` [PATCH] bus: mvebu-mbus: " Rob Herring
2018-08-28  1:52 ` [PATCH] clk: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28 15:44   ` [PATCH v2] " Rob Herring
2018-08-28 15:44     ` Rob Herring
2018-08-28 22:15     ` Stephen Boyd
2018-08-28 22:15       ` Stephen Boyd
2018-08-28 22:15       ` Stephen Boyd
2018-08-29 11:13       ` Rob Herring
2018-08-29 11:13         ` Rob Herring
2018-08-29 11:13         ` Rob Herring
2018-08-30 16:50         ` Stephen Boyd
2018-08-30 16:50           ` Stephen Boyd
2018-08-30 16:50           ` Stephen Boyd
2018-08-30 16:50           ` Stephen Boyd
2018-09-03  5:29   ` [PATCH] " Chen-Yu Tsai
2018-09-03  5:29     ` Chen-Yu Tsai
2018-09-07 15:08   ` Rob Herring
2018-09-07 15:08     ` Rob Herring
2018-09-07 15:08     ` Rob Herring
2018-08-28  1:52 ` [PATCH] clocksource: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-29 15:23   ` Daniel Lezcano
2018-08-29 15:23     ` Daniel Lezcano
2018-08-28  1:52 ` [PATCH] cpufreq: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-09-03  6:34   ` Viresh Kumar
2018-09-03  6:34     ` Viresh Kumar
2018-09-14  8:27   ` Rafael J. Wysocki
2018-09-14  8:27     ` Rafael J. Wysocki
2018-08-28  1:52 ` [PATCH] devfreq: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  2:06   ` Chanwoo Choi
2018-08-28  2:06     ` Chanwoo Choi
     [not found]   ` <CGME20180828015312epcas2p2fdb2ee1d1b292f212ec773dd8fb4bfc7@epcms1p8>
2018-08-29  2:10     ` MyungJoo Ham
2018-08-29  2:10       ` MyungJoo Ham
2018-08-29  2:10       ` MyungJoo Ham
2018-08-28  1:52 ` [PATCH] firmware: " Rob Herring
2018-09-28 21:47   ` Rob Herring
2018-10-01  9:09     ` Sudeep Holla
2018-08-28  1:52 ` [PATCH] gpio: " Rob Herring
2018-08-29 12:02   ` Linus Walleij
     [not found] ` <20180828015252.28511-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2018-08-28  1:52   ` [PATCH] drm: " Rob Herring
2018-08-28  1:52     ` Rob Herring
2018-08-28 12:54     ` Maarten Lankhorst
2018-08-28 12:54       ` Maarten Lankhorst
     [not found]       ` <597b1b06-e1fb-dcf1-67f4-d6685fd38cad-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2018-08-28 15:04         ` Rob Herring
2018-08-28 15:04           ` Rob Herring
2018-08-28 15:04     ` Rob Clark
2018-08-28  1:52 ` [PATCH] hwmon: " Rob Herring
2018-08-28 13:57   ` Guenter Roeck
2018-08-28 15:11     ` Rob Herring
2018-08-28 18:43   ` Guenter Roeck
2018-08-28  1:52 ` [PATCH] i2c: " Rob Herring
2018-08-28  6:33   ` Peter Rosin
2018-08-29 18:42     ` Rob Herring
2018-08-29 19:52       ` Peter Rosin
2018-08-30  0:49         ` Rob Herring
2018-08-28  1:52 ` [PATCH] iio: " Rob Herring
2018-09-02 17:55   ` Jonathan Cameron
2018-08-28  1:52 ` [PATCH] input: sun4i-lradc: " Rob Herring
2018-09-28 21:54   ` Rob Herring
2018-09-28 23:59   ` Dmitry Torokhov
2018-08-28  1:52 ` [PATCH] irqchip: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-09-08 12:15   ` Thomas Gleixner
2018-09-08 12:15     ` Thomas Gleixner
2018-09-10 13:55     ` Rob Herring
2018-09-10 13:55       ` Rob Herring
2018-08-28  1:52 ` [PATCH] leds: " Rob Herring
2018-08-28 11:07   ` Sakari Ailus
2018-08-28 11:37   ` Pavel Machek
2018-08-28 20:37   ` Jacek Anaszewski
2018-08-28  1:52 ` [PATCH] macintosh: " Rob Herring
2018-08-28  1:52 ` [PATCH] mailbox: " Rob Herring
2018-09-28 21:54   ` Rob Herring
2018-08-28  1:52 ` [PATCH] media: " Rob Herring
2018-08-28 10:06   ` Laurent Pinchart
2018-08-28 12:21     ` Rob Herring
2018-09-12 15:10       ` Mauro Carvalho Chehab
2018-08-28  1:52 ` [PATCH] memory: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28 10:42   ` Thierry Reding
2018-08-28 10:42     ` Thierry Reding
2018-08-28 15:33   ` Tony Lindgren
2018-08-28 15:33     ` Tony Lindgren
2018-08-29  7:44   ` Krzysztof Kozlowski
2018-08-29  7:44     ` Krzysztof Kozlowski
2018-08-29  7:44     ` Krzysztof Kozlowski
2018-08-28  1:52 ` [PATCH] mfd: max77620: " Rob Herring
2018-09-10 15:24   ` Lee Jones
2018-08-28  1:52 ` [PATCH] misc: " Rob Herring
2018-08-28  2:35   ` Andrew Donnellan
2018-08-28  1:52 ` [PATCH] mmc: " Rob Herring
2018-08-28  7:59   ` Adrian Hunter
2018-08-28 11:41   ` Ulf Hansson
2018-08-28  1:52 ` [PATCH] mtd: rawnand: " Rob Herring
2018-08-28  9:16   ` Boris Brezillon
2018-09-04 21:58   ` Miquel Raynal
2018-08-28  1:52 ` [PATCH] net: ethernet: " Rob Herring
2018-08-28 15:44   ` [PATCH v2] " Rob Herring
2018-08-28 15:45     ` Matthias Brugger
2018-08-29  2:06     ` Sean Wang
2018-08-30  2:41     ` David Miller
2018-08-28  1:52 ` [PATCH] net: phy: " Rob Herring
2018-08-28 16:57   ` Florian Fainelli
2018-08-28  1:52 ` [PATCH] net: wireless: ath: " Rob Herring
2018-08-28 11:19   ` Kalle Valo
2018-09-03 16:58   ` Kalle Valo
2018-08-28  1:52 ` [PATCH] nvmem: " Rob Herring
2018-09-28 22:06   ` Rob Herring
2018-10-01  9:05     ` Srinivas Kandagatla
2018-08-28  1:52 ` [PATCH] perf: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28 17:02   ` Will Deacon
2018-08-28 17:02     ` Will Deacon
2018-09-28 21:19     ` Rob Herring
2018-09-28 21:19       ` Rob Herring
2018-10-01  9:18       ` Will Deacon
2018-10-01  9:18         ` Will Deacon
2018-08-28  1:52 ` [PATCH] phy: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-09-04  2:24   ` Kishon Vijay Abraham I
2018-09-04  2:24     ` Kishon Vijay Abraham I
2018-09-04 14:21     ` Rob Herring
2018-09-04 14:21       ` Rob Herring
2018-08-28  1:52 ` [PATCH] pinctrl: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  1:57   ` A.s. Dong
2018-08-28  1:57     ` A.s. Dong
2018-08-28  1:57     ` A.s. Dong
2018-08-28  1:57     ` A.s. Dong
2018-08-28  7:21   ` Alexandre Belloni
2018-08-28  7:21     ` Alexandre Belloni
2018-08-28  7:21     ` Alexandre Belloni
2018-08-28  7:21     ` Alexandre Belloni
2018-08-28 15:34   ` Tony Lindgren
2018-08-28 15:34     ` Tony Lindgren
2018-08-28 15:34     ` Tony Lindgren
2018-08-28 15:34     ` Tony Lindgren
2018-08-29  2:13   ` Sean Wang
2018-08-29  2:13     ` Sean Wang
2018-08-29  2:13     ` Sean Wang
2018-08-29  2:13     ` Sean Wang
2018-08-29  2:15   ` Chen-Yu Tsai
2018-08-29  2:15     ` Chen-Yu Tsai
2018-08-29  2:15     ` Chen-Yu Tsai
2018-08-29  2:15     ` Chen-Yu Tsai
2018-08-29  8:31   ` Heiko Stübner
2018-08-29  8:31     ` Heiko Stübner
2018-08-29  8:31     ` Heiko Stübner
2018-08-29 12:07   ` Linus Walleij
2018-08-29 12:07     ` Linus Walleij
2018-08-29 12:07     ` Linus Walleij
2018-08-29 12:07     ` Linus Walleij
2018-09-03  9:57   ` Patrice CHOTARD
2018-09-03  9:57     ` Patrice CHOTARD
2018-09-03  9:57     ` Patrice CHOTARD
2018-09-03  9:57     ` Patrice CHOTARD
2018-08-28  1:52 ` [PATCH] regulator: " Rob Herring
2018-08-29  7:47   ` Krzysztof Kozlowski
2018-08-28  1:52 ` [PATCH] rpmsg: " Rob Herring
2018-08-28  3:08   ` Bjorn Andersson
2018-08-28  1:52 ` Rob Herring [this message]
2018-08-28  1:52   ` [PATCH] soc: " Rob Herring
2018-08-28 10:44   ` Thierry Reding
2018-08-28 10:44     ` Thierry Reding
2018-08-28 18:48   ` Li Yang
2018-08-28 18:48     ` Li Yang
2018-08-28 18:48     ` Li Yang
2018-08-28 22:14     ` Tyrel Datwyler
2018-08-28 22:14       ` Tyrel Datwyler
2018-08-28 22:14       ` Tyrel Datwyler
2018-08-28 22:29       ` Li Yang
2018-08-28 22:29         ` Li Yang
2018-08-28 22:29         ` Li Yang
2018-08-28 22:46         ` Tyrel Datwyler
2018-08-28 22:46           ` Tyrel Datwyler
2018-08-28 22:46           ` Tyrel Datwyler
2018-08-29  8:21   ` Heiko Stübner
2018-08-29  8:21     ` Heiko Stübner
2018-08-29  8:57   ` Qiang Zhao
2018-08-29  8:57     ` Qiang Zhao
2018-08-29  8:57     ` Qiang Zhao
2018-09-30 17:08   ` Andy Gross
2018-09-30 17:08     ` Andy Gross
2018-09-30 17:08     ` Andy Gross
2018-08-28  1:52 ` [PATCH] staging: " Rob Herring
2018-08-28  6:34   ` Ian Arkver
2018-08-28 15:44   ` [PATCH v2] " Rob Herring
2018-09-12 15:17     ` Mauro Carvalho Chehab
2018-09-12 20:26       ` Rob Herring
2018-09-12 21:07         ` Mauro Carvalho Chehab
2018-09-12 22:48           ` Rob Herring
2018-09-13 11:50         ` Joe Perches
2018-09-13 21:26           ` Rob Herring
2018-09-14 11:41             ` Joe Perches
2018-08-28  1:52 ` [PATCH] thermal: " Rob Herring
2018-08-30 14:58   ` Daniel Lezcano
2018-08-28  1:52 ` [PATCH] tty: " Rob Herring
2018-09-28 22:09   ` Rob Herring
2018-09-28 22:09     ` Rob Herring
2018-09-28 22:32     ` Rob Herring
2018-09-28 22:32       ` Rob Herring
2018-08-28  1:52 ` [PATCH] uio: " Rob Herring
2018-08-28  1:52 ` [PATCH] backlight: pm8941: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  1:52 ` [PATCH] fbdev: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-09-28 22:11   ` Rob Herring
2018-09-28 22:11     ` Rob Herring
2018-09-28 22:11     ` Rob Herring
2018-10-08 10:41   ` Bartlomiej Zolnierkiewicz
2018-10-08 10:41     ` Bartlomiej Zolnierkiewicz
2018-10-08 10:41     ` Bartlomiej Zolnierkiewicz
2018-08-28  1:52 ` [PATCH] sound: " Rob Herring
2018-08-28  7:54   ` Andy Shevchenko
2018-08-28  7:54     ` Andy Shevchenko
2018-08-28  7:54     ` Andy Shevchenko
2018-08-28 15:44   ` [PATCH v2] " Rob Herring
2018-08-28 15:44     ` Rob Herring
2018-09-03 21:45     ` [alsa-devel] " Takashi Iwai
2018-09-03 21:45       ` Takashi Iwai
2018-08-28  1:52 ` [PATCH] ASoC: " Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  1:52   ` Rob Herring
2018-08-28  7:57   ` Andy Shevchenko
2018-08-28  7:57     ` Andy Shevchenko
2018-08-28  7:57     ` Andy Shevchenko
2018-08-28  7:57     ` Andy Shevchenko
2018-08-28 15:44   ` [PATCH v2] " Rob Herring
2018-08-28 15:44     ` Rob Herring
2018-08-28 15:44     ` Rob Herring
2018-08-28 18:03     ` Mark Brown
2018-08-28 18:03       ` Mark Brown
2018-08-28 18:03       ` Mark Brown
2018-08-28 18:03       ` Mark Brown
2018-08-28  6:55 ` [PATCH] ARM: omap: " Joe Perches
2018-08-28  6:55   ` Joe Perches
2018-08-28 15:44 ` [PATCH v2] " Rob Herring
2018-08-28 15:44   ` Rob Herring
2018-09-07 17:14   ` Tony Lindgren
2018-09-07 17:14     ` Tony Lindgren
2018-08-28  1:52 dmaengine: " Rob Herring
2018-08-28  1:52 ` [PATCH] " Rob Herring
2018-08-28  5:16 Vinod Koul
2018-08-28  5:16 ` [PATCH] " Vinod

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180828015252.28511-43-robh@kernel.org \
    --to=robh@kernel.org \
    --cc=andy.gross@linaro.org \
    --cc=david.brown@linaro.org \
    --cc=heiko@sntech.de \
    --cc=jonathanh@nvidia.com \
    --cc=leoyang.li@nxp.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-soc@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=qiang.zhao@nxp.com \
    --cc=ssantosh@kernel.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.