diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-01-20 10:57:14 +0300 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2012-01-23 19:38:52 -0500 |
| commit | 29924b9f8f39e37275cff93740835d28b9e6fb36 (patch) | |
| tree | 9e825881abbad761bef2b8cfba7731a2a1ef47cd /drivers/acpi | |
| parent | 4c40aed869a200a621b53bcb491c5ee8a34ef5f1 (diff) | |
| download | linux-29924b9f8f39e37275cff93740835d28b9e6fb36.tar.bz2 | |
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 bb20b03ddc39..c89b0e5a2293 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 */ |