From: Dan Carpenter Date: Fri, 20 Jan 2012 07:57:14 +0000 (+0300) Subject: ACPI, APEI, EINJ, cleanup 0 vs NULL confusion X-Git-Tag: upstream/snapshot3+hdmi~8254^2~2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=29924b9f8f39e37275cff93740835d28b9e6fb36;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git 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 Acked-by: Huang Ying Signed-off-by: Len Brown --- 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 */