of: address: Allow to specify nonposted-mmio per-device

quic-kdybcio · robherring · commit 47026c4ffedd · 2025-03-21T11:27:56.000-05:00
Certain IP blocks may strictly require/expect a nE mapping to function correctly, while others may be fine without it (which is preferred for performance reasons). Allow specifying nonposted-mmio on a per-device basis. Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Link: https://lore.kernel.org/r/20250319-topic-nonposted_mmio-v1-2-dfb886fbd15f@oss.qualcomm.com Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
diff --git a/drivers/of/address.c b/drivers/of/address.c
@@ -1032,10 +1032,11 @@ EXPORT_SYMBOL_GPL(of_dma_is_coherent);
 static bool of_mmio_is_nonposted(const struct device_node *np)
 {
 	struct device_node *parent __free(device_node) = of_get_parent(np);
-	if (!parent)
-		return false;
 
-	return of_property_read_bool(parent, "nonposted-mmio");
+	if (of_property_read_bool(np, "nonposted-mmio"))
+		return true;
+
+	return parent && of_property_read_bool(parent, "nonposted-mmio");
 }
 
 static int __of_address_to_resource(struct device_node *dev, int index, int bar_no,

Original file line number	Diff line number	Diff line change
`@@ -1032,10 +1032,11 @@ EXPORT_SYMBOL_GPL(of_dma_is_coherent);`
`1032`	`1032`	`static bool of_mmio_is_nonposted(const struct device_node *np)`
`1033`	`1033`	`{`
`1034`	`1034`	`struct device_node *parent __free(device_node) = of_get_parent(np);`
`1035`		`- if (!parent)`
`1036`		`- return false;`
`1037`	`1035`
`1038`		`- return of_property_read_bool(parent, "nonposted-mmio");`
	`1036`	`+ if (of_property_read_bool(np, "nonposted-mmio"))`
	`1037`	`+ return true;`
	`1038`	`+`
	`1039`	`+ return parent && of_property_read_bool(parent, "nonposted-mmio");`
`1039`	`1040`	`}`
`1040`	`1041`
`1041`	`1042`	`static int __of_address_to_resource(struct device_node *dev, int index, int bar_no,`