On 2020/4/9 下午2:02, Markus Elfring wrote:
The function “platform_get_irq” can log an error already.
Thus omit a redundant message for the exception handling in the
calling function.
It looks to me that not all error path of platform_get_irq() were loggd.
How do you think about to clarify the handling of the error code “-EPROBE_DEFER” any more?
https://elixir.bootlin.com/linux/v5.6.3/source/drivers/base/platform.c#L202
Aha, it looks to me that I'm on the wrong branch. Sorry.
And git grep told me there're other users of platform_get_irq() that check and log by themselves.
Source code analysis can point further update candidates out, can't it?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/base/platform.c?id=7723f4c5ecdb8d832f049f8483beb0d1081cedf6
Yes, it can.
Would you like to achieve collateral evolution?
Yes and it's probably be better to put all the fixes in one series.
For this patch.
Acked-by: Jason Wang <jasowang@xxxxxxxxxx>
Thanks
Regards,
Markus
