Re: [PATCH v2 3/3] i2c: smbus: Use device_*() functions instead of of_*()

From: Andy Shevchenko
Date: Wed Jan 12 2022 - 10:43:02 EST


On Wed, Jan 12, 2022 at 4:15 PM Akhil R <akhilrajeev@xxxxxxxxxx> wrote:
>
> Change of_*() functions to device_*() for firmware agnostic usage.
> This allows to have smbus_alert interrupt without any changes

the smbus_alert

> in the controller drivers using ACPI table.

the ACPI

...

This change reveals potential issue:

> - irq = of_irq_get_byname(adapter->dev.of_node, "smbus_alert");
> + irq = device_irq_get_byname(adapter->dev.parent, "smbus_alert");

> if (irq <= 0)

I guess this '= 0' part should be fixed first.

> return irq;

--
With Best Regards,
Andy Shevchenko