EDAC/amd64: Remove unneeded breaks
authorTom Rix <trix@redhat.com>
Mon, 19 Oct 2020 19:35:24 +0000 (12:35 -0700)
committerBorislav Petkov <bp@suse.de>
Mon, 26 Oct 2020 11:07:50 +0000 (12:07 +0100)
A break is not needed if it is preceded by a return.

Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Reviewed-by: Robert Richter <rric@kernel.org>
Link: https://lkml.kernel.org/r/20201019193524.13391-1-trix@redhat.com
drivers/edac/amd64_edac.c

index 1362274..3bac76e 100644 (file)
@@ -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;
 }