Re: [PATCH 02/11] watchdog: gemini/ftwdt010: rename driver and symbols

[Guenter Roeck <linux@roeck-us.net>]

On Thu, Aug 24, 2017 at 10:45:27PM +0200, Linus Walleij wrote:
> On Mon, Aug 14, 2017 at 5:01 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> > On Sat, Aug 12, 2017 at 08:43:09PM +0200, Linus Walleij wrote:
> 
> >> -config GEMINI_WATCHDOG
> >> -     tristate "Gemini watchdog"
> >> -     depends on ARCH_GEMINI
> >> +config FTWDT010_WATCHDOG
> >> +     tristate "Faraday Technology FTWDT010 watchdog"
> >> +     depends on ARM || COMPILE_TEST
> >
> > Did you test this ? I am kind of wary about COMPILE_TEST in watchdog drivers.
> > I tried to enable that a while ago, and we had to spend a couple of releases
> > fixing the fallout (even though I had tried to make sure that everything
> > builds).
> 
> Incidentally, yes!
> 
Ok. Note that I though that too, only 0day didn't pick up all corner cases.

Guenter

> I push my tree to kernelorg where the zeroday autobuilders pick it up
> and compile all kind of archs.
> 
> I'm pushing the updated patches too.
> 
> The big culprit used to be um linux, since it doesn't do MMIO.
> But I think that has been solved recently because I don't see
> complaints about it anymore.
> 
> Yours,
> Linus Walleij
> --
> To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html