On 03/01/2015 08:27 AM, Sascha Hauer wrote: >> -int of_property_read_string(struct device_node *np, const char *propname, >> +int of_property_read_string(const struct device_node *np, const char *propname, >> const char **out_string) > > Is this necessary for something or just a nice to have? The functions > are takes from the kernel and the device_node arguments cannot be const > there because the functions take a reference on the node. I see, I'll remove the patch and adopt my code. Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |