<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body>
    <p><font face="segoe ui,sans-serif"><span style="color: rgba(0, 0, 0, 0.85); font-size: 14px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: pre; background-color: rgb(255, 255, 255); text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial; display: inline !important; float: none;">Hi tglx and </span><font size="2">Miquèl</font></font><span style="color: rgba(0, 0, 0, 0.85); font-family: -apple-system, blinkmacsystemfont, "Helvetica Neue", helvetica, "segoe ui", arial, roboto, "PingFang SC", miui, "Hiragino Sans GB", "Microsoft Yahei", sans-serif, "Source Han Serif"; font-size: 14px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: pre; background-color: rgb(255, 255, 255); text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial; display: inline !important; float: none;">,

Just a gentle ping on this patchset. I understand you're likely busy with many other tasks, but any feedback or guidance on how to proceed would be greatly appreciated.

As discussed in the previous thread, there was some uncertainty about whether we should directly integrate dev_err_probe() into devm_request_threaded_irq(), or if it's better to create a wrapper function.

Thanks for your time!

Thanks,
        Panchuang</span></p>
    <blockquote type="cite" cite="mid:%20%3CTY1PPFECF6DCA7C0493FE1041C661918E91DA7BA@TY1PPFECF6DCA7C.apcprd06.prod.outlook.com">
      <pre wrap="" class="moz-quote-pre">Hi tglx and Miquèl,

On Tue, Jun 24 2025 at 16:13, tglx wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="" class="moz-quote-pre">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?
</pre>
      </blockquote>
      <pre wrap="" class="moz-quote-pre">
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]<a class="moz-txt-link-freetext" href="https://lore.kernel.org/all/87h6qpyzkd.ffs@tglx/">https://lore.kernel.org/all/87h6qpyzkd.ffs@tglx/</a>

Thanks,

        PanChuang
</pre>
    </blockquote>
  </body>
</html>