On 12/22/21 16:24, Jiasheng Jiang wrote: > It can be found that platform_get_irq() returns nagative code but not > null when fails. > The comment of the platform_get_irq clearly shows that. > Therefore it should be better to remove the useless check. Nope, platform_get_irq() can actually return 0 (the comment for that function is wrong). So testing for !irq is valid and must be kept. > > Fixes: fd990556f0fa ("ata: move library code from ahci_platform.c to libahci_platform.c") > Signed-off-by: Jiasheng Jiang <jiasheng@xxxxxxxxxxx> > --- > drivers/ata/libahci_platform.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/ata/libahci_platform.c b/drivers/ata/libahci_platform.c > index b2f552088291..5ec68f138c28 100644 > --- a/drivers/ata/libahci_platform.c > +++ b/drivers/ata/libahci_platform.c > @@ -587,8 +587,6 @@ int ahci_platform_init_host(struct platform_device *pdev, > dev_err(dev, "no irq\n"); > return irq; > } > - if (!irq) > - return -EINVAL; > > hpriv->irq = irq; > -- Damien Le Moal Western Digital Research