Re: [PATCH] watchdog: Add hardware dependency to BCM7038 driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux