Hi! On 18/01/17 22:33, Frank Rowand wrote: >> This is necessary for for_each_of_allnodes() to work in the modules. >> >> Signed-off-by: Alexander Sverdlin <alexander.sverdlin@xxxxxxxxx> >> Cc: Rob Herring <robh+dt@xxxxxxxxxx> >> Cc: Frank Rowand <frowand.list@xxxxxxxxx> >> Cc: devicetree@xxxxxxxxxxxxxxx >> --- >> drivers/of/base.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/of/base.c b/drivers/of/base.c >> index 2eb4dea62b84..f576c33e0b84 100644 >> --- a/drivers/of/base.c >> +++ b/drivers/of/base.c >> @@ -270,6 +270,7 @@ struct device_node *__of_find_all_nodes(struct device_node *prev) >> } >> return np; >> } >> +EXPORT_SYMBOL_GPL(__of_find_all_nodes); >> >> /** >> * of_find_all_nodes - Get next node in global list >> > Quoting Rob from the last request for this: > > "What driver needs this? This isn't really a function I'd expect drivers to use." This one and the one from previous patch/mail are used in a module which reads DT overlays from pluggable extension boards, eventually performs some sanity checks/corrections and applies these overlays. This eventual corrections require low-level access to DT. I cannot judge, how relevant this usecase is for the community, but I was driven by the fact, that the functions which require the functions I export are defined statically in .h files and therefore are exposed to the modules. -- Best regards, Alexander Sverdlin. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
