| Message ID | 20201026091103.67780-1-zhangqilong3@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [-next] EDAC/amd64: Discard unnecessary break | expand |
On Mon, Oct 26, 2020 at 05:11:03PM +0800, Zhang Qilong wrote: > A 'break' is unnecessary because of previous 'return'. > > Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com> > --- > drivers/edac/amd64_edac.c | 7 ------- > 1 file changed, 7 deletions(-) https://git.kernel.org/pub/scm/linux/kernel/git/ras/ras.git/commit/?h=edac-misc&id=f09056c1decec889721c24c72f99730376918044 Thx.
diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index 1362274d840b..a1c9e68b35d9 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -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,13 @@ 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; }
A 'break' is unnecessary because of previous 'return'. Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com> --- drivers/edac/amd64_edac.c | 7 ------- 1 file changed, 7 deletions(-)