The source code defines a macro NULLFUNC

  #define NULLFUNC 0

and uses it as a generic null function pointer constant.  This is
superfluous.  Just use NULL instead.

Signed-off-by: Ian Abbott <[email protected]>
---
 drivers/staging/comedi/drivers/amplc_pci224.c |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/comedi/drivers/amplc_pci224.c 
b/drivers/staging/comedi/drivers/amplc_pci224.c
index 43f07f8..a1e23ba 100644
--- a/drivers/staging/comedi/drivers/amplc_pci224.c
+++ b/drivers/staging/comedi/drivers/amplc_pci224.c
@@ -265,9 +265,6 @@ Caveats:
 /* Combine old and new bits. */
 #define COMBINE(old, new, mask)        (((old) & ~(mask)) | ((new) & (mask)))
 
-/* A generic null function pointer value.  */
-#define NULLFUNC       0
-
 /* Current CPU.  XXX should this be hard_smp_processor_id()? */
 #define THISCPU                smp_processor_id()
 
@@ -703,7 +700,7 @@ pci224_ao_inttrig_start(struct comedi_device *dev, struct 
comedi_subdevice *s,
        if (trignum != 0)
                return -EINVAL;
 
-       s->async->inttrig = NULLFUNC;
+       s->async->inttrig = NULL;
        pci224_ao_start(dev, s);
 
        return 1;
-- 
1.7.8.6

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to