<div dir="ltr"><div>LGTM</div><div><br></div><div>Acked-by: Luke Nowakowski-Krijger <<a href="mailto:luke.nowakowskikrijger@canonical.com">luke.nowakowskikrijger@canonical.com</a>><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 18, 2022 at 11:25 AM Zachary Tahenakos <<a href="mailto:zachary.tahenakos@canonical.com">zachary.tahenakos@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><a href="https://bugs.launchpad.net/bugs/1969374" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/1969374</a><br>
<br>
SRU Justification<br>
<br>
[Impact]<br>
<br>
The latest PR to fix a certification issue has introduced a build failure. Specifically:<br>
net: mellanox: mlxbf_gige: Replace non-standard interrupt handling<br>
<br>
A build of the kernel fails with the following error:<br>
drivers/net/ethernet/mellanox/mlxbf_gige/mlxbf_gige_main.c:380:12: error: implicit declaration of function 'acpi_dev_gpio_irq_get_by'; did you mean 'acpi_dev_gpio_irq_get'? [-Werror=implicit-function-declaration]<br>
<br>
[Fix]<br>
Cherry-picking:<br>
809390219fb9c gpiolib: acpi: Allow to find GpioInt() resource by name and index<br>
<br>
The above commit reworks GpioInt() to allow finding the resource via name and index while preserving previous functionality.<br>
<br>
[Test Case]<br>
Beyond ensuring build still works, no testing of drivers using the reworked acpi_dev_gpio_irq_get function will be done, but it should still work as previously designed when NULL is passed for name.<br>
<br>
[Where problems could occur]<br>
Chance of regression should be very low with this change.<br>
<br>
Andy Shevchenko (1):<br>
  gpiolib: acpi: Allow to find GpioInt() resource by name and index<br>
<br>
 drivers/gpio/gpiolib-acpi.c | 12 ++++++++----<br>
 include/linux/acpi.h        | 10 ++++++++--<br>
 2 files changed, 16 insertions(+), 6 deletions(-)<br>
<br>
-- <br>
2.25.1<br>
<br>
<br>
-- <br>
kernel-team mailing list<br>
<a href="mailto:kernel-team@lists.ubuntu.com" target="_blank">kernel-team@lists.ubuntu.com</a><br>
<a href="https://lists.ubuntu.com/mailman/listinfo/kernel-team" rel="noreferrer" target="_blank">https://lists.ubuntu.com/mailman/listinfo/kernel-team</a><br>
</blockquote></div></div>