Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Sender: Guenter Roeck <groeck7@gmail.com>
Message-ID: <f1abf571-c903-ad51-10eb-1e20d2b3ab75@roeck-us.net>
Date:   Wed, 20 Sep 2023 13:14:45 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.1
Subject: Re: [PATCH 4/6] watchdog: ar7_wdt: remove driver to prepare for
 platform removal
Content-Language: en-US
To:     Wolfram Sang <wsa+renesas@sang-engineering.com>,
        linux-mips@vger.kernel.org
Cc:     Jonas Gorski <jonas.gorski@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Wim Van Sebroeck <wim@linux-watchdog.org>,
        linux-watchdog@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20230920201035.3445-1-wsa+renesas@sang-engineering.com>
 <20230920201035.3445-5-wsa+renesas@sang-engineering.com>
From:   Guenter Roeck <linux@roeck-us.net>
In-Reply-To: <20230920201035.3445-5-wsa+renesas@sang-engineering.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 9/20/23 13:10, Wolfram Sang wrote:
> AR7 is going to be removed from the Kernel, so remove its watchdog
> support. This allows us to remove the platform because this driver
> includes a platform specific header.
> 
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

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

> ---
>   drivers/watchdog/Kconfig   |   6 -
>   drivers/watchdog/Makefile  |   1 -
>   drivers/watchdog/ar7_wdt.c | 315 -------------------------------------
>   3 files changed, 322 deletions(-)
>   delete mode 100644 drivers/watchdog/ar7_wdt.c
> 
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index 751458959411..b9f715d73fe8 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -1768,12 +1768,6 @@ config SIBYTE_WDOG
>   	  To compile this driver as a loadable module, choose M here.
>   	  The module will be called sb_wdog.
>   
> -config AR7_WDT
> -	tristate "TI AR7 Watchdog Timer"
> -	depends on AR7 || (MIPS && 32BIT && COMPILE_TEST)
> -	help
> -	  Hardware driver for the TI AR7 Watchdog Timer.
> -
>   config TXX9_WDT
>   	tristate "Toshiba TXx9 Watchdog Timer"
>   	depends on CPU_TX49XX || (MIPS && COMPILE_TEST)
> diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile
> index 7eab9de311cb..7cbc34514ec1 100644
> --- a/drivers/watchdog/Makefile
> +++ b/drivers/watchdog/Makefile
> @@ -168,7 +168,6 @@ obj-$(CONFIG_INDYDOG) += indydog.o
>   obj-$(CONFIG_JZ4740_WDT) += jz4740_wdt.o
>   obj-$(CONFIG_WDT_MTX1) += mtx-1_wdt.o
>   obj-$(CONFIG_SIBYTE_WDOG) += sb_wdog.o
> -obj-$(CONFIG_AR7_WDT) += ar7_wdt.o
>   obj-$(CONFIG_TXX9_WDT) += txx9wdt.o
>   obj-$(CONFIG_OCTEON_WDT) += octeon-wdt.o
>   octeon-wdt-y := octeon-wdt-main.o octeon-wdt-nmi.o
> diff --git a/drivers/watchdog/ar7_wdt.c b/drivers/watchdog/ar7_wdt.c
> deleted file mode 100644
> index cdcaeb0961ac..000000000000