Messages in this thread | | | Subject | Re: [PATCH v3] watchdog: Add watchdog timer support for the WinSystems EBC-C384 | From | Guenter Roeck <> | Date | Tue, 26 Jan 2016 07:30:22 -0800 |
| |
On 01/26/2016 06:31 AM, William Breathitt Gray wrote: > On 01/25/2016 5:11 PM, Guenter Roeck wrote: >>> + dev_warn(dev, "Invalid timeout (%u seconds), using default (%u seconds)\n", >>> + timeout, WATCHDOG_TIMEOUT); >> >> Multi-line alignment is off by one character. > > I used tabs to align the lines to 8-character boundaries. Should I add > an additional space to align the "timeout" argument with the "dev" > argument? >
Continuation line alignment should be with "(", so, yes, on the same column as "dev".
Thanks, Guenter
| |