Re: [PATCH 2/2] i2c: designware: use polling when IRQ resource is 0

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Tue, Nov 03, 2026 at 10:00:59AM +0100, Samuel Tirwa wrote:
> When an Intel LPSS I2C controller has no usable interrupt (firmware
> reports PCI INTx as not connected), intel-lpss passes 0 as the IRQ base,
> which leaves the child device with an IRQ resource whose start is 0.
> platform_get_irq_optional() then returns 0 instead of -ENXIO, so the
> DesignWare driver never enabled ACCESS_POLLING and request_irq(0, ...)
> failed.
> 
> Treat an IRQ number of 0 the same as 'no IRQ resource' and enable
> polling mode.

Slow down. You have made a mess in the mailing list: no email thread, no
versioning between the series of the same, no cover letter. Take your time to
become a familiar with the process first.

...

>  	irq = platform_get_irq_optional(pdev, 0);
> -	if (irq == -ENXIO)
> +	if (irq == -ENXIO || irq == 0)
>  		flags |= ACCESS_POLLING;
>  	else if (irq < 0)
>  		return irq;

NAK. irq==0 gives a warning and has not be a valid output from
platform_get_irq*().

-- 
With Best Regards,
Andy Shevchenko