Commit f09056c1 authored by Tom Rix's avatar Tom Rix Committed by Borislav Petkov
Browse files

EDAC/amd64: Remove unneeded breaks 



A break is not needed if it is preceded by a return.

Signed-off-by: default avatarTom Rix <trix@redhat.com>
Signed-off-by: default avatarBorislav Petkov <bp@suse.de>
Reviewed-by: default avatarRobert Richter <rric@kernel.org>
Link: https://lkml.kernel.org/r/20201019193524.13391-1-trix@redhat.com
parent 3650b228

drivers/edac/amd64_edac.c

+0 −8
Original line number Diff line number Diff line
@@ -2461,14 +2461,11 @@ static int map_err_sym_to_channel(int err_sym, int sym_size) 
		case 0x20:

		case 0x21:

			return 0;

			break;

		case 0x22:

		case 0x23:

			return 1;

			break;

		default:

			return err_sym >> 4;

			break;

		}

	/* x8 symbols */

	else
@@ -2478,17 +2475,12 @@ static int map_err_sym_to_channel(int err_sym, int sym_size) 
			WARN(1, KERN_ERR "Invalid error symbol: 0x%x\n",

					  err_sym);

			return -1;

			break;



		case 0x11:

			return 0;

			break;

		case 0x12:

			return 1;

			break;

		default:

			return err_sym >> 3;

			break;

		}

	return -1;

}