Skip to contentSkip to conversation

irqchip: Convert to using %pOF instead of full_name

PATCH In review Rob Herring 8 participants 8 messages Jul 18, 2017 – Aug 23, 2017

Subsystems (12)
Arm i.MX and Layerscape Arm MediaTek Tegra architecture support Arm STM32 +8 more subsystems
lore
Share & tools
lore

Rob Herring

author
#0 lore raw

PATCH 1/1 irqchip: Convert to using %pOF instead of full_name

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
---
 drivers/irqchip/irq-bcm2835.c            |  9 ++++-----
 drivers/irqchip/irq-bcm2836.c            |  5 ++---
 drivers/irqchip/irq-crossbar.c           |  6 +++---
 drivers/irqchip/irq-digicolor.c          |  8 ++++----
 drivers/irqchip/irq-dw-apb-ictl.c        | 12 ++++++------
 drivers/irqchip/irq-gic-v3-its-pci-msi.c |  2 +-
 drivers/irqchip/irq-gic-v3-its.c         |  6 +++---
 drivers/irqchip/irq-gic-v3.c             | 11 ++++-------
 drivers/irqchip/irq-imx-gpcv2.c          |  4 ++--
 drivers/irqchip/irq-lpc32xx.c            |  2 +-
 drivers/irqchip/irq-mtk-sysirq.c         |  3 +--
 drivers/irqchip/irq-mxs.c                |  4 ++--
 drivers/irqchip/irq-stm32-exti.c         |  8 ++++----
 drivers/irqchip/irq-sun4i.c              |  6 +++---
 drivers/irqchip/irq-tegra.c              | 16 ++++++++--------
 drivers/irqchip/irq-xilinx-intc.c        |  4 ++--
 16 files changed, 50 insertions(+), 56 deletions(-)
