On 06/10/2017 04:29 PM, Eugeniu Rosca wrote: > On Sat, Jun 03, 2017 at 10:14:54PM +0200, Eugeniu Rosca wrote: >> Implement WDIOC_GETBOOTSTATUS in the test code to be able to check if >> watchdog drivers have proper support of WDIOF_CARDRESET. If this support >> is in place, then watchdog-test gets the ability to check the status/reason >> of the last boot (POR/Watchdog). Aside from the main feature, include a >> number of fixes following the spirt of v1 and v2 comments. >> >> v3: >> - Fix checkpatch.pl --strict --max-line-length=120 -f watchdog-test.c >> - Switch from manual argv[] parsing to getopt() >> - Include minor fixes >> - Convert `--bootstatus` into a "oneshot" command (skip keepalive loop) >> > > Hello Shuah, > > I would really appreciate any kind of feedback, either positive or > negative, on the provided patches. I don't think silence is a > constructive way of working between community members. > Hi Eugeniu, I have several patches sitting in my Inbox. I was out sick a few days last week. It sometimes takes longer than a week to get feedback, please keep that in mind. I will review your series and get back to you in a day or two. I have a few fixes patches I need to review. thanks, -- Shuah -- To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html