On Wed, Jul 8, 2026 at 4:02 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On 04/06/2026 06:28, Rosen Penev wrote:
> > @@ -271,10 +265,16 @@ static int fsl_ifc_ctrl_probe(struct platform_device *dev)
> >       if (ret != 0) {
> >               dev_err(&dev->dev, "failed to install irq (%d)\n",
> >                       fsl_ifc_ctrl_dev->irq);
> > -             goto err_unmap_nandirq;
> > +             goto err;
> >       }
> >
> > -     if (fsl_ifc_ctrl_dev->nand_irq) {
> > +     /* get the nand machine irq */
> > +     fsl_ifc_ctrl_dev->nand_irq = platform_get_irq_optional(dev, 1);
> > +     if (fsl_ifc_ctrl_dev->nand_irq < 0) {
> > +             ret = fsl_ifc_ctrl_dev->nand_irq;
>
> How did you test it? Looks like you change the logic and fail the probe
> on missing IRQ.
Yeah this was wrong. I assumed the optional part was the irq itself
but it's just the error message.
>
>
> Best regards,
> Krzysztof