On Mon, Nov 24, 2014 at 01:58:50PM +0100, Romain Perier wrote: > 2014-11-24 12:35 GMT+01:00 Johan Hovold <johan@xxxxxxxxxx>: > > You should also fix the commit message, which claims to define a "helper > > function which is compatible with both properties", something which was > > no longer the case. > > > So I need to resend this patch which would only introduce an helper > function which checks for "system-power-controller" property (it would > also have a new commit message). > As you suggested, this helper might be named > "of_device_is_system_power_controller" ? Correct. Rename the helper and property name you introduced to "of_device_is_system_power_controller" and "system-power-controller" and update the act8865 driver to use the new names (all in one patch). Send the patch to Mark (and keep everyone else one CC) so he can apply it to the regulator tree. Other drivers that are currently using "ti,system-power-controller" can continue to do so (for backward compatibility), while we later also add a call to your helper function (and update binding documentation) and also update the device trees for boards that use this functionality. Thanks, Johan -- 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
