On Wed, Sep 16, 2015 at 02:54:53PM -0500, Andreas Dannenberg wrote: > On Wed, Sep 16, 2015 at 05:10:06PM +0900, Krzysztof Kozlowski wrote: > > On 16.09.2015 02:58, Andreas Dannenberg wrote: > > > static DEVICE_ATTR(ovp_voltage, S_IRUGO, bq24257_show_ovp_voltage, NULL); > > > static DEVICE_ATTR(in_dpm_voltage, S_IRUGO, bq24257_show_in_dpm_voltage, NULL); > > > > This applies to previous patches actually: DEVICE_ATTR_RO? > > Ok. Will simplify. Actually I looked into this more and realized that the use of the pre-configured DEVICE_ATTR_* definitions imposes certain function names, which would mean renaming the functions including dropping the bq24257_ prefix those function names have, making the code inconsistent and decreasing the unique-ness of those function names in the global Kernel namespace. So I rather leave the more general DEVICE_ATTR() definitions in place. Regards, -- Andreas Dannenberg Texas Instruments Inc -- 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