The unconditioned jump will prohibit to ever reach the break-statement. Deleting this needless statement, the code becomes more understandable. Signed-off-by: Fabian Krueger <fabian.krueger@xxxxxx> Signed-off-by: Michael Scheiderer <michael.scheiderer@xxxxxx> --- drivers/staging/kpc2000/kpc2000_spi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index f258f369e065..e65f0c34db50 100644 --- a/drivers/staging/kpc2000/kpc2000_spi.c +++ b/drivers/staging/kpc2000/kpc2000_spi.c @@ -515,7 +515,6 @@ kp_spi_probe(struct platform_device *pldev) default: dev_err(&pldev->dev, "Unknown hardware, cant know what partition table to use!\n"); goto free_master; - break; } return status; -- 2.17.1 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel