Re: [PATCH 3/8] watchdog: wm831x watchdog really needs mfd

From: Arnd Bergmann
Date: Tue Feb 28 2017 - 17:15:31 EST


On Tue, Feb 28, 2017 at 10:44 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote:
> On Tue, Feb 28, 2017 at 10:36 PM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
>
>>> config WM831X_WATCHDOG
>>> tristate "WM831x watchdog"
>>> - depends on MFD_WM831X || COMPILE_TEST
>>> + depends on MFD_WM831X
>>
>> I prefer the fix proposed by Randy:
>>
>> - depends on MFD_WM831X || COMPILE_TEST
>> + depends on MFD_WM831X || (MFD_WM831X=y && COMPILE_TEST)
>>
>
> Agreed, that's better.

Actually I misread. They should be completely equivalent, the part after
the || has no effect here.

Arnd