From: Shiju Jose Date: Mon, 19 Aug 2024 10:44:58 +0000 (+0100) Subject: rasdaemon: ras-mce-handler: Fix checkpatch errors X-Git-Tag: v0.8.2~32 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=51e3d1f9ea65806234eb72ee488c122d722729fb;p=users%2Fmchehab%2Frasdaemon.git rasdaemon: ras-mce-handler: Fix checkpatch errors Fix following checkpatch error in ras-mce-handler.c Delete below obselte code under #if 0 ... #endif WARNING: Consider removing the code enclosed by this #if 0 and its #endif WARNING: Consider removing the code enclosed by this #if 0 and its #endif Signed-off-by: Shiju Jose Signed-off-by: Mauro Carvalho Chehab --- diff --git a/ras-mce-handler.c b/ras-mce-handler.c index ea58ac0..bf1fac0 100644 --- a/ras-mce-handler.c +++ b/ras-mce-handler.c @@ -314,28 +314,10 @@ static void report_mce_event(struct ras_events *ras, if (*e->mc_location) trace_seq_printf(s, ", %s", e->mc_location); -#if 0 - /* - * While the logic for decoding tsc is there at mcelog, why to - * decode/print it, if we already got the uptime from the - * tracing event? Let's just discard it for now. - */ - trace_seq_printf(s, ", tsc= %d", e->tsc); - trace_seq_printf(s, ", walltime= %d", e->walltime); -#endif - trace_seq_printf(s, ", cpu_type= %s", cputype_name[mce->cputype]); trace_seq_printf(s, ", cpu= %d", e->cpu); trace_seq_printf(s, ", socketid= %d", e->socketid); -#if 0 - /* - * The CPU vendor is already reported from mce->cputype - */ - trace_seq_printf(s, ", cpuvendor= %d", e->cpuvendor); - trace_seq_printf(s, ", cpuid= %d", e->cpuid); -#endif - if (e->ip) trace_seq_printf(s, ", ip= %llx%s", (long long)e->ip,