Commit 8cc6b422 authored by Yang Li's avatar Yang Li Committed by Greg Kroah-Hartman
Browse files

slimbus: qcom: Remove unnecessary print function dev_err() 



The print function dev_err() is redundant because
platform_get_irq_byname() already prints an error.

Eliminate the follow coccicheck warning:
./drivers/slimbus/qcom-ctrl.c:514:2-9: line 514 is redundant because
platform_get_irq() already prints an error

Reported-by: default avatarAbaci Robot <abaci@linux.alibaba.com>
Signed-off-by: default avatarYang Li <yang.lee@linux.alibaba.com>
Signed-off-by: default avatarSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Link: https://lore.kernel.org/r/20220510095205.337-3-srinivas.kandagatla@linaro.org


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 225baab0

drivers/slimbus/qcom-ctrl.c

+1 −3
Original line number Original line Diff line number Diff line
@@ -510,10 +510,8 @@ static int qcom_slim_probe(struct platform_device *pdev) 
	}

	}





	ctrl->irq = platform_get_irq(pdev, 0);

	ctrl->irq = platform_get_irq(pdev, 0);

	if (ctrl->irq < 0) {

	if (ctrl->irq < 0)

		dev_err(&pdev->dev, "no slimbus IRQ\n");

		return ctrl->irq;

		return ctrl->irq;

	}





	sctrl = &ctrl->ctrl;

	sctrl = &ctrl->ctrl;

	sctrl->dev = &pdev->dev;

	sctrl->dev = &pdev->dev;