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 at redhat.com>
Thanks
>
> Regards,
> Markus
>