[PATCH] ASoC: codecs: fix the right check and simplify code

Markus Elfring Markus.Elfring at web.de
Sun Sep 8 07:36:07 PDT 2024


> In the file drivers/base/platform.c, the return description of
> platform_get_irq is 'non-zero IRQ number on success, negative
> error number on failure.', so the check is wrong, fix it. And
> when get irq failed, the function platform_get_irq logs an error
> message.

1. You may occasionally put more than 64 characters into text lines
   of such a change description.

2. How do you think about to split any changes into separate update steps?
   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.11-rc6#n81

3. I find the wording “fix the right check” confusing.


Regards,
Markus



More information about the Linux-mediatek mailing list