[PATCH] watchdog: w83697hf_wdt: return ENODEV if no device was found

[PATCH] watchdog: w83697hf_wdt: return ENODEV if no device was found

[Stanislav Kholmanskikh]

Most WDT driver modules return ENODEV during modprobe if
no valid device was found, but w83697hf_wdt returns EIO.

Let w83697hf_wdt return ENODEV.

Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
---
 drivers/watchdog/w83697hf_wdt.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/watchdog/w83697hf_wdt.c b/drivers/watchdog/w83697hf_wdt.c
index aaf2995..68b45fc 100644
--- a/drivers/watchdog/w83697hf_wdt.c
+++ b/drivers/watchdog/w83697hf_wdt.c
@@ -402,7 +402,7 @@ static int __init wdt_init(void)
 
 	if (!found) {
 		pr_err("No W83697HF/HG could be found\n");
-		ret = -EIO;
+		ret = -ENODEV;
 		goto out;
 	}
 
-- 
1.7.1

Re: [PATCH] watchdog: w83697hf_wdt: return ENODEV if no device was found

[Guenter Roeck]

On Mon, Jan 27, 2014 at 11:04:33AM +0400, Stanislav Kholmanskikh wrote:
> Most WDT driver modules return ENODEV during modprobe if
> no valid device was found, but w83697hf_wdt returns EIO.
> 
> Let w83697hf_wdt return ENODEV.
> 
> Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Re: [PATCH] watchdog: w83697hf_wdt: return ENODEV if no device was found

[Stanislav Kholmanskikh]

Hi!

On 01/27/2014 08:33 PM, Guenter Roeck wrote:
> On Mon, Jan 27, 2014 at 11:04:33AM +0400, Stanislav Kholmanskikh wrote:
>> Most WDT driver modules return ENODEV during modprobe if
>> no valid device was found, but w83697hf_wdt returns EIO.
>>
>> Let w83697hf_wdt return ENODEV.
>>
>> Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
>
> Reviewed-by: Guenter Roeck <linux@roeck-us.net>

Thank you.

I'm a newbie here.

Should I do something to move it forward (like send this to an another 
mail list), or just wait? :)

Thanks.

> --
> To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

Re: [PATCH] watchdog: w83697hf_wdt: return ENODEV if no device was found

[Guenter Roeck]

On 02/05/2014 10:25 PM, Stanislav Kholmanskikh wrote:
> Hi!
>
> On 01/27/2014 08:33 PM, Guenter Roeck wrote:
>> On Mon, Jan 27, 2014 at 11:04:33AM +0400, Stanislav Kholmanskikh wrote:
>>> Most WDT driver modules return ENODEV during modprobe if
>>> no valid device was found, but w83697hf_wdt returns EIO.
>>>
>>> Let w83697hf_wdt return ENODEV.
>>>
>>> Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
>>
>> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
>
> Thank you.
>
> I'm a newbie here.
>
> Should I do something to move it forward (like send this to an another mail list), or just wait? :)
>

Just wait. I'll apply it to my tree and send a reminder to Wim at some point.
Usually Wim is pretty good at picking things up himself, though.

Of course, fun part in this case is that I actually submitted a patch
to remove this driver ;-). Not as bad as it sounds - reason was that
the w83627hf driver now supports the w83697hf as well, and the w83697hf
driver is thus redundant. I can understand Wim if he wants to keep
the driver around for a while, though.

Guenter

Re: [PATCH] watchdog: w83697hf_wdt: return ENODEV if no device was found

[Stanislav Kholmanskikh]

On 02/06/2014 11:25 AM, Guenter Roeck wrote:
> On 02/05/2014 10:25 PM, Stanislav Kholmanskikh wrote:
>> Hi!
>>
>> On 01/27/2014 08:33 PM, Guenter Roeck wrote:
>>> On Mon, Jan 27, 2014 at 11:04:33AM +0400, Stanislav Kholmanskikh wrote:
>>>> Most WDT driver modules return ENODEV during modprobe if
>>>> no valid device was found, but w83697hf_wdt returns EIO.
>>>>
>>>> Let w83697hf_wdt return ENODEV.
>>>>
>>>> Signed-off-by: Stanislav Kholmanskikh
>>>> <stanislav.kholmanskikh@oracle.com>
>>>
>>> Reviewed-by: Guenter Roeck <linux@roeck-us.net>
>>
>> Thank you.
>>
>> I'm a newbie here.
>>
>> Should I do something to move it forward (like send this to an another
>> mail list), or just wait? :)
>>
>
> Just wait. I'll apply it to my tree and send a reminder to Wim at some
> point.
> Usually Wim is pretty good at picking things up himself, though.
>
> Of course, fun part in this case is that I actually submitted a patch
> to remove this driver ;-). Not as bad as it sounds - reason was that
> the w83627hf driver now supports the w83697hf as well, and the w83697hf
> driver is thus redundant. I can understand Wim if he wants to keep
> the driver around for a while, though.
>

Ok. Got it.

Both outcomes are good for me :)


> Guenter
>
> --
> To unsubscribe from this list: send the line "unsubscribe
> linux-watchdog" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html