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

[Arnd Bergmann]

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