On Mon, May 21, 2018 at 3:49 PM, Alexandru Gagniuc <mr.nuke.me@xxxxxxxxx> wrote: > ghes_severity() is a misnomer in this case, as it implies the severity > of the entire GHES structure. Instead, it maps one CPER value to a > GHES_SEV* value. ghes_cper_severity() is clearer. It looks like the *real* reason for this change is that you re-introduce ghes_severity() as a different function in the second patch. There are a couple of reasons to avoid that, one of them being that people will now have to remember what the function did in which kernel versions. Also, the current name is good enough IMO, so I'm not going to apply this patch. -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html