* Re: [PATCH] ARM: imx6: add missing put_device() call in imx6q_suspend_init()
@ 2020-06-04 19:07 Markus Elfring
2020-06-05 9:08 ` yukuai (C)
0 siblings, 1 reply; 12+ messages in thread
From: Markus Elfring @ 2020-06-04 19:07 UTC (permalink / raw)
To: Yu Kuai, kernel, linux-arm-kernel, linux-imx
Cc: Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Shawn Guo, Yi Zhang, kernel-janitors, LKML
> if of_find_device_by_node() succeed, imx6q_suspend_init() doesn't have a
> corresponding put_device(). Thus add a jump target to fix the exception
> handling for this function implementation.
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
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-04 19:07 [PATCH] ARM: imx6: add missing put_device() call in imx6q_suspend_init() Markus Elfring
@ 2020-06-05 9:08 ` yukuai (C)
2020-06-05 10:15 ` Markus Elfring
0 siblings, 1 reply; 12+ messages in thread
From: yukuai (C) @ 2020-06-05 9:08 UTC (permalink / raw)
To: Markus Elfring, kernel, linux-arm-kernel, linux-imx
Cc: Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Shawn Guo, Yi Zhang, kernel-janitors, LKML
On 2020/6/5 3:07, Markus Elfring wrote:
>> if of_find_device_by_node() succeed, imx6q_suspend_init() doesn't have a
>> corresponding put_device(). Thus add a jump target to fix the exception
>> handling for this function implementation.
>
> 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
Hi, Markus
It is useful indeed. Although I didn't run cocci script, since it can
produce too many false result which is hard to filter out.
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()'.
Any idea why these pathes didn't get applied ?
Best regards,
Yu Kuai
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-05 9:08 ` yukuai (C)
@ 2020-06-05 10:15 ` Markus Elfring
2020-06-23 7:32 ` Shawn Guo
0 siblings, 1 reply; 12+ messages in thread
From: Markus Elfring @ 2020-06-05 10:15 UTC (permalink / raw)
To: Yu Kuai, kernel, linux-arm-kernel, linux-imx
Cc: Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Shawn Guo, Yi Zhang, kernel-janitors, LKML
>> 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.
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-05 10:15 ` Markus Elfring
@ 2020-06-23 7:32 ` Shawn Guo
2020-06-23 7:48 ` Markus Elfring
0 siblings, 1 reply; 12+ messages in thread
From: Shawn Guo @ 2020-06-23 7:32 UTC (permalink / raw)
To: Markus Elfring
Cc: Yu Kuai, kernel, linux-arm-kernel, linux-imx, Anson Huang,
Fabio Estevam, Russell King, Sascha Hauer, Yi Zhang,
kernel-janitors, LKML
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 7:32 ` Shawn Guo
@ 2020-06-23 7:48 ` Markus Elfring
2020-06-23 11:02 ` Shawn Guo
0 siblings, 1 reply; 12+ messages in thread
From: Markus Elfring @ 2020-06-23 7:48 UTC (permalink / raw)
To: Shawn Guo, kernel, linux-arm-kernel, linux-imx
Cc: Yu Kuai, Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Yi Zhang, kernel-janitors, LKML
>>>> 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.
…
>>> 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?
You can get relevant information from the referenced message archive interfaces,
can't you?
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 7:48 ` Markus Elfring
@ 2020-06-23 11:02 ` Shawn Guo
2020-06-23 12:00 ` Markus Elfring
0 siblings, 1 reply; 12+ messages in thread
From: Shawn Guo @ 2020-06-23 11:02 UTC (permalink / raw)
To: Markus Elfring
Cc: kernel, linux-arm-kernel, linux-imx, Yu Kuai, Anson Huang,
Fabio Estevam, Russell King, Sascha Hauer, Yi Zhang,
kernel-janitors, LKML
On Tue, Jun 23, 2020 at 09:48:52AM +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.
> …
> >>> 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?
>
> You can get relevant information from the referenced message archive interfaces,
> can't you?
Well, I'm asking you to resend to make sure the following:
- Use correct maintainer mailbox address.
- You still care about the patch.
- The patch applies to v5.8-rc.
Shawn
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 11:02 ` Shawn Guo
@ 2020-06-23 12:00 ` Markus Elfring
2020-06-23 12:09 ` Shawn Guo
0 siblings, 1 reply; 12+ messages in thread
From: Markus Elfring @ 2020-06-23 12:00 UTC (permalink / raw)
To: Shawn Guo, kernel, linux-arm-kernel, linux-imx
Cc: Yu Kuai, Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Yi Zhang, kernel-janitors, LKML
>>>>>> 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
…
>> You can get relevant information from the referenced message archive interfaces,
>> can't you?
>
> Well, I'm asking you to resend to make sure the following:
>
> - Use correct maintainer mailbox address.
Were the selected message recipients appropriate at 2019-11-09?
> - You still care about the patch.
Partly, yes.
> - The patch applies to v5.8-rc.
Would you like to try it out if my proposal is still valid?
Does the change approach by Yu Kuai supersede it?
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 12:00 ` Markus Elfring
@ 2020-06-23 12:09 ` Shawn Guo
2020-06-23 12:28 ` Markus Elfring
0 siblings, 1 reply; 12+ messages in thread
From: Shawn Guo @ 2020-06-23 12:09 UTC (permalink / raw)
To: Markus Elfring
Cc: kernel, linux-arm-kernel, linux-imx, Yu Kuai, Anson Huang,
Fabio Estevam, Russell King, Sascha Hauer, Yi Zhang,
kernel-janitors, LKML
On Tue, Jun 23, 2020 at 02:00:09PM +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
> …
> >> You can get relevant information from the referenced message archive interfaces,
> >> can't you?
> >
> > Well, I'm asking you to resend to make sure the following:
> >
> > - Use correct maintainer mailbox address.
>
> Were the selected message recipients appropriate at 2019-11-09?
At least, my mailbox was not.
>
>
> > - You still care about the patch.
>
> Partly, yes.
>
>
> > - The patch applies to v5.8-rc.
>
> Would you like to try it out if my proposal is still valid?
>
> Does the change approach by Yu Kuai supersede it?
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 12:09 ` Shawn Guo
@ 2020-06-23 12:28 ` Markus Elfring
2020-06-23 12:36 ` Shawn Guo
0 siblings, 1 reply; 12+ messages in thread
From: Markus Elfring @ 2020-06-23 12:28 UTC (permalink / raw)
To: Shawn Guo, kernel, linux-arm-kernel, linux-imx
Cc: Yu Kuai, Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Yi Zhang, kernel-janitors, LKML
>>>>>>>> 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
…
>>> - The patch applies to v5.8-rc.
>>
>> Would you like to try it out if my proposal is still valid?
Are you going to compare the published patches any further?
>> Does the change approach by Yu Kuai supersede it?
Which patch variant will be integrated finally?
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 12:28 ` Markus Elfring
@ 2020-06-23 12:36 ` Shawn Guo
2020-06-23 14:33 ` Markus Elfring
2020-06-24 14:26 ` Markus Elfring
0 siblings, 2 replies; 12+ messages in thread
From: Shawn Guo @ 2020-06-23 12:36 UTC (permalink / raw)
To: Markus Elfring
Cc: Sascha Hauer, linux-arm-kernel, NXP Linux Team, Yu Kuai,
Anson Huang, Fabio Estevam, Russell King, Sascha Hauer, Yi Zhang,
kernel-janitors, LKML
On Tue, Jun 23, 2020 at 8:29 PM Markus Elfring <Markus.Elfring@web.de> wrote:
>
> >>>>>>>> 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
> …
> >>> - The patch applies to v5.8-rc.
> >>
> >> Would you like to try it out if my proposal is still valid?
>
> Are you going to compare the published patches any further?
>
>
> >> Does the change approach by Yu Kuai supersede it?
>
> Which patch variant will be integrated finally?
Just picked up Yu Kuai's patch.
Shawn
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 12:36 ` Shawn Guo
@ 2020-06-23 14:33 ` Markus Elfring
2020-06-24 14:26 ` Markus Elfring
1 sibling, 0 replies; 12+ messages in thread
From: Markus Elfring @ 2020-06-23 14:33 UTC (permalink / raw)
To: Shawn Guo, kernel, linux-arm-kernel, linux-imx
Cc: Yu Kuai, Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Yi Zhang, kernel-janitors, LKML
>>>>>>>>>> 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
…
>> Are you going to compare the published patches any further?
>>
>>
>>>> Does the change approach by Yu Kuai supersede it?
>>
>> Which patch variant will be integrated finally?
>
> Just picked up Yu Kuai's patch.
Did you adjust any details?
https://lkml.org/lkml/2020/6/23/542
https://lore.kernel.org/patchwork/comment/1457270/
With which delay will a corresponding commit be published?
https://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git/log/arch/arm/mach-imx/pm-imx6.c
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: ARM: imx6: add missing put_device() call in imx6q_suspend_init()
2020-06-23 12:36 ` Shawn Guo
2020-06-23 14:33 ` Markus Elfring
@ 2020-06-24 14:26 ` Markus Elfring
1 sibling, 0 replies; 12+ messages in thread
From: Markus Elfring @ 2020-06-24 14:26 UTC (permalink / raw)
To: Shawn Guo, kernel, linux-arm-kernel, linux-imx
Cc: Yu Kuai, Anson Huang, Fabio Estevam, Russell King, Sascha Hauer,
Yi Zhang, kernel-janitors, LKML
>> Are you going to compare the published patches any further?
…
>> Which patch variant will be integrated finally?
>
> Just picked up Yu Kuai's patch.
Is it interesting anyhow that you committed a change description
which contained a typo (on 2020-06-23 20:11:30 +0800)?
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/arch/arm/mach-imx/pm-imx6.c?id=0f92519843a05e97b58c6076edb23d747e8be262
Would it have been helpful to add the tag “Fixes”?
Regards,
Markus
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2020-06-24 14:27 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-04 19:07 [PATCH] ARM: imx6: add missing put_device() call in imx6q_suspend_init() Markus Elfring
2020-06-05 9:08 ` yukuai (C)
2020-06-05 10:15 ` Markus Elfring
2020-06-23 7:32 ` Shawn Guo
2020-06-23 7:48 ` Markus Elfring
2020-06-23 11:02 ` Shawn Guo
2020-06-23 12:00 ` Markus Elfring
2020-06-23 12:09 ` Shawn Guo
2020-06-23 12:28 ` Markus Elfring
2020-06-23 12:36 ` Shawn Guo
2020-06-23 14:33 ` Markus Elfring
2020-06-24 14:26 ` Markus Elfring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).