diff mbox

powerpc/mce: Remove unused but set variable

Message ID 20161117161921.13232-1-tklauser@distanz.ch (mailing list archive)
State Accepted
Headers show

Commit Message

Tobias Klauser Nov. 17, 2016, 4:19 p.m. UTC
Remove the unused but set variable srr1 in save_mce_event() to
fix the following GCC warning when building with 'W=1':

  arch/powerpc/kernel/mce.c:75:11: warning: variable 'srr1' set but not used [-Wunused-but-set-variable]

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
---
 arch/powerpc/kernel/mce.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Michael Ellerman Nov. 22, 2016, 12:34 a.m. UTC | #1
On Thu, 2016-11-17 at 16:19:21 UTC, Tobias Klauser wrote:
> Remove the unused but set variable srr1 in save_mce_event() to
> fix the following GCC warning when building with 'W=1':
> 
>   arch/powerpc/kernel/mce.c:75:11: warning: variable 'srr1' set but not used [-Wunused-but-set-variable]
> 
> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

Applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/d6d56ec738dcac4ac7716ccb7a08a4

cheers
diff mbox

Patch

diff --git a/arch/powerpc/kernel/mce.c b/arch/powerpc/kernel/mce.c
index 5e7ece0fda9f..c6923ff45131 100644
--- a/arch/powerpc/kernel/mce.c
+++ b/arch/powerpc/kernel/mce.c
@@ -72,7 +72,6 @@  void save_mce_event(struct pt_regs *regs, long handled,
 		    struct mce_error_info *mce_err,
 		    uint64_t nip, uint64_t addr)
 {
-	uint64_t srr1;
 	int index = __this_cpu_inc_return(mce_nest_count) - 1;
 	struct machine_check_event *mce = this_cpu_ptr(&mce_event[index]);
 
@@ -99,8 +98,6 @@  void save_mce_event(struct pt_regs *regs, long handled,
 		mce->disposition = MCE_DISPOSITION_NOT_RECOVERED;
 	mce->severity = MCE_SEV_ERROR_SYNC;
 
-	srr1 = regs->msr;
-
 	/*
 	 * Populate the mce error_type and type-specific error_type.
 	 */