Hi Steve,

On 05/15/2015 01:13 AM, Opensource [Steve Twiss] wrote:

Hi Guenter,

Thank you for your comments again,
Here are my responses.

Regards,
Steve

On 15 May 2015 03:13, Guenter Roeck

quoted

Subject: Re: [PATCH V2 3/4] watchdog: da9062: DA9062 watchdog driver

[...]

[ ... ]

quoted

quoted

+
+	irq = platform_get_irq_byname(pdev, "WDG_WARN");
+	if (irq < 0) {
+		dev_err(wdt->hw->dev, "Failed to get IRQ.\n");
+		ret = irq;
+		goto error;

Just return; the label does not serve a useful purpose. Same for the other
goto statements below.

Agreed. This is changed now.

quoted

Also, is the interrupt mandatory ? All it does is to display a message.
Looks very optional to me.

It is a place holder for something more application specific.
I could remove it, but I figured it would just get re-added when somebody takes the
driver and modifies it for their needs.

If this is a problem however, it can go.
Please advise ..

Then this someone should add the code. For the time being, it just increases
kernel size and may cause the driver to fail for no good reason. Plus, given
the driver apparently works without interrupt, even then it should be optional,
and the driver does not have to fail loading if it is not supported on a
given platform.

Thanks,
Guenter