diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2017-08-18 02:35:33 -0400 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2017-08-29 10:29:02 -0400 |
commit | eb304e800d491d5168df61a999beebe8042e7e58 (patch) | |
tree | 276b135b5ce0f4ec9fc1e53eaf142e07e3f4e91f | |
parent | 41b0dbfac0da515ad62edf9256414c56ee217364 (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.c | 5 |
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); | ||
96 | out: | ||
97 | session->response = NULL; | 96 | session->response = NULL; |
98 | } | 97 | } |
99 | 98 | ||