[PATCH 21/30] staging: unisys: remove unnecessary goto in parser_param_start()

Benjamin Romer benjamin.romer at unisys.com
Tue Feb 10 17:58:55 UTC 2015


The goto isn't necessary here so replace it with a simple return
statement.

Signed-off-by: Benjamin Romer <benjamin.romer at unisys.com>
---
 drivers/staging/unisys/visorchipset/parser.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/staging/unisys/visorchipset/parser.c b/drivers/staging/unisys/visorchipset/parser.c
index 2f997af..3678d39 100644
--- a/drivers/staging/unisys/visorchipset/parser.c
+++ b/drivers/staging/unisys/visorchipset/parser.c
@@ -217,7 +217,7 @@ parser_param_start(PARSER_CONTEXT *ctx, PARSER_WHICH_STRING which_string)
 	if (ctx == NULL) {
 		ERRDRV("%s (%s:%d) - no context",
 		       __func__, __FILE__, __LINE__);
-		goto Away;
+		return;
 	}
 	phdr = (struct spar_controlvm_parameters_header *)(ctx->data);
 	switch (which_string) {
@@ -241,9 +241,6 @@ parser_param_start(PARSER_CONTEXT *ctx, PARSER_WHICH_STRING which_string)
 		ERRDRV("%s - bad which_string %d", __func__, which_string);
 		break;
 	}
-
-Away:
-	return;
 }
 
 void
-- 
2.1.0



More information about the devel mailing list