回复: [PATCH v6 01/24] genirq/devres: Add devm_request_threaded_irq_probe() and devm_request_irq_probe()

潘闯 panchuang at vivo.com
Tue Jun 24 19:54:38 PDT 2025


Hi tglx and Miquèl,

On Tue, Jun 24 2025 at 16:13, tglx wrote:
> As you cited correctly I said back then:
> 
>   "So there is certainly an argument to be made that this particular
>   function should print a well formatted and informative error
>   message."
>
> This particular function is: devm_request_threaded_irq().
>
> IOW, I did not ask you should go an create a new one, right?

Thank you all for your valuable feedback and guidance regarding the recent patch. I am a bit confused because, in the v1 version, no new function was added. However, in v2, tglx suggested introducing a new function[1]. Now, I would greatly appreciate your advice on the appropriate approach:

  Should we directly integrate dev_err_probe() inside devm_request_threaded_irq()?
  Or should we create a wrapper function instead?

Thank you for your guidance.

[1]https://lore.kernel.org/all/87h6qpyzkd.ffs@tglx/

Thanks,

        PanChuang


More information about the Linux-mediatek mailing list