On Wed, Sep 27, 2017 at 01:14:38PM -0400, David Kershner wrote: > + if (chipset_dev->poll_jiffies != POLLJIFFIES_CONTROLVM_SLOW) > + chipset_dev->poll_jiffies = POLLJIFFIES_CONTROLVM_SLOW; > + } else { > + if (chipset_dev->poll_jiffies != POLLJIFFIES_CONTROLVM_FAST) > + chipset_dev->poll_jiffies = POLLJIFFIES_CONTROLVM_FAST; If you want, in a later patch you could just remove the conditions here and say: chipset_dev->poll_jiffies = POLLJIFFIES_CONTROLVM_SLOW; } else { chipset_dev->poll_jiffies = POLLJIFFIES_CONTROLVM_FAST; } Btw, I'm loving this patchset. Thank you! regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel