Commit c3d66a16 authored by Miaoqian Lin's avatar Miaoqian Lin Committed by Nishanth Menon
Browse files

soc: ti: wkup_m3_ipc: Fix IRQ check in wkup_m3_ipc_probe 



platform_get_irq() returns negative error number instead 0 on failure.
And the doc of platform_get_irq() provides a usage example:

    int irq = platform_get_irq(pdev, 0);
    if (irq < 0)
        return irq;

Fix the check of return value to catch errors correctly.

Fixes: cdd5de50 ("soc: ti: Add wkup_m3_ipc driver")
Signed-off-by: default avatarMiaoqian Lin <linmq006@gmail.com>
Signed-off-by: default avatarNishanth Menon <nm@ti.com>
Acked-by: default avatarDave Gerlach <d-gerlach@ti.com>
Link: https://lore.kernel.org/r/20220114062840.16620-1-linmq006@gmail.com
parent a181bcfc

drivers/soc/ti/wkup_m3_ipc.c

+2 −2
Original line number Diff line number Diff line
@@ -450,9 +450,9 @@ static int wkup_m3_ipc_probe(struct platform_device *pdev) 
		return PTR_ERR(m3_ipc->ipc_mem_base);



	irq = platform_get_irq(pdev, 0);

	if (!irq) {

	if (irq < 0) {

		dev_err(&pdev->dev, "no irq resource\n");

		return -ENXIO;

		return irq;

	}



	ret = devm_request_irq(dev, irq, wkup_m3_txev_handler,