smatch complains about
net/iucv/iucv.c:1119 __iucv_message_receive() warn: inconsistent indenting

While touching this line, also make the return logic consistent and thus
get rid of a goto label.

Signed-off-by: Julian Wiedmann <j...@linux.ibm.com>
Reviewed-by: Karsten Graul <kgr...@linux.ibm.com>
---
 net/iucv/iucv.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/net/iucv/iucv.c b/net/iucv/iucv.c
index cd2e468852e7..349c6ac3313f 100644
--- a/net/iucv/iucv.c
+++ b/net/iucv/iucv.c
@@ -1116,10 +1116,9 @@ int __iucv_message_receive(struct iucv_path *path, 
struct iucv_message *msg,
        if (msg->flags & IUCV_IPRMDATA)
                return iucv_message_receive_iprmdata(path, msg, flags,
                                                     buffer, size, residual);
-        if (cpumask_empty(&iucv_buffer_cpumask)) {
-               rc = -EIO;
-               goto out;
-       }
+       if (cpumask_empty(&iucv_buffer_cpumask))
+               return -EIO;
+
        parm = iucv_param[smp_processor_id()];
        memset(parm, 0, sizeof(union iucv_param));
        parm->db.ipbfadr1 = (u32)(addr_t) buffer;
@@ -1135,7 +1134,6 @@ int __iucv_message_receive(struct iucv_path *path, struct 
iucv_message *msg,
                if (residual)
                        *residual = parm->db.ipbfln1f;
        }
-out:
        return rc;
 }
 EXPORT_SYMBOL(__iucv_message_receive);
-- 
2.17.1

Reply via email to