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

Reply via email to