On Tue, Apr 02, 2013 at 01:40:03PM +0200, Nicolas Kaiser wrote: > I noticed an assignment that gets immediately overwritten, contradicting > a nearby comment. > According to the comment, 'optval' should get disabled half of the time. > > As at the time of the conditional assignment optval was already set to > different nonzero values, it looks to me like the trailing assignment, > which was introduced in commit 737e1a71, may be unintended. Good catch. Applied. thanks, Dave -- To unsubscribe from this list: send the line "unsubscribe trinity" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html