Re: [PATCH 0/2] watchdog: mtx-1: Relax build dependencies

Denis Efremov <[email protected]> Thu, 12 Dec 2019 02:46:45 +0300
Newsgroups gmane.linux.kernel,gmane.linux.watchdog,gmane.linux.ports.mips.general
Message-ID <[email protected]>
Hi,

Thanks for the fix.
I tested the compilation with these patches.
You can add my:
Tested-by: Denis Efremov <[email protected]>

Look like this error could be fixed the same way:
In file included from drivers/watchdog/ar7_wdt.c:29:
./arch/mips/include/asm/mach-ar7/ar7.h: In function ‘ar7_is_titan’:
./arch/mips/include/asm/mach-ar7/ar7.h:111:24: error: implicit declaration of function ‘KSEG1ADDR’; did you mean ‘CKSEG1ADDR’? [-Werror=implicit-function-declaration]

On 12.12.2019 00:02, Florian Fainelli wrote:
> Hi Wim, Guenter,
> 
> This came up with Denis trying to fix a MIPS-related build failure:
> 
> https://lore.kernel.org/linux-mips/[email protected]/
> 
> Florian Fainelli (2):
>   watchdog: mtx-1: Drop au1000.h header inclusion
>   watchdog: Relax dependencies for CONFIG_WDT_MTX1
> 
>  drivers/watchdog/Kconfig     | 2 +-
>  drivers/watchdog/mtx-1_wdt.c | 2 --
>  2 files changed, 1 insertion(+), 3 deletions(-)
>

Thanks,
Denis