Messages in this thread | | | Date | Fri, 6 Oct 2023 17:27:48 +0800 | Subject | Re: [PATCH 0/5] watchdog: eiois200_wdt: Add EIO-IS200 Watchdog Driver | From | Wenkai <> |
| |
Guenter Roeck 於 10/6/2023 11:02 AM 寫道: > On Thu, Oct 05, 2023 at 04:51:18PM +0800, advantech.susiteam@gmail.com wrote: >> From: Wenkai <advantech.susiteam@gmail.com> >> >> This patch series aims to add support for the Advantech EIO-IS200 >> Embedded Controller's watchdog timer to the Linux kernel. The EIO-IS200 >> is a widely used embedded controller, and this series introduces a >> native driver for its watchdog timer functionality within the Linux >> ecosystem. >> > I am not going to review this patch series. This is just ne watchdog driver. > One patch is sufficient. > > Guenter Hi Guenter,
Advantech's EIO-IS200 watchdog supports 5 output pins: RESET, Power Button, SCI, IRQ, and GPIO. The most traditional scenario is that the Pretimeout triggers IRQ, and the timeout triggers RESET.
However, unfortunately, for industrial usages, there are various use cases, which require certain mechanisms and logic to manage which signal is output when Pretimeout and timeout expire. I am concerned that consolidating all these features into a single patch for upstream may lead to confusion and make the source code less readable and understandable.
Therefore, I have divided the implementation into 5 separate patches, aiming to make the code more comprehensible and acceptable. If it's acceptable to you, I am more than willing to provide a single patch as per your preference.
I would also like to note that this watchdog driver is part of the EIO-IS200 MFD (Multi-Function Device) driver family. It serves as one of the child-drivers of the drivers/mfd/eiois200_core core driver. It's important to mention that without the presence of drivers/mfd/eiois200_core, this child-driver eiois200_wdt cannot be compiled or used.
Should we wait until the core driver (drivers/mfd/eiois200_core) is successfully incorporated before upstreaming the watchdog child-driver, or would you prefer to proceed with these patches independently?
Best regards, Wenkai
| |