Eric Blake <[email protected]> writes: > On 10/02/2017 06:42 AM, [email protected] wrote: >> Hi, >> >> This series seems to have some coding style problems. See output below for >> more information: >> > >> Checking PATCH 2/3: watchdog.h: Drop local redefinition of actions enum... >> ERROR: braces {} are necessary for all arms of this statement >> #57: FILE: hw/watchdog/watchdog.c:86: >> + if (action < 0) >> [...] > > Pre-existing problem, but would indeed be easy to fix up as part of this > series. I don't know if that means Markus should respin the pull request?
I would've touched it up in my tree had I seen it in time, but I doubt it's worth the bother now.
