On Wed, Dec 13, 2023 at 10:45:23AM +0100, Werner Fischer wrote:
> Commit 893dc8b5c978 ("watchdog: it87: Drop support for resetting watchdog
> though CIR and Game port") removed the try_gameport variable, and left
> max_units setting redundant.
>
> To clean up the code, this patch removes this redundant setting.
>
> Signed-off-by: Werner Fischer <devlists@xxxxxxxx>
Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> ---
> drivers/watchdog/it87_wdt.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/watchdog/it87_wdt.c b/drivers/watchdog/it87_wdt.c
> index 6b7f7ec03633..ca377096bdd7 100644
> --- a/drivers/watchdog/it87_wdt.c
> +++ b/drivers/watchdog/it87_wdt.c
> @@ -274,10 +274,6 @@ static int __init it87_wdt_init(void)
> case IT8712_ID:
> max_units = (chip_rev < 8) ? 255 : 65535;
> break;
> - case IT8716_ID:
> - case IT8726_ID:
> - max_units = 65535;
> - break;
> case IT8607_ID:
> case IT8613_ID:
> case IT8620_ID:
> @@ -287,9 +283,11 @@ static int __init it87_wdt_init(void)
> case IT8655_ID:
> case IT8665_ID:
> case IT8686_ID:
> + case IT8716_ID:
> case IT8718_ID:
> case IT8720_ID:
> case IT8721_ID:
> + case IT8726_ID:
> case IT8728_ID:
> case IT8772_ID:
> case IT8783_ID:
> --
> 2.39.2
>
>
