On Thu 2013-05-09 10:09:48, Guenter Roeck wrote: > The following patch series introduces a marker for power management functions > and data. This this marker, #ifdef CONFIG_PM and #ifdef CONFIG_PM_SLEEP > can be removed from most of the code. This ensures that the conditional code > still compiles but is not included in the object file. Was it compile-tested for both PM_SLEEP and !PM_SLEEP cases? If driver wants to have pm-specific fields in its structures, this will make it harder. Ifdefs are ugly, and this has better compiler coverage in !PM_SLEEP case. Good. OTOH most people are running with PM_SLEEP, so advantage is not great... Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
