On 02/28/17 15:16, Guenter Roeck wrote: > On Tue, Feb 28, 2017 at 10:48:27PM +0100, Arnd Bergmann wrote: >> 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. >> > Yes, I figured that out later as well. Agreed. Arnd's is simpler. My patch might keep someone else from making the same mistake. ;) -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
