On Mon, Apr 24, 2017 at 12:19:18PM +0530, Rahul Bedarkar wrote: > Hello Guenter, Heiko, > > On Mon, Apr 24, 2017 at 9:28 AM, Heiko Schocher <hs@xxxxxxx> wrote: > > Hello Guenter, Rahul, > > > > Hmm.. I like this patch too, but have also no idea, what is preffered. > > > > Looking into drivers/hwmon > > > > pollux:linux hs [master] $ grep -lr __maybe_unused drivers/hwmon/ > > drivers/hwmon/tmp108.c > > drivers/hwmon/nct6775.c > > drivers/hwmon/hwmon-vid.c > > drivers/hwmon/max31722.c > > > > Ok, there are hwmon drivers, which use this version already ... > > Yes, some hwmon drivers already use this approach. Some drivers in > other sub systems also using it from start or moving towards this > approach. > Yes, but as I mentioned it is unsettled if one or the other approach is preferred, which makes me a bit wary. I'll be open to accepting patches for jc42 and nct6883 since I am the author of those drivers. Thanks, Guenter > > > > pollux:linux hs [master] $ grep -lr CONFIG_PM drivers/hwmon/ > > drivers/hwmon/max6639.c > > drivers/hwmon/jc42.c > > drivers/hwmon/fam15h_power.c > > drivers/hwmon/tmp102.c > > drivers/hwmon/gpio-fan.c > > drivers/hwmon/pwm-fan.c > > drivers/hwmon/tmp103.c > > drivers/hwmon/pmbus/Makefile > > drivers/hwmon/lm75.c > > drivers/hwmon/nct6683.c > > drivers/hwmon/adt7x10.h > > drivers/hwmon/w83627hf.c > > drivers/hwmon/abituguru3.c > > drivers/hwmon/Makefile > > drivers/hwmon/acpi_power_meter.c > > drivers/hwmon/adt7x10.c > > drivers/hwmon/abituguru.c > > drivers/hwmon/w83627ehf.c > > > > May this conversion should be done in a patch, which touches more > > devices? > > I'm happy send patches converting remaining drivers once this is > settled or accepted. > > Thanks for your reviews. > > Regards, > Rahul -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html