Re: [PATCH] Input-polldev:Check poll_interval range while registering input device

On Thu, Jul 16, 2015 at 7:57 PM, Anshul Garg [off-list ref] wrote:

+       if (dev->poll_interval > dev->poll_interval_max ||
+                       dev->poll_interval < dev->poll_interval_min) {
+               pr_err("Poll interval Range incorrect\n");
+               return -EINVAL;
+       }

        if (!dev->poll_interval)
                dev->poll_interval = 500;

So priviously, if poll_interval was 0 you got a sane default of 500,
and now you get -EINVAL.

Is this really useful and correct, and nobody using it is passing 0?

regards
Marcus