Thread (4 messages) read the whole thread 4 messages, 3 authors, 2021-12-25

Re: [PATCH v3] soc: ti: smartreflex: Use platform_get_irq_optional() to get the interrupt

From: Andy Shevchenko <hidden>
Date: 2021-12-25 17:19:23
Also in: linux-pm, lkml

On Sat, Dec 25, 2021 at 7:10 PM Lad, Prabhakar
[off-list ref] wrote:
On Sat, Dec 25, 2021 at 4:56 PM Andy Shevchenko
[off-list ref] wrote:
quoted
On Sat, Dec 25, 2021 at 4:08 AM Lad Prabhakar
[off-list ref] wrote:
...
quoted
quoted
+       ret = 0;
I do not see the context, is it really necessary?
Yes the change is necessary as the probe callback returns ret on
success, and in the below code flow there might be a case where ret
inst set. So setting the ret here.
I can't believe this, so I have checked the code and it seems it uses
return ret at the end where it should be simply return 0. So, please,
fix it there.

-- 
With Best Regards,
Andy Shevchenko

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help