Commit d342c060 authored Mar 08, 2016 by David Kershner Committed by Greg Kroah-Hartman Mar 10, 2016

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>

parent 5bf49a6c

drivers/staging/unisys/visorbus/visorchannel.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -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;
		}

Admin message