On Mon, Nov 10, 2014 at 8:47 PM, al.stone@linaro.org wrote:
+int fwnode_property_read_string_array(struct fwnode_handle *fwnode,
const char *propname, char **val,
size_t nval)
+{
if (is_of_node(fwnode))
return of_property_read_string_array(of_node(fwnode), propname,
val, nval);
I figured someone would have noticed this already, but I don't see any posts on the subject. The above code causes this build warning:
drivers/base/property.c: In function ‘fwnode_property_read_string_array’: drivers/base/property.c:561:12: warning: passing argument 3 of ‘of_property_read_string_array’ from incompatible pointer type [enabled by default] val, nval); ^ In file included from drivers/base/property.c:16:0: include/linux/of.h:748:19: note: expected ‘const char **’ but argument is of type ‘char **’ static inline int of_property_read_string_array(struct device_node *np, ^
The 'val' parameter is "char **", but of_property_read_string_array() has this prototype:
static inline int of_property_read_string_array(struct device_node *np, const char *propname, const char **out_strs, size_t sz)