Re: [PATCH v3] watchdog: Add watchdog timer support for the WinSystems EBC-C384

[Guenter Roeck]

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