Messages in this thread | | | Date | Thu, 23 Dec 2021 09:44:39 +0900 | Subject | Re: [PATCH] ata: libahci_platform: Remove abundant check | From | Damien Le Moal <> |
| |
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. > > Fixes: fd990556f0fa ("ata: move library code from ahci_platform.c to libahci_platform.c") > Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
Another note: please use scripts/get_maintainer.pl to see to whom you should address patches. This one was not addressed to me.
> --- > 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
| |