Trivial changes from review of last PR.

arch/arm/src/stm32/stm32_comp.c

@@ -87,6 +87,7 @@
 /****************************************************************************
  * Pre-processor Definitions
  ****************************************************************************/
+
 /* COMP2 default configuration **********************************************/
 
 #ifdef CONFIG_STM32_COMP2

configs/nucleo-f334r8/src/stm32_comp.c

@@ -86,6 +86,8 @@ int stm32_comp_setup(void)
 
   if (!initialized)
     {
+      /* Get the comparator interface */
+
 #ifdef CONFIG_STM32_COMP2
       comp = stm32_compinitialize(2);
       if (comp == NULL)
@@ -113,6 +115,8 @@ int stm32_comp_setup(void)
         }
 #endif
 
+      /* Register the comparator character driver at /dev/comp0 */
+
       ret = comp_register("/dev/comp0", comp);
       if (ret < 0)
         {

drivers/analog/Kconfig

@@ -110,12 +110,11 @@ endif # if ADC_PGA11X
 endif # ADC
 
 config COMP
-       bool "Analog Comparator"
+	bool "Analog Comparator"
-       default n
+	default n
-       ---help---
+	---help---
 		Select to enable support for Analog Comparators (COMPs).
 
-
 config DAC
 	bool "Digital-to-Analog Conversion"
 	default n

drivers/analog/Make.defs

@@ -1,7 +1,7 @@
 ############################################################################
 # drivers/analog/Make.defs
 #
-#   Copyright (C) 2011-2012 Gregory Nutt. All rights reserved.
+#   Copyright (C) 2011-2012, 2017 Gregory Nutt. All rights reserved.
 #   Author: Gregory Nutt <gnutt@nuttx.org>
 #
 # Redistribution and use in source and binary forms, with or without
@@ -104,4 +104,3 @@ ifeq ($(CONFIG_COMP),y)
 endif
 endif
 endif
-

drivers/analog/comp.c

@@ -77,7 +77,10 @@ static const struct file_operations comp_fops =
   NULL,                         /* seek */
   comp_ioctl                    /* ioctl */
 #ifndef CONFIG_DISABLE_POLL
-  , 0
+  , NULL                        /* poll */
+#endif
+#ifndef CONFIG_DISABLE_PSEUDOFS_OPERATIONS
+  , NULL                        /* unlink */
 #endif
 };