Re: [PATCH 3/3] watchdog: bcm63xx_wdt: support BCM4908
From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2021-10-27 22:29:00
Also in:
linux-arm-kernel, linux-mips, linux-watchdog
From: Florian Fainelli <f.fainelli@gmail.com>
Date: 2021-10-27 22:29:00
Also in:
linux-arm-kernel, linux-mips, linux-watchdog
On 10/27/21 3:22 PM, Guenter Roeck wrote:
On Wed, Oct 27, 2021 at 11:55:31PM +0200, Rafał Miłecki wrote:quoted
From: Rafał Miłecki <rafal@milecki.pl> So far bcm63xx_wdt included support for Broadcom's watchdog block of old MIPS devices only. It was also a fully platform (non-DT) driver. The same block was recently found on BCM4908 with just a slightly different registers layout. Extend this driver to support OF and add support for per-chipset registers offsets. Signed-off-by: Rafał Miłecki <rafal@milecki.pl>Please convert the driver to use the watchdog subsystem first. Also, when doing so, please explain why you don't use the bcm7038 driver instead, since it serves the same hardware interface and already supports the watchdog core.
This is too ugly unfortunately so yes let us use bcm7038_wdt.c and let me know if you need help with converting arch/mips/bcm63xx to use the bcm7038_wdt.c -- Florian