diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-01-20 07:57:14 (GMT) |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2012-01-24 00:38:52 (GMT) |
commit | 29924b9f8f39e37275cff93740835d28b9e6fb36 (patch) | |
tree | 9e825881abbad761bef2b8cfba7731a2a1ef47cd /drivers/acpi | |
parent | 4c40aed869a200a621b53bcb491c5ee8a34ef5f1 (diff) | |
download | linux-29924b9f8f39e37275cff93740835d28b9e6fb36.tar.xz |
ACPI, APEI, EINJ, cleanup 0 vs NULL confusion
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>
Diffstat (limited to 'drivers/acpi')
-rw-r--r-- | drivers/acpi/apei/einj.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/acpi/apei/einj.c b/drivers/acpi/apei/einj.c index bb20b03..c89b0e5 100644 --- a/drivers/acpi/apei/einj.c +++ b/drivers/acpi/apei/einj.c @@ -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 */ |