summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHeiko Carstens <heiko.carstens@de.ibm.com>2017-08-18 02:35:33 -0400
committerMartin Schwidefsky <schwidefsky@de.ibm.com>2017-08-29 10:29:02 -0400
commiteb304e800d491d5168df61a999beebe8042e7e58 (patch)
tree276b135b5ce0f4ec9fc1e53eaf142e07e3f4e91f
parent41b0dbfac0da515ad62edf9256414c56ee217364 (diff)
s390/vmcp: simplify vmcp_response_free()
Get rid of the goto and "out" label within vmcp_response_free() which I added. This just makes the code harder to read than necessary. Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
-rw-r--r--drivers/s390/char/vmcp.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/s390/char/vmcp.c b/drivers/s390/char/vmcp.c
index 0aa50afa5063..7898bbcc28fc 100644
--- a/drivers/s390/char/vmcp.c
+++ b/drivers/s390/char/vmcp.c
@@ -90,10 +90,9 @@ static void vmcp_response_free(struct vmcp_session *session)
90 page = phys_to_page((unsigned long)session->response); 90 page = phys_to_page((unsigned long)session->response);
91 cma_release(vmcp_cma, page, nr_pages); 91 cma_release(vmcp_cma, page, nr_pages);
92 session->cma_alloc = 0; 92 session->cma_alloc = 0;
93 goto out; 93 } else {
94 free_pages((unsigned long)session->response, order);
94 } 95 }
95 free_pages((unsigned long)session->response, order);
96out:
97 session->response = NULL; 96 session->response = NULL;
98} 97}
99 98