On 12/8/21 5:53 PM, cgel.zte@xxxxxxxxx wrote:
From: chiminghao <chi.minghao@xxxxxxxxxx>
return value form directly instead of
taking this in another redundant variable.
Reported-by: Zeal Robot <zealci@xxxxxxxxxx>
Signed-off-by: chiminghao <chi.minghao@xxxxxxxxxx>
No, this is an old watchdog driver. Functional fixes for those only, please.
Guenter
---
drivers/watchdog/pcwd_pci.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/watchdog/pcwd_pci.c b/drivers/watchdog/pcwd_pci.c
index 54d86fcb1837..edbc0f28d399 100644
--- a/drivers/watchdog/pcwd_pci.c
+++ b/drivers/watchdog/pcwd_pci.c
@@ -215,10 +215,7 @@ static inline void pcipcwd_check_temperature_support(void)
static int pcipcwd_get_option_switches(void)
{
- int option_switches;
-
- option_switches = inb_p(pcipcwd_private.io_addr + 3);
- return option_switches;
+ return inb_p(pcipcwd_private.io_addr + 3);
}
static void pcipcwd_show_card_info(void)
