On Mon, May 27, 2019 at 04:19:29PM +0100, Charles Keepax wrote: > In preparation for future refactoring factor out the fetch of the IRQ > into its own helper function. > > Signed-off-by: Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxx> > --- > > +static int i2c_acpi_get_irq(struct acpi_device *adev) > +{ > + struct list_head resource_list; > + int irq = -ENOENT; > + int ret; > + > + INIT_LIST_HEAD(&resource_list); > + > + ret = acpi_dev_get_resources(adev, &resource_list, > + i2c_acpi_add_resource, &irq); > + if (ret < 0) > + return -EINVAL; > + > + acpi_dev_free_resource_list(&resource_list); > + > + return irq; > +} > + Sorry just noticed I forgot to add the kernel doc. Will fix that up in the next spin, or do an incremental patch. Thanks, Charles