Re: [PATCH v4] hwmon: (asus-ec-sensors) merge setup functions

From: Guenter Roeck
Date: Wed Feb 16 2022 - 14:13:23 EST


On 2/16/22 11:00, Eugene Shalygin wrote:
On Wed, 16 Feb 2022 at 19:49, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:

* If your patch generates checkpatch errors, warnings, or check messages,
please refrain from explanations such as "I prefer that coding style".
Keep in mind that each unnecessary message helps hiding a real problem,
and a consistent coding style makes it easier for others to understand
and review the code.

Yes, of course. My question is what do I do with 6 other braces single
statement if's now?


Nothing as part of this patch. My fault for not bringing this up earlier,
but that is no reason to not clean up the code while touching it.

You _could_ submit a separate patch fixing everything reported by
checkpatch, but that is up to you.

Thanks,
Guenter