On 05/21/2015 12:49 AM, Viresh Kumar wrote: > On 21-05-15, 00:32, Nishanth Menon wrote: >> Why not just mark it as a property of an OPP rather than a phandle? just >> thinking that this might setup a precedence for future needs like >> "shutdown OPP" or "Reboot OPP" or something different that some other >> SoC might have.. > > AFAIU, a property should be present in the OPP if and only if any OPP > can set it. But in this case, only one OPP from the entire list can > set it. So, its more a property of the list rather than every OPP > within it. And then there wouldn't be any need to code that would > check bugs in dtbs where multiple OPPs have it set. > True.. fair enough. Acked-by: Nishanth Menon <nm@xxxxxx> -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html