On Mon, Feb 22, 2016 at 4:16 PM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote: > On Mon, Feb 22, 2016 at 11:16:30AM -0800, Justin Chen wrote: >> On Mon, Feb 22, 2016 at 10:43 AM, Jean Delvare <jdelvare@xxxxxxx> wrote: >> > Hello Justin, >> > >> > Le Monday 22 February 2016 à 09:58 -0800, Justin Chen a écrit : >> >> This driver is based off of a Broadcom watchdog block, it is not tied >> >> to the MIPS core. >> >> The watchdog driver is named BCM7038 because that was the first chip >> >> that featured this watchdog block. Their are other chips that use this >> >> driver that are not MIPS based. >> > >> > The Kconfig help text doesn't say this at all. With the current wording >> > there is no reason why anyone would enable this option if not building a >> > kernel for the Broadcom BCM7038. >> Sorry about that! I can submit a patch to fix this. >> > >> > Which other systems reuse this watchdog block exactly? In the kernel >> > tree I can't see anyone instantiating this device. >> BRCMSTB chips use this block as well as other BCM chips. >> >> "This driver is for a watchdog block contained in all Broadcom Set-top >> Box chips since BCM7038. BCM7038 was made public during the 2004 CES, >> and since then, many chips use this watchdog block including some cable >> modem chips." > Hello Guenter, > That is a bit vague. Can you be more specific ? > > Also, while the driver may not be limited to MIPS, I am relatively sure > that it has some architecture limits. MIPS, ARM, and ARM64, maybe ? Yes you are correct, this driver would be limited to Broadcom chips. I would suggest "depends on ARCH_BRCMSTB || BMIPS_GENERIC || COMPILE_TEST" > > Thanks, > Guenter Thanks, Justin -- 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