]> www.infradead.org Git - users/mchehab/rasdaemon.git/commitdiff
rasdaemon: ras-mce-handler: Fix checkpatch errors
authorShiju Jose <shiju.jose@huawei.com>
Mon, 19 Aug 2024 10:44:58 +0000 (11:44 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Mon, 18 Nov 2024 12:27:18 +0000 (13:27 +0100)
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 <shiju.jose@huawei.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
ras-mce-handler.c

index ea58ac09126701bec7fa7145dfe317277271a3e0..bf1fac013a760d249ae058daeb39c015a79dfec9 100644 (file)
@@ -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,