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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux