[v3,1/1] hwmon:(nct7904) Set default timeout

2020-05-26 Thread yuechao.zhao
From: Yuechao Zhao The timeout module parameter should not be used for setting the default timeout. Because, if you set the timeout = 0, the default timeout will be meaningless. And the timeout module parameter of 0 means "no timeout module paraameter specified". Signed-off-by: Yuechao Zhao ---

[v2,1/1] hwmon:(nct7904) Set default timeout

2020-05-26 Thread yuechao.zhao
From: Yuechao Zhao The timeout module parameter should not be used for setting the default timeout. Because, if you set the timeout = 0, the default timeout will be meaningless. And the timeout module parameter of 0 means "no timeout module paraameter specified". Signed-off-by: Yuechao Zhao ---

[v1,1/1 2/2] Set the default timeout

2020-05-26 Thread yuechao.zhao
From: Yuechao Zhao The timeout module parameter should not be used for setting the default timeout. Because, if you set the timeout = 0, the default timeout will be meaningless. And the wahtchdog_init_timeout() can not detect this error because the timeout module parameter of 0 means "no timeout

[v1,1/1 1/2] Check the timeout module parameter is in the min-max range

2020-05-26 Thread yuechao.zhao
From: Yuechao Zhao Check the return value of 'watchdog_init_timeout()' for checking the timeout module parameter is in the min-max range. Signed-off-by: Yuechao Zhao --- drivers/hwmon/nct7904.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/hwmon/nct7904.c b/dr

[v1,1/1] watchdog: (nct7904) Fix enable watchdog incorrectly

2020-05-13 Thread yuechao.zhao
From: Yuechao Zhao Use incorrect register to enable watchdog in nct7904_wdt_ping() Signed-off-by: Yuechao Zhao --- drivers/hwmon/nct7904.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hwmon/nct7904.c b/drivers/hwmon/nct7904.c index 04f2a8e..18c95be 100644 --- a/d