this is more clear.
Signed-off-by: Denis Cheng <crquan@gmail.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
 
 static unsigned int dev_poll(struct file *file, poll_table *wait)
 {
+       unsigned int mask = 0;
+
        poll_wait(file, &send_wq, wait);
 
        spin_lock(&ops_lock);
-       if (!list_empty(&send_list)) {
-               spin_unlock(&ops_lock);
-               return POLLIN | POLLRDNORM;
-       }
+       if (!list_empty(&send_list))
+               mask = POLLIN | POLLRDNORM;
        spin_unlock(&ops_lock);
-       return 0;
+
+       return mask;
 }
 
 static const struct file_operations dev_fops = {