diff options
| author | David Kershner <david.kershner@unisys.com> | 2016-03-08 21:28:28 -0500 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-03-10 19:27:39 -0800 |
| commit | d342c060433a0f5b6b710cf2ab2591576c2f2d62 (patch) | |
| tree | b7242f2d8363e6503bb1d70cb92d1fce99ac36f6 /drivers/staging/unisys | |
| parent | 5bf49a6c74f4221e7ede7c741d3506518728471e (diff) | |
| download | linux-d342c060433a0f5b6b710cf2ab2591576c2f2d62.tar.bz2 | |
staging: unisys: visorbus: clean up visorchannel_clear
Goto label was ambiguous change to out_free_page to signify that
we will always go through the path even on success.
Signed-off-by: David Kershner <david.kershner@unisys.com>
Signed-off-by: Timothy Sell <timothy.sell@unisys.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/unisys')
| -rw-r--r-- | drivers/staging/unisys/visorbus/visorchannel.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/unisys/visorbus/visorchannel.c b/drivers/staging/unisys/visorbus/visorchannel.c index ba26b5443211..b68a904ac617 100644 --- a/drivers/staging/unisys/visorbus/visorchannel.c +++ b/drivers/staging/unisys/visorbus/visorchannel.c @@ -293,14 +293,14 @@ visorchannel_clear(struct visorchannel *channel, ulong offset, u8 ch, err = visorchannel_write(channel, offset + written, buf, thisbytes); if (err) - goto cleanup; + goto out_free_page; written += thisbytes; nbytes -= thisbytes; } err = 0; -cleanup: +out_free_page: free_page((unsigned long)buf); return err; } |