powerpc/scom: Return NULL instead of 0
authorBreno Leitao <leitao@debian.org>
Mon, 22 Oct 2018 14:54:18 +0000 (11:54 -0300)
committerMichael Ellerman <mpe@ellerman.id.au>
Sun, 25 Nov 2018 06:11:21 +0000 (17:11 +1100)
Function scom_map_device() returns data type 'scom_map_t', which is a
typedef for 'void *'. This functions is currently returning NULL and zero,
which causes the following warning by 'sparse':

arch/powerpc/sysdev/scom.c:63:24: warning: Using plain integer as NULL pointer
arch/powerpc/sysdev/scom.c:86:24: warning: Using plain integer as NULL pointer

This patch simply replaces zero by NULL.

Signed-off-by: Breno Leitao <leitao@debian.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/sysdev/scom.c

index 0f6fd5d04d332372aa74d94caf6d17fce3402559..a707b24a7ddb665f3b8e12bac9206de62a6527d4 100644 (file)
@@ -60,7 +60,7 @@ scom_map_t scom_map_device(struct device_node *dev, int index)
        parent = scom_find_parent(dev);
 
        if (parent == NULL)
-               return 0;
+               return NULL;
 
        /*
         * We support "scom-reg" properties for adding scom registers
@@ -83,7 +83,7 @@ scom_map_t scom_map_device(struct device_node *dev, int index)
        size >>= 2;
 
        if (index >= (size / (2*cells)))
-               return 0;
+               return NULL;
 
        reg = of_read_number(&prop[index * cells * 2], cells);
        cnt = of_read_number(&prop[index * cells * 2 + cells], cells);