ACPI, APEI, EINJ, cleanup 0 vs NULL confusion
Dan Carpenter [Fri, 20 Jan 2012 07:57:14 +0000 (10:57 +0300)]
This function is returning pointers.  Sparse complains here:
drivers/acpi/apei/einj.c:262:32: warning:
Using plain integer as NULL pointer

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Huang Ying <ying.huang@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>

drivers/acpi/apei/einj.c

index bb20b03..c89b0e5 100644 (file)
@@ -259,15 +259,15 @@ static void *einj_get_parameter_address(void)
 
                v4param = ioremap(paddrv4, sizeof(*v4param));
                if (!v4param)
-                       return 0;
+                       return NULL;
                if (readq(&v4param->reserved1) || readq(&v4param->reserved2)) {
                        iounmap(v4param);
-                       return 0;
+                       return NULL;
                }
                return v4param;
        }
 
-       return 0;
+       return NULL;
 }
 
 /* do sanity check to trigger table */