Scott Wood writes: > +static inline void *find_node_by_prop_value_str(const void *prev, > + const char *propname, > + const char *propval) That name sure is a mouthful. Since we mostly (only?) use it with propname == "device_type", why not make a helper with a nice, (relatively) short name like "find_node_by_type" for that case? Paul.