From: Colin Ian King <colin.king@canonical.com>
Date: Wed, 8 May 2019 22:42:01 +0000 (+0100)
Subject: EDAC/sb_edac: Remove redundant update of tad_base
X-Git-Tag: v5.4-rc1~647^2~9
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0042e9e7a5f662d7b8226316369d02c3553d6400;p=platform%2Fkernel%2Flinux-rpi.git

EDAC/sb_edac: Remove redundant update of tad_base

The variable tad_base is being set to a value that is never read and is
being over-written on the next iteration of a for-loop. This assignment
is therefore redundant and can be removed.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Borislav Petkov <bp@suse.de>
Acked-by: Tony Luck <tony.luck@intel.com>
Cc: James Morse <james.morse@arm.com>
Cc: kernel-janitors@vger.kernel.org
Cc: linux-edac <linux-edac@vger.kernel.org>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Link: https://lkml.kernel.org/r/20190508224201.27120-1-colin.king@canonical.com
---

diff --git a/drivers/edac/sb_edac.c b/drivers/edac/sb_edac.c
index fa700a1..37746b0 100644
--- a/drivers/edac/sb_edac.c
+++ b/drivers/edac/sb_edac.c
@@ -1511,7 +1511,6 @@ static int knl_get_dimm_capacity(struct sbridge_pvt *pvt, u64 *mc_sizes)
 						sad_actual_size[mc] += tad_size;
 					}
 				}
-				tad_base = tad_limit+1;
 			}
 		}