diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-01-20 02:57:14 -0500 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2012-01-23 19:38:52 -0500 |
commit | 29924b9f8f39e37275cff93740835d28b9e6fb36 (patch) | |
tree | 9e825881abbad761bef2b8cfba7731a2a1ef47cd /drivers/acpi | |
parent | 4c40aed869a200a621b53bcb491c5ee8a34ef5f1 (diff) |
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) | |||
259 | 259 | ||
260 | v4param = ioremap(paddrv4, sizeof(*v4param)); | 260 | v4param = ioremap(paddrv4, sizeof(*v4param)); |
261 | if (!v4param) | 261 | if (!v4param) |
262 | return 0; | 262 | return NULL; |
263 | if (readq(&v4param->reserved1) || readq(&v4param->reserved2)) { | 263 | if (readq(&v4param->reserved1) || readq(&v4param->reserved2)) { |
264 | iounmap(v4param); | 264 | iounmap(v4param); |
265 | return 0; | 265 | return NULL; |
266 | } | 266 | } |
267 | return v4param; | 267 | return v4param; |
268 | } | 268 | } |
269 | 269 | ||
270 | return 0; | 270 | return NULL; |
271 | } | 271 | } |
272 | 272 | ||
273 | /* do sanity check to trigger table */ | 273 | /* do sanity check to trigger table */ |