drivers/irqchip/irq-bcm2835.c +4 -5
index 44d7c38dde47..d2da8a1e6b1b 100644
@@ -147,13 +147,12 @@ static int __init armctrl_of_init(struct device_node *node,
 
 	base = of_iomap(node, 0);
 	if (!base)
-		panic("%s: unable to map IC registers\n",
-			node->full_name);
+		panic("%pOF: unable to map IC registers\n", node);
 
 	intc.domain = irq_domain_add_linear(node, MAKE_HWIRQ(NR_BANKS, 0),
 			&armctrl_ops, NULL);
 	if (!intc.domain)
-		panic("%s: unable to create IRQ domain\n", node->full_name);
+		panic("%pOF: unable to create IRQ domain\n", node);
 
 	for (b = 0; b < NR_BANKS; b++) {
 		intc.pending[b] = base + reg_pending[b];

@@ -173,8 +172,8 @@ static int __init armctrl_of_init(struct device_node *node,
 		int parent_irq = irq_of_parse_and_map(node, 0);
 
 		if (!parent_irq) {
-			panic("%s: unable to get parent interrupt.\n",
-			      node->full_name);
+			panic("%pOF: unable to get parent interrupt.\n",
+			      node);
 		}
 		irq_set_chained_handler(parent_irq, bcm2836_chained_handle_irq);
 	} else {
drivers/irqchip/irq-bcm2836.c +2 -3
index e7463e3c0814..dc8c1e3eafe7 100644
@@ -282,8 +282,7 @@ static int __init bcm2836_arm_irqchip_l1_intc_of_init(struct device_node *node,
 {
 	intc.base = of_iomap(node, 0);
 	if (!intc.base) {
-		panic("%s: unable to map local interrupt registers\n",
-			node->full_name);
+		panic("%pOF: unable to map local interrupt registers\n", node);
 	}
 
 	bcm2835_init_local_timer_frequency();

@@ -292,7 +291,7 @@ static int __init bcm2836_arm_irqchip_l1_intc_of_init(struct device_node *node,
 					    &bcm2836_arm_irqchip_intc_ops,
 					    NULL);
 	if (!intc.domain)
-		panic("%s: unable to create IRQ domain\n", node->full_name);
+		panic("%pOF: unable to create IRQ domain\n", node);
 
 	bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_CNTPSIRQ,
 					 &bcm2836_arm_irqchip_timer);
drivers/irqchip/irq-crossbar.c +3 -3
index f96601268f71..99d97d7e3fd7 100644
@@ -341,13 +341,13 @@ static int __init irqcrossbar_init(struct device_node *node,
 	int err;
 
 	if (!parent) {
-		pr_err("%s: no parent, giving up\n", node->full_name);
+		pr_err("%pOF: no parent, giving up\n", node);
 		return -ENODEV;
 	}
 
 	parent_domain = irq_find_host(parent);
 	if (!parent_domain) {
-		pr_err("%s: unable to obtain parent domain\n", node->full_name);
+		pr_err("%pOF: unable to obtain parent domain\n", node);
 		return -ENXIO;
 	}
 

@@ -360,7 +360,7 @@ static int __init irqcrossbar_init(struct device_node *node,
 					  node, &crossbar_domain_ops,
 					  NULL);
 	if (!domain) {
-		pr_err("%s: failed to allocated domain\n", node->full_name);
+		pr_err("%pOF: failed to allocated domain\n", node);
 		return -ENOMEM;
 	}
drivers/irqchip/irq-digicolor.c +4 -4
index dad85e74c37c..df5752cfbc09 100644
@@ -78,7 +78,7 @@ static int __init digicolor_of_init(struct device_node *node,
 
 	reg_base = of_iomap(node, 0);
 	if (!reg_base) {
-		pr_err("%s: unable to map IC registers\n", node->full_name);
+		pr_err("%pOF: unable to map IC registers\n", node);
 		return -ENXIO;
 	}
 

@@ -88,7 +88,7 @@ static int __init digicolor_of_init(struct device_node *node,
 
 	ucregs = syscon_regmap_lookup_by_phandle(node, "syscon");
 	if (IS_ERR(ucregs)) {
-		pr_err("%s: unable to map UC registers\n", node->full_name);
+		pr_err("%pOF: unable to map UC registers\n", node);
 		return PTR_ERR(ucregs);
 	}
 	/* channel 1, regular IRQs */

@@ -97,7 +97,7 @@ static int __init digicolor_of_init(struct device_node *node,
 	digicolor_irq_domain =
 		irq_domain_add_linear(node, 64, &irq_generic_chip_ops, NULL);
 	if (!digicolor_irq_domain) {
-		pr_err("%s: unable to create IRQ domain\n", node->full_name);
+		pr_err("%pOF: unable to create IRQ domain\n", node);
 		return -ENOMEM;
 	}
 

@@ -105,7 +105,7 @@ static int __init digicolor_of_init(struct device_node *node,
 					     "digicolor_irq", handle_level_irq,
 					     clr, 0, 0);
 	if (ret) {
-		pr_err("%s: unable to allocate IRQ gc\n", node->full_name);
+		pr_err("%pOF: unable to allocate IRQ gc\n", node);
 		return ret;
 	}
drivers/irqchip/irq-dw-apb-ictl.c +6 -6
index 052f266364c0..0a19618ce2c8 100644
@@ -79,24 +79,24 @@ static int __init dw_apb_ictl_init(struct device_node *np,
 	/* Map the parent interrupt for the chained handler */
 	irq = irq_of_parse_and_map(np, 0);
 	if (irq <= 0) {
-		pr_err("%s: unable to parse irq\n", np->full_name);
+		pr_err("%pOF: unable to parse irq\n", np);
 		return -EINVAL;
 	}
 
 	ret = of_address_to_resource(np, 0, &r);
 	if (ret) {
-		pr_err("%s: unable to get resource\n", np->full_name);
+		pr_err("%pOF: unable to get resource\n", np);
 		return ret;
 	}
 
 	if (!request_mem_region(r.start, resource_size(&r), np->full_name)) {
-		pr_err("%s: unable to request mem region\n", np->full_name);
+		pr_err("%pOF: unable to request mem region\n", np);
 		return -ENOMEM;
 	}
 
 	iobase = ioremap(r.start, resource_size(&r));
 	if (!iobase) {
-		pr_err("%s: unable to map resource\n", np->full_name);
+		pr_err("%pOF: unable to map resource\n", np);
 		ret = -ENOMEM;
 		goto err_release;
 	}

@@ -123,7 +123,7 @@ static int __init dw_apb_ictl_init(struct device_node *np,
 	domain = irq_domain_add_linear(np, nrirqs,
 				       &irq_generic_chip_ops, NULL);
 	if (!domain) {
-		pr_err("%s: unable to add irq domain\n", np->full_name);
+		pr_err("%pOF: unable to add irq domain\n", np);
 		ret = -ENOMEM;
 		goto err_unmap;
 	}

@@ -132,7 +132,7 @@ static int __init dw_apb_ictl_init(struct device_node *np,
 					     handle_level_irq, clr, 0,
 					     IRQ_GC_INIT_MASK_CACHE);
 	if (ret) {
-		pr_err("%s: unable to alloc irq domain gc\n", np->full_name);
+		pr_err("%pOF: unable to alloc irq domain gc\n", np);
 		goto err_unmap;
 	}
drivers/irqchip/irq-gic-v3-its-pci-msi.c +1 -1
index 77931214d954..14a8c0a7e095 100644
@@ -138,7 +138,7 @@ static int __init its_pci_of_msi_init(void)
 		if (its_pci_msi_init_one(of_node_to_fwnode(np), np->full_name))
 			continue;
 
-		pr_info("PCI/MSI: %s domain created\n", np->full_name);
+		pr_info("PCI/MSI: %pOF domain created\n", np);
 	}
 
 	return 0;
drivers/irqchip/irq-gic-v3-its.c +3 -3
index 68932873eebc..62298ee360bc 100644
@@ -1816,13 +1816,13 @@ static int __init its_of_probe(struct device_node *node)
 	for (np = of_find_matching_node(node, its_device_id); np;
 	     np = of_find_matching_node(np, its_device_id)) {
 		if (!of_property_read_bool(np, "msi-controller")) {
-			pr_warn("%s: no msi-controller property, ITS ignored\n",
-				np->full_name);
+			pr_warn("%pOF: no msi-controller property, ITS ignored\n",
+				np);
 			continue;
 		}
 
 		if (of_address_to_resource(np, 0, &res)) {
-			pr_warn("%s: no regs?\n", np->full_name);
+			pr_warn("%pOF: no regs?\n", np);
 			continue;
 		}
drivers/irqchip/irq-gic-v3.c +4 -7
index dbffb7ab6203..afae0a9d0df9 100644
@@ -1057,7 +1057,7 @@ static void __init gic_populate_ppi_partitions(struct device_node *gic_node)
 			if (WARN_ON(cpu == -1))
 				continue;
 
-			pr_cont("%s[%d] ", cpu_node->full_name, cpu);
+			pr_cont("%pOF[%d] ", cpu_node, cpu);
 
 			cpumask_set_cpu(cpu, &part->mask);
 		}

@@ -1125,15 +1125,13 @@ static int __init gic_of_init(struct device_node *node, struct device_node *pare
 
 	dist_base = of_iomap(node, 0);
 	if (!dist_base) {
-		pr_err("%s: unable to map gic dist registers\n",
-			node->full_name);
+		pr_err("%pOF: unable to map gic dist registers\n", node);
 		return -ENXIO;
 	}
 
 	err = gic_validate_dist_version(dist_base);
 	if (err) {
-		pr_err("%s: no distributor detected, giving up\n",
-			node->full_name);
+		pr_err("%pOF: no distributor detected, giving up\n", node);
 		goto out_unmap_dist;
 	}
 

@@ -1153,8 +1151,7 @@ static int __init gic_of_init(struct device_node *node, struct device_node *pare
 		ret = of_address_to_resource(node, 1 + i, &res);
 		rdist_regs[i].redist_base = of_iomap(node, 1 + i);
 		if (ret || !rdist_regs[i].redist_base) {
-			pr_err("%s: couldn't map region %d\n",
-			       node->full_name, i);
+			pr_err("%pOF: couldn't map region %d\n", node, i);
 			err = -ENODEV;
 			goto out_unmap_rdist;
 		}
drivers/irqchip/irq-imx-gpcv2.c +2 -2
index bb36f572e322..675eda5ff2b8 100644
@@ -214,13 +214,13 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node,
 	int i;
 
 	if (!parent) {
-		pr_err("%s: no parent, giving up\n", node->full_name);
+		pr_err("%pOF: no parent, giving up\n", node);
 		return -ENODEV;
 	}
 
 	parent_domain = irq_find_host(parent);
 	if (!parent_domain) {
-		pr_err("%s: unable to get parent domain\n", node->full_name);
+		pr_err("%pOF: unable to get parent domain\n", node);
 		return -ENXIO;
 	}
drivers/irqchip/irq-lpc32xx.c +1 -1
index 1034aeb2e98a..a48357d369b5 100644
@@ -191,7 +191,7 @@ static int __init lpc32xx_of_ic_init(struct device_node *node,
 
 	irqc->base = of_iomap(node, 0);
 	if (!irqc->base) {
-		pr_err("%s: unable to map registers\n", node->full_name);
+		pr_err("%pOF: unable to map registers\n", node);
 		kfree(irqc);
 		return -EINVAL;
 	}
drivers/irqchip/irq-mtk-sysirq.c +1 -2
index eeac512ec5a8..90aaf190157f 100644
@@ -178,8 +178,7 @@ static int __init mtk_sysirq_of_init(struct device_node *node,
 		chip_data->intpol_words[i] = size / 4;
 		chip_data->intpol_bases[i] = of_iomap(node, i);
 		if (ret || !chip_data->intpol_bases[i]) {
-			pr_err("%s: couldn't map region %d\n",
-			       node->full_name, i);
+			pr_err("%pOF: couldn't map region %d\n", node, i);
 			ret = -ENODEV;
 			goto out_free_intpol;
 		}
drivers/irqchip/irq-mxs.c +2 -2
index 05fa9f7af53c..e8b31f52e071 100644
@@ -179,7 +179,7 @@ static void __init icoll_add_domain(struct device_node *np,
 					     &icoll_irq_domain_ops, NULL);
 
 	if (!icoll_domain)
-		panic("%s: unable to create irq domain", np->full_name);
+		panic("%pOF: unable to create irq domain", np);
 }
 
 static void __iomem * __init icoll_init_iobase(struct device_node *np)

@@ -188,7 +188,7 @@ static void __iomem * __init icoll_init_iobase(struct device_node *np)
 
 	icoll_base = of_io_request_and_map(np, 0, np->name);
 	if (IS_ERR(icoll_base))
-		panic("%s: unable to map resource", np->full_name);
+		panic("%pOF: unable to map resource", np);
 	return icoll_base;
 }
drivers/irqchip/irq-stm32-exti.c +4 -4
index 491568c95aa5..45363ff8d06f 100644
@@ -140,7 +140,7 @@ static int __init stm32_exti_init(struct device_node *node,
 
 	base = of_iomap(node, 0);
 	if (!base) {
-		pr_err("%s: Unable to map registers\n", node->full_name);
+		pr_err("%pOF: Unable to map registers\n", node);
 		return -ENOMEM;
 	}
 

@@ -149,7 +149,7 @@ static int __init stm32_exti_init(struct device_node *node,
 	nr_exti = fls(readl_relaxed(base + EXTI_RTSR));
 	writel_relaxed(0, base + EXTI_RTSR);
 
-	pr_info("%s: %d External IRQs detected\n", node->full_name, nr_exti);
+	pr_info("%pOF: %d External IRQs detected\n", node, nr_exti);
 
 	domain = irq_domain_add_linear(node, nr_exti,
 				       &irq_exti_domain_ops, NULL);

@@ -163,8 +163,8 @@ static int __init stm32_exti_init(struct device_node *node,
 	ret = irq_alloc_domain_generic_chips(domain, nr_exti, 1, "exti",
 					     handle_edge_irq, clr, 0, 0);
 	if (ret) {
-		pr_err("%s: Could not allocate generic interrupt chip.\n",
-			node->full_name);
+		pr_err("%pOF: Could not allocate generic interrupt chip.\n",
+			node);
 		goto out_free_domain;
 	}
drivers/irqchip/irq-sun4i.c +3 -3
index 376b28074e0d..e3e5b9132b75 100644
@@ -97,8 +97,8 @@ static int __init sun4i_of_init(struct device_node *node,
 {
 	sun4i_irq_base = of_iomap(node, 0);
 	if (!sun4i_irq_base)
-		panic("%s: unable to map IC registers\n",
-			node->full_name);
+		panic("%pOF: unable to map IC registers\n",
+			node);
 
 	/* Disable all interrupts */
 	writel(0, sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(0));

@@ -124,7 +124,7 @@ static int __init sun4i_of_init(struct device_node *node,
 	sun4i_irq_domain = irq_domain_add_linear(node, 3 * 32,
 						 &sun4i_irq_ops, NULL);
 	if (!sun4i_irq_domain)
-		panic("%s: unable to create IRQ domain\n", node->full_name);
+		panic("%pOF: unable to create IRQ domain\n", node);
 
 	set_handle_irq(sun4i_handle_irq);
drivers/irqchip/irq-tegra.c +8 -8
index 3973a14bb15b..0abc0cd1c32e 100644
@@ -291,13 +291,13 @@ static int __init tegra_ictlr_init(struct device_node *node,
 	int err;
 
 	if (!parent) {
-		pr_err("%s: no parent, giving up\n", node->full_name);
+		pr_err("%pOF: no parent, giving up\n", node);
 		return -ENODEV;
 	}
 
 	parent_domain = irq_find_host(parent);
 	if (!parent_domain) {
-		pr_err("%s: unable to obtain parent domain\n", node->full_name);
+		pr_err("%pOF: unable to obtain parent domain\n", node);
 		return -ENXIO;
 	}
 

@@ -329,29 +329,29 @@ static int __init tegra_ictlr_init(struct device_node *node,
 	}
 
 	if (!num_ictlrs) {
-		pr_err("%s: no valid regions, giving up\n", node->full_name);
+		pr_err("%pOF: no valid regions, giving up\n", node);
 		err = -ENOMEM;
 		goto out_free;
 	}
 
 	WARN(num_ictlrs != soc->num_ictlrs,
-	     "%s: Found %u interrupt controllers in DT; expected %u.\n",
-	     node->full_name, num_ictlrs, soc->num_ictlrs);
+	     "%pOF: Found %u interrupt controllers in DT; expected %u.\n",
+	     node, num_ictlrs, soc->num_ictlrs);
 
 
 	domain = irq_domain_add_hierarchy(parent_domain, 0, num_ictlrs * 32,
 					  node, &tegra_ictlr_domain_ops,
 					  lic);
 	if (!domain) {
-		pr_err("%s: failed to allocated domain\n", node->full_name);
+		pr_err("%pOF: failed to allocated domain\n", node);
 		err = -ENOMEM;
 		goto out_unmap;
 	}
 
 	tegra_ictlr_syscore_init();
 
-	pr_info("%s: %d interrupts forwarded to %s\n",
-		node->full_name, num_ictlrs * 32, parent->full_name);
+	pr_info("%pOF: %d interrupts forwarded to %pOF\n",
+		node, num_ictlrs * 32, parent);
 
 	return 0;
drivers/irqchip/irq-xilinx-intc.c +2 -2
index 3db7ab1c9741..e3043ded8973 100644
@@ -186,8 +186,8 @@ static int __init xilinx_intc_of_init(struct device_node *intc,
 	if (irqc->intr_mask >> nr_irq)
 		pr_warn("irq-xilinx: mismatch in kind-of-intr param\n");
 
-	pr_info("irq-xilinx: %s: num_irq=%d, edge=0x%x\n",
-		intc->full_name, nr_irq, irqc->intr_mask);
+	pr_info("irq-xilinx: %pOF: num_irq=%d, edge=0x%x\n",
+		intc, nr_irq, irqc->intr_mask);
 
 
 	/*

--
2.11.0

maintainer Acked-by on v1 1/1
#1 lore raw
Quoted 31 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
Quoted 1 line

---

For

Quoted 1 line
 drivers/irqchip/irq-lpc32xx.c            |  2 +-
Acked-by Vladimir Zapolskiy <[email protected]>

--
With best wishes,
Vladimir

maintainer Acked-by on v1 1/1
#2 lore raw
In reply to Rob Herring

Hi Rob,

Quoted 31 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
Quoted 3 lines

---

 drivers/irqchip/irq-digicolor.c          |  8 ++++----
Acked-by Baruch Siach <[email protected]>

Thanks,
baruch

Signature · 3 lines
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
Acked-by on v1 1/1
#3 lore raw
In reply to Rob Herring
Quoted 31 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
Quoted 15 lines
---
 drivers/irqchip/irq-bcm2835.c            |  9 ++++-----
 drivers/irqchip/irq-bcm2836.c            |  5 ++---
 drivers/irqchip/irq-crossbar.c           |  6 +++---
 drivers/irqchip/irq-digicolor.c          |  8 ++++----
 drivers/irqchip/irq-dw-apb-ictl.c        | 12 ++++++------
 drivers/irqchip/irq-gic-v3-its-pci-msi.c |  2 +-
 drivers/irqchip/irq-gic-v3-its.c         |  6 +++---
 drivers/irqchip/irq-gic-v3.c             | 11 ++++-------
 drivers/irqchip/irq-imx-gpcv2.c          |  4 ++--
 drivers/irqchip/irq-lpc32xx.c            |  2 +-
 drivers/irqchip/irq-mtk-sysirq.c         |  3 +--
 drivers/irqchip/irq-mxs.c                |  4 ++--
 drivers/irqchip/irq-stm32-exti.c         |  8 ++++----
 drivers/irqchip/irq-sun4i.c              |  6 +++---

for sun4i:

Acked-by Maxime Ripard <[email protected]>

Maxime

Signature · 10 lines
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170719/bec97007/attachment.sig>
Acked-by on v1 1/1
#4 lore raw
In reply to Rob Herring
Quoted 41 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
---
  drivers/irqchip/irq-bcm2835.c            |  9 ++++-----
  drivers/irqchip/irq-bcm2836.c            |  5 ++---
  drivers/irqchip/irq-crossbar.c           |  6 +++---
  drivers/irqchip/irq-digicolor.c          |  8 ++++----
  drivers/irqchip/irq-dw-apb-ictl.c        | 12 ++++++------
  drivers/irqchip/irq-gic-v3-its-pci-msi.c |  2 +-
  drivers/irqchip/irq-gic-v3-its.c         |  6 +++---
  drivers/irqchip/irq-gic-v3.c             | 11 ++++-------
  drivers/irqchip/irq-imx-gpcv2.c          |  4 ++--
Quoted 8 lines
  drivers/irqchip/irq-lpc32xx.c            |  2 +-
  drivers/irqchip/irq-mtk-sysirq.c         |  3 +--
  drivers/irqchip/irq-mxs.c                |  4 ++--
  drivers/irqchip/irq-stm32-exti.c         |  8 ++++----
  drivers/irqchip/irq-sun4i.c              |  6 +++---
  drivers/irqchip/irq-tegra.c              | 16 ++++++++--------
  drivers/irqchip/irq-xilinx-intc.c        |  4 ++--
  16 files changed, 50 insertions(+), 56 deletions(-)

For irq-stm32-exti.c:

Acked-by Alexandre TORGUE <[email protected]>

Regards
Alex

Quoted 450 lines
drivers/irqchip/irq-bcm2835.c +4 -5
index 44d7c38dde47..d2da8a1e6b1b 100644
@@ -147,13 +147,12 @@ static int __init armctrl_of_init(struct device_node *node,
 
  	base = of_iomap(node, 0);
  	if (!base)
-		panic("%s: unable to map IC registers\n",
-			node->full_name);
+		panic("%pOF: unable to map IC registers\n", node);
 
  	intc.domain = irq_domain_add_linear(node, MAKE_HWIRQ(NR_BANKS, 0),
  			&armctrl_ops, NULL);
  	if (!intc.domain)
-		panic("%s: unable to create IRQ domain\n", node->full_name);
+		panic("%pOF: unable to create IRQ domain\n", node);
 
  	for (b = 0; b < NR_BANKS; b++) {
  		intc.pending[b] = base + reg_pending[b];

@@ -173,8 +172,8 @@ static int __init armctrl_of_init(struct device_node *node,
  		int parent_irq = irq_of_parse_and_map(node, 0);
 
  		if (!parent_irq) {
-			panic("%s: unable to get parent interrupt.\n",
-			      node->full_name);
+			panic("%pOF: unable to get parent interrupt.\n",
+			      node);
  		}
  		irq_set_chained_handler(parent_irq, bcm2836_chained_handle_irq);
  	} else {
drivers/irqchip/irq-bcm2836.c +2 -3
index e7463e3c0814..dc8c1e3eafe7 100644
@@ -282,8 +282,7 @@ static int __init bcm2836_arm_irqchip_l1_intc_of_init(struct device_node *node,
  {
  	intc.base = of_iomap(node, 0);
  	if (!intc.base) {
-		panic("%s: unable to map local interrupt registers\n",
-			node->full_name);
+		panic("%pOF: unable to map local interrupt registers\n", node);
  	}
 
  	bcm2835_init_local_timer_frequency();

@@ -292,7 +291,7 @@ static int __init bcm2836_arm_irqchip_l1_intc_of_init(struct device_node *node,
  					    &bcm2836_arm_irqchip_intc_ops,
  					    NULL);
  	if (!intc.domain)
-		panic("%s: unable to create IRQ domain\n", node->full_name);
+		panic("%pOF: unable to create IRQ domain\n", node);
 
  	bcm2836_arm_irqchip_register_irq(LOCAL_IRQ_CNTPSIRQ,
  					 &bcm2836_arm_irqchip_timer);
drivers/irqchip/irq-crossbar.c +3 -3
index f96601268f71..99d97d7e3fd7 100644
@@ -341,13 +341,13 @@ static int __init irqcrossbar_init(struct device_node *node,
  	int err;
 
  	if (!parent) {
-		pr_err("%s: no parent, giving up\n", node->full_name);
+		pr_err("%pOF: no parent, giving up\n", node);
  		return -ENODEV;
  	}
 
  	parent_domain = irq_find_host(parent);
  	if (!parent_domain) {
-		pr_err("%s: unable to obtain parent domain\n", node->full_name);
+		pr_err("%pOF: unable to obtain parent domain\n", node);
  		return -ENXIO;
  	}
 

@@ -360,7 +360,7 @@ static int __init irqcrossbar_init(struct device_node *node,
  					  node, &crossbar_domain_ops,
  					  NULL);
  	if (!domain) {
-		pr_err("%s: failed to allocated domain\n", node->full_name);
+		pr_err("%pOF: failed to allocated domain\n", node);
  		return -ENOMEM;
  	}
drivers/irqchip/irq-digicolor.c +4 -4
index dad85e74c37c..df5752cfbc09 100644
@@ -78,7 +78,7 @@ static int __init digicolor_of_init(struct device_node *node,
 
  	reg_base = of_iomap(node, 0);
  	if (!reg_base) {
-		pr_err("%s: unable to map IC registers\n", node->full_name);
+		pr_err("%pOF: unable to map IC registers\n", node);
  		return -ENXIO;
  	}
 

@@ -88,7 +88,7 @@ static int __init digicolor_of_init(struct device_node *node,
 
  	ucregs = syscon_regmap_lookup_by_phandle(node, "syscon");
  	if (IS_ERR(ucregs)) {
-		pr_err("%s: unable to map UC registers\n", node->full_name);
+		pr_err("%pOF: unable to map UC registers\n", node);
  		return PTR_ERR(ucregs);
  	}
  	/* channel 1, regular IRQs */

@@ -97,7 +97,7 @@ static int __init digicolor_of_init(struct device_node *node,
  	digicolor_irq_domain =
  		irq_domain_add_linear(node, 64, &irq_generic_chip_ops, NULL);
  	if (!digicolor_irq_domain) {
-		pr_err("%s: unable to create IRQ domain\n", node->full_name);
+		pr_err("%pOF: unable to create IRQ domain\n", node);
  		return -ENOMEM;
  	}
 

@@ -105,7 +105,7 @@ static int __init digicolor_of_init(struct device_node *node,
  					     "digicolor_irq", handle_level_irq,
  					     clr, 0, 0);
  	if (ret) {
-		pr_err("%s: unable to allocate IRQ gc\n", node->full_name);
+		pr_err("%pOF: unable to allocate IRQ gc\n", node);
  		return ret;
  	}
drivers/irqchip/irq-dw-apb-ictl.c +6 -6
index 052f266364c0..0a19618ce2c8 100644
@@ -79,24 +79,24 @@ static int __init dw_apb_ictl_init(struct device_node *np,
  	/* Map the parent interrupt for the chained handler */
  	irq = irq_of_parse_and_map(np, 0);
  	if (irq <= 0) {
-		pr_err("%s: unable to parse irq\n", np->full_name);
+		pr_err("%pOF: unable to parse irq\n", np);
  		return -EINVAL;
  	}
 
  	ret = of_address_to_resource(np, 0, &r);
  	if (ret) {
-		pr_err("%s: unable to get resource\n", np->full_name);
+		pr_err("%pOF: unable to get resource\n", np);
  		return ret;
  	}
 
  	if (!request_mem_region(r.start, resource_size(&r), np->full_name)) {
-		pr_err("%s: unable to request mem region\n", np->full_name);
+		pr_err("%pOF: unable to request mem region\n", np);
  		return -ENOMEM;
  	}
 
  	iobase = ioremap(r.start, resource_size(&r));
  	if (!iobase) {
-		pr_err("%s: unable to map resource\n", np->full_name);
+		pr_err("%pOF: unable to map resource\n", np);
  		ret = -ENOMEM;
  		goto err_release;
  	}

@@ -123,7 +123,7 @@ static int __init dw_apb_ictl_init(struct device_node *np,
  	domain = irq_domain_add_linear(np, nrirqs,
  				       &irq_generic_chip_ops, NULL);
  	if (!domain) {
-		pr_err("%s: unable to add irq domain\n", np->full_name);
+		pr_err("%pOF: unable to add irq domain\n", np);
  		ret = -ENOMEM;
  		goto err_unmap;
  	}

@@ -132,7 +132,7 @@ static int __init dw_apb_ictl_init(struct device_node *np,
  					     handle_level_irq, clr, 0,
  					     IRQ_GC_INIT_MASK_CACHE);
  	if (ret) {
-		pr_err("%s: unable to alloc irq domain gc\n", np->full_name);
+		pr_err("%pOF: unable to alloc irq domain gc\n", np);
  		goto err_unmap;
  	}
drivers/irqchip/irq-gic-v3-its-pci-msi.c +1 -1
index 77931214d954..14a8c0a7e095 100644
@@ -138,7 +138,7 @@ static int __init its_pci_of_msi_init(void)
  		if (its_pci_msi_init_one(of_node_to_fwnode(np), np->full_name))
  			continue;
 
-		pr_info("PCI/MSI: %s domain created\n", np->full_name);
+		pr_info("PCI/MSI: %pOF domain created\n", np);
  	}
 
  	return 0;
drivers/irqchip/irq-gic-v3-its.c +3 -3
index 68932873eebc..62298ee360bc 100644
@@ -1816,13 +1816,13 @@ static int __init its_of_probe(struct device_node *node)
  	for (np = of_find_matching_node(node, its_device_id); np;
  	     np = of_find_matching_node(np, its_device_id)) {
  		if (!of_property_read_bool(np, "msi-controller")) {
-			pr_warn("%s: no msi-controller property, ITS ignored\n",
-				np->full_name);
+			pr_warn("%pOF: no msi-controller property, ITS ignored\n",
+				np);
  			continue;
  		}
 
  		if (of_address_to_resource(np, 0, &res)) {
-			pr_warn("%s: no regs?\n", np->full_name);
+			pr_warn("%pOF: no regs?\n", np);
  			continue;
  		}
drivers/irqchip/irq-gic-v3.c +4 -7
index dbffb7ab6203..afae0a9d0df9 100644
@@ -1057,7 +1057,7 @@ static void __init gic_populate_ppi_partitions(struct device_node *gic_node)
  			if (WARN_ON(cpu == -1))
  				continue;
 
-			pr_cont("%s[%d] ", cpu_node->full_name, cpu);
+			pr_cont("%pOF[%d] ", cpu_node, cpu);
 
  			cpumask_set_cpu(cpu, &part->mask);
  		}

@@ -1125,15 +1125,13 @@ static int __init gic_of_init(struct device_node *node, struct device_node *pare
 
  	dist_base = of_iomap(node, 0);
  	if (!dist_base) {
-		pr_err("%s: unable to map gic dist registers\n",
-			node->full_name);
+		pr_err("%pOF: unable to map gic dist registers\n", node);
  		return -ENXIO;
  	}
 
  	err = gic_validate_dist_version(dist_base);
  	if (err) {
-		pr_err("%s: no distributor detected, giving up\n",
-			node->full_name);
+		pr_err("%pOF: no distributor detected, giving up\n", node);
  		goto out_unmap_dist;
  	}
 

@@ -1153,8 +1151,7 @@ static int __init gic_of_init(struct device_node *node, struct device_node *pare
  		ret = of_address_to_resource(node, 1 + i, &res);
  		rdist_regs[i].redist_base = of_iomap(node, 1 + i);
  		if (ret || !rdist_regs[i].redist_base) {
-			pr_err("%s: couldn't map region %d\n",
-			       node->full_name, i);
+			pr_err("%pOF: couldn't map region %d\n", node, i);
  			err = -ENODEV;
  			goto out_unmap_rdist;
  		}
drivers/irqchip/irq-imx-gpcv2.c +2 -2
index bb36f572e322..675eda5ff2b8 100644
@@ -214,13 +214,13 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node,
  	int i;
 
  	if (!parent) {
-		pr_err("%s: no parent, giving up\n", node->full_name);
+		pr_err("%pOF: no parent, giving up\n", node);
  		return -ENODEV;
  	}
 
  	parent_domain = irq_find_host(parent);
  	if (!parent_domain) {
-		pr_err("%s: unable to get parent domain\n", node->full_name);
+		pr_err("%pOF: unable to get parent domain\n", node);
  		return -ENXIO;
  	}
drivers/irqchip/irq-lpc32xx.c +1 -1
index 1034aeb2e98a..a48357d369b5 100644
@@ -191,7 +191,7 @@ static int __init lpc32xx_of_ic_init(struct device_node *node,
 
  	irqc->base = of_iomap(node, 0);
  	if (!irqc->base) {
-		pr_err("%s: unable to map registers\n", node->full_name);
+		pr_err("%pOF: unable to map registers\n", node);
  		kfree(irqc);
  		return -EINVAL;
  	}
drivers/irqchip/irq-mtk-sysirq.c +1 -2
index eeac512ec5a8..90aaf190157f 100644
@@ -178,8 +178,7 @@ static int __init mtk_sysirq_of_init(struct device_node *node,
  		chip_data->intpol_words[i] = size / 4;
  		chip_data->intpol_bases[i] = of_iomap(node, i);
  		if (ret || !chip_data->intpol_bases[i]) {
-			pr_err("%s: couldn't map region %d\n",
-			       node->full_name, i);
+			pr_err("%pOF: couldn't map region %d\n", node, i);
  			ret = -ENODEV;
  			goto out_free_intpol;
  		}
drivers/irqchip/irq-mxs.c +2 -2
index 05fa9f7af53c..e8b31f52e071 100644
@@ -179,7 +179,7 @@ static void __init icoll_add_domain(struct device_node *np,
  					     &icoll_irq_domain_ops, NULL);
 
  	if (!icoll_domain)
-		panic("%s: unable to create irq domain", np->full_name);
+		panic("%pOF: unable to create irq domain", np);
  }
 
  static void __iomem * __init icoll_init_iobase(struct device_node *np)

@@ -188,7 +188,7 @@ static void __iomem * __init icoll_init_iobase(struct device_node *np)
 
  	icoll_base = of_io_request_and_map(np, 0, np->name);
  	if (IS_ERR(icoll_base))
-		panic("%s: unable to map resource", np->full_name);
+		panic("%pOF: unable to map resource", np);
  	return icoll_base;
  }
drivers/irqchip/irq-stm32-exti.c +4 -4
index 491568c95aa5..45363ff8d06f 100644
@@ -140,7 +140,7 @@ static int __init stm32_exti_init(struct device_node *node,
 
  	base = of_iomap(node, 0);
  	if (!base) {
-		pr_err("%s: Unable to map registers\n", node->full_name);
+		pr_err("%pOF: Unable to map registers\n", node);
  		return -ENOMEM;
  	}
 

@@ -149,7 +149,7 @@ static int __init stm32_exti_init(struct device_node *node,
  	nr_exti = fls(readl_relaxed(base + EXTI_RTSR));
  	writel_relaxed(0, base + EXTI_RTSR);
 
-	pr_info("%s: %d External IRQs detected\n", node->full_name, nr_exti);
+	pr_info("%pOF: %d External IRQs detected\n", node, nr_exti);
 
  	domain = irq_domain_add_linear(node, nr_exti,
  				       &irq_exti_domain_ops, NULL);

@@ -163,8 +163,8 @@ static int __init stm32_exti_init(struct device_node *node,
  	ret = irq_alloc_domain_generic_chips(domain, nr_exti, 1, "exti",
  					     handle_edge_irq, clr, 0, 0);
  	if (ret) {
-		pr_err("%s: Could not allocate generic interrupt chip.\n",
-			node->full_name);
+		pr_err("%pOF: Could not allocate generic interrupt chip.\n",
+			node);
  		goto out_free_domain;
  	}
drivers/irqchip/irq-sun4i.c +3 -3
index 376b28074e0d..e3e5b9132b75 100644
@@ -97,8 +97,8 @@ static int __init sun4i_of_init(struct device_node *node,
  {
  	sun4i_irq_base = of_iomap(node, 0);
  	if (!sun4i_irq_base)
-		panic("%s: unable to map IC registers\n",
-			node->full_name);
+		panic("%pOF: unable to map IC registers\n",
+			node);
 
  	/* Disable all interrupts */
  	writel(0, sun4i_irq_base + SUN4I_IRQ_ENABLE_REG(0));

@@ -124,7 +124,7 @@ static int __init sun4i_of_init(struct device_node *node,
  	sun4i_irq_domain = irq_domain_add_linear(node, 3 * 32,
  						 &sun4i_irq_ops, NULL);
  	if (!sun4i_irq_domain)
-		panic("%s: unable to create IRQ domain\n", node->full_name);
+		panic("%pOF: unable to create IRQ domain\n", node);
 
  	set_handle_irq(sun4i_handle_irq);
drivers/irqchip/irq-tegra.c +8 -8
index 3973a14bb15b..0abc0cd1c32e 100644
@@ -291,13 +291,13 @@ static int __init tegra_ictlr_init(struct device_node *node,
  	int err;
 
  	if (!parent) {
-		pr_err("%s: no parent, giving up\n", node->full_name);
+		pr_err("%pOF: no parent, giving up\n", node);
  		return -ENODEV;
  	}
 
  	parent_domain = irq_find_host(parent);
  	if (!parent_domain) {
-		pr_err("%s: unable to obtain parent domain\n", node->full_name);
+		pr_err("%pOF: unable to obtain parent domain\n", node);
  		return -ENXIO;
  	}
 

@@ -329,29 +329,29 @@ static int __init tegra_ictlr_init(struct device_node *node,
  	}
 
  	if (!num_ictlrs) {
-		pr_err("%s: no valid regions, giving up\n", node->full_name);
+		pr_err("%pOF: no valid regions, giving up\n", node);
  		err = -ENOMEM;
  		goto out_free;
  	}
 
  	WARN(num_ictlrs != soc->num_ictlrs,
-	     "%s: Found %u interrupt controllers in DT; expected %u.\n",
-	     node->full_name, num_ictlrs, soc->num_ictlrs);
+	     "%pOF: Found %u interrupt controllers in DT; expected %u.\n",
+	     node, num_ictlrs, soc->num_ictlrs);
 
 
  	domain = irq_domain_add_hierarchy(parent_domain, 0, num_ictlrs * 32,
  					  node, &tegra_ictlr_domain_ops,
  					  lic);
  	if (!domain) {
-		pr_err("%s: failed to allocated domain\n", node->full_name);
+		pr_err("%pOF: failed to allocated domain\n", node);
  		err = -ENOMEM;
  		goto out_unmap;
  	}
 
  	tegra_ictlr_syscore_init();
 
-	pr_info("%s: %d interrupts forwarded to %s\n",
-		node->full_name, num_ictlrs * 32, parent->full_name);
+	pr_info("%pOF: %d interrupts forwarded to %pOF\n",
+		node, num_ictlrs * 32, parent);
 
  	return 0;
drivers/irqchip/irq-xilinx-intc.c +2 -3
index 3db7ab1c9741..e3043ded8973 100644
@@ -186,8 +186,8 @@ static int __init xilinx_intc_of_init(struct device_node *intc,
  	if (irqc->intr_mask >> nr_irq)
  		pr_warn("irq-xilinx: mismatch in kind-of-intr param\n");
 
-	pr_info("irq-xilinx: %s: num_irq=%d, edge=0x%x\n",
-		intc->full_name, nr_irq, irqc->intr_mask);
+	pr_info("irq-xilinx: %pOF: num_irq=%d, edge=0x%x\n",
+		intc, nr_irq, irqc->intr_mask);
 
 
  	/*
--

2.11.0

maintainer Reviewed-by on v1 1/1
#5 lore raw
In reply to Rob Herring
Quoted 31 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
Quoted 12 lines
---
  drivers/irqchip/irq-bcm2835.c            |  9 ++++-----
  drivers/irqchip/irq-bcm2836.c            |  5 ++---
  drivers/irqchip/irq-crossbar.c           |  6 +++---
  drivers/irqchip/irq-digicolor.c          |  8 ++++----
  drivers/irqchip/irq-dw-apb-ictl.c        | 12 ++++++------
  drivers/irqchip/irq-gic-v3-its-pci-msi.c |  2 +-
  drivers/irqchip/irq-gic-v3-its.c         |  6 +++---
  drivers/irqchip/irq-gic-v3.c             | 11 ++++-------
  drivers/irqchip/irq-imx-gpcv2.c          |  4 ++--
  drivers/irqchip/irq-lpc32xx.c            |  2 +-
  drivers/irqchip/irq-mtk-sysirq.c         |  3 +--

For mediatek:

Reviewed-by Matthias Brugger <[email protected]>
Acked-by on v1 1/1
#6 lore raw
In reply to Rob Herring
Quoted 4 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

For bcm283x:

Acked-by Eric Anholt <[email protected]>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170719/78deb361/attachment.sig>

#7 lore raw
In reply to Rob Herring
Quoted 41 lines · Rob Herring

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node.

Signed-off-by Rob Herring <[email protected]>
Cc Thomas Gleixner <[email protected]>
Cc Jason Cooper <[email protected]>
Cc Marc Zyngier <[email protected]>
Cc Lee Jones <[email protected]>
Cc Eric Anholt <[email protected]>
Cc Stefan Wahren <[email protected]>
Cc Florian Fainelli <[email protected]>
Cc Ray Jui <[email protected]>
Cc Scott Branden <[email protected]>
Cc bcm-kernel-feedback-list at broadcom.com
Cc Baruch Siach <[email protected]>
Cc Vladimir Zapolskiy <[email protected]>
Cc Sylvain Lemieux <[email protected]>
Cc Matthias Brugger <[email protected]>
Cc Maxime Coquelin <[email protected]>
Cc Alexandre Torgue <[email protected]>
Cc Maxime Ripard <[email protected]>
Cc Chen-Yu Tsai <[email protected]>
Cc Thierry Reding <[email protected]>
Cc Jonathan Hunter <[email protected]>
Cc Michal Simek <[email protected]>
Cc "S?ren Brinkmann" <[email protected]>
Cc linux-rpi-kernel at lists.infradead.org
Cc linux-arm-kernel at lists.infradead.org
Cc linux-mediatek at lists.infradead.org
Cc linux-tegra at vger.kernel.org
---
 drivers/irqchip/irq-bcm2835.c            |  9 ++++-----
 drivers/irqchip/irq-bcm2836.c            |  5 ++---
 drivers/irqchip/irq-crossbar.c           |  6 +++---
 drivers/irqchip/irq-digicolor.c          |  8 ++++----
 drivers/irqchip/irq-dw-apb-ictl.c        | 12 ++++++------
 drivers/irqchip/irq-gic-v3-its-pci-msi.c |  2 +-
 drivers/irqchip/irq-gic-v3-its.c         |  6 +++---
 drivers/irqchip/irq-gic-v3.c             | 11 ++++-------
 drivers/irqchip/irq-imx-gpcv2.c          |  4 ++--
Quoted 8 lines
 drivers/irqchip/irq-lpc32xx.c            |  2 +-
 drivers/irqchip/irq-mtk-sysirq.c         |  3 +--
 drivers/irqchip/irq-mxs.c                |  4 ++--
 drivers/irqchip/irq-stm32-exti.c         |  8 ++++----
 drivers/irqchip/irq-sun4i.c              |  6 +++---
 drivers/irqchip/irq-tegra.c              | 16 ++++++++--------
 drivers/irqchip/irq-xilinx-intc.c        |  4 ++--
 16 files changed, 50 insertions(+), 56 deletions(-)

Queued for 4.14.

Thanks,

	M.
Signature · 1 line
Jazz is not dead. It just smells funny...