ARM: imx6: add missing put_device() call in imx6q_suspend_init()

Shawn Guo shawnguo at kernel.org
Tue Jun 23 03:32:21 EDT 2020


On Fri, Jun 05, 2020 at 12:15:32PM +0200, Markus Elfring wrote:
> >> Do you find a previous update suggestion useful?
> >>
> >> ARM: imx6: Add missing put_device() call in imx6q_suspend_init()
> >> https://lore.kernel.org/linux-arm-kernel/5acd7308-f6e1-4b1e-c744-bb2e5fdca1be@web.de/
> >> https://lore.kernel.org/patchwork/patch/1151158/
> >> https://lkml.org/lkml/2019/11/9/125
>> > It is useful indeed.
> 
> Thanks for your positive feedback.
> 
> 
> > Although I didn't run cocci script, since it can produce too many false result
> > which is hard to filter out.
> 
> Would you like to clarify any corresponding software analysis options?
> 
> 
> > BTW, I see you haver done the work already, I guess I won't send any patches
> > related to 'missing put_device after of_find_device_by_node()'.
> 
> You may continue also with contributions in such a direction.
> I would like to point out that other developers occasionally got into the mood
> to improve implementation details in similar ways already.
> 
> 
> > Any idea why these pathes didn't get applied ?
> 
> I can make assumptions about the reasons for the possibly questionable handling
> of such patches.

Markus,

Could you resend it to my kernel.org address?

Shawn



More information about the linux-arm-kernel mailing list