[...] >>> +int of_genpd_parse_idle_states(struct device_node *dn, >>> + struct genpd_power_state **states, int *n) >> >> >> Instead of taking **states as a parameter, let's instead return it as >> a pointer for the allocated struct. In case of failures, let's return >> ERR_PTR(). >> > Hmm.. I thought about it. There are 2 return values from this function. > If we return a pointer to the allocated memory, we still have to return > the size of it as an argument. I wasn't happy splitting the return > values in 2 different places. Ahh, right. So let's keep it as is! [...] Kind regards Uffe -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
