lkml.org 
[lkml]   [2020]   [Jul]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] thermal: sun8i: Be loud when probe fails
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

\
 
 \ /
  Last update: 2020-07-08 15:30    [W:2.015 / U:0.000 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site