H Hartley Sweeten wrote: >> + for (wchan = 0; wchan < pwm->nchan; wchan++) { >> + if (pwm->channels[wchan].flags & FLAG_REQUESTED) { >> > > Shouldn't this be: > > & BIT(FLAG_REQUESTED) > > Yes. I've fixed that one several times now! I think I need to review my git-fu, especially rebasing. > Nitpick... Can you move the #ifndef/#define to after the comment? > I did that in all the others, I guess I missed this one... b.g. -- Bill Gatliff bgat@xxxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-embedded" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html