Hi! > This patch merges the fb_powerup and fb_powerdown hooks in a single > operation fb_power with an additional "state" parameter ranging > from 0 (running) to 3 (poweroff).
No, please don't. Having int with 0/3 being valid specifying what to do is evil. enum is less evil, but what's wrong about having 3 functions? -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
