dev_type_poll() functions return POLL* value or 0 if nothing is
supported.  Diff below fixes the cdev_ipmi_init() definition by
passing selfalse instead of returning ENODEV in a poll function.

While here make comments match reality, ok?

Index: sys/conf.h
===================================================================
RCS file: /cvs/src/sys/sys/conf.h,v
retrieving revision 1.149
diff -u -p -r1.149 conf.h
--- sys/conf.h  3 Apr 2020 13:31:11 -0000       1.149
+++ sys/conf.h  16 Apr 2020 07:55:09 -0000
@@ -460,6 +460,7 @@ extern struct cdevsw cdevsw[];
        (dev_type_stop((*))) enodev, 0, dev_init(c,n,poll), \
        (dev_type_mmap((*))) enodev, 0, D_CLONE, dev_init(c,n,kqfilter) }
 
+/* open, close, ioctl */
 #define cdev_pvbus_init(c,n) { \
        dev_init(c,n,open), dev_init(c,n,close), \
        (dev_type_read((*))) enodev, \
@@ -468,11 +469,11 @@ extern struct cdevsw cdevsw[];
        (dev_type_stop((*))) enodev, 0, selfalse, \
        (dev_type_mmap((*))) enodev }
 
-/* open, close, read, write, poll, ioctl, nokqfilter */
+/* open, close, ioctl */
 #define cdev_ipmi_init(c,n) { \
        dev_init(c,n,open), dev_init(c,n,close), (dev_type_read((*))) enodev, \
        (dev_type_write((*))) enodev, dev_init(c,n,ioctl), \
-       (dev_type_stop((*))) enodev, 0, (dev_type_poll((*))) enodev, \
+       (dev_type_stop((*))) enodev, 0, selfalse, \
        (dev_type_mmap((*))) enodev, 0 }
 
 /* open, close, ioctl, mmap */

Reply via email to