On 11/12/2014, 10:04 AM, Kevin Cernekee wrote: >> This should actually return bool and use true/false. > > Well, the other APIs currently return an int: > > extern int of_device_is_compatible(const struct device_node *device, > const char *); > extern int of_device_is_available(const struct device_node *device); > [...] > extern int of_machine_is_compatible(const char *compat); > > Do you think it is best to change all of them at once, or just the > newly introduced function? Possibly fix all these in a separate patch and then add the new one fixed :). -- js suse labs -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html