[PATCH] thermal: sun8i: Be loud when probe fails
Maxime Ripard
maxime at cerno.tech
Wed Jul 8 09:29:47 EDT 2020
On Wed, Jul 08, 2020 at 12:55:27PM +0200, Ondrej Jirman wrote:
> @@ -523,10 +547,17 @@ static int sun8i_ths_probe(struct platform_device *pdev)
> ret = devm_request_threaded_irq(dev, irq, NULL,
> sun8i_irq_thread,
> IRQF_ONESHOT, "ths", tmdev);
> - if (ret)
> - return ret;
> + if (ret) {
> + dev_err(dev, "Failed to request irq (%d)\n", ret);
> + goto err_out;
> + }
>
> + dev_info(dev, "Thermal sensor ready!\n");
> return 0;
I missed that in my first mail, but I'm not sure we want to print
anything on success. This doesn't bring any value and that will only
make it harder to find errors in other drivers.
Maxime
More information about the linux-arm-kernel
mailing list