On Wed, 2010-02-03 at 12:21 -0500, Mark Lord wrote:..Joe Perches wrote:[]String constants that are continued on subsequent lines with \
are not good.
- dev_warn(ap->dev, "Failed to compute ATA timing %d, \NAK. Just stick the ret); part onto the same line and be done with it.
- set PIO_0 timing\n", ret);
+ dev_warn(ap->dev, "Failed to compute ATA timing %d, set PIO_0 timing\n",
+ ret);
:resend that I mistakenly sent only to Mark, with a couple more comments
I disagree.
Look at the long line wrapping style of the rest of the file
and most of the kernel source. No additional arguments to
functions are generally used after column 80.
If it's ever agreed that all lines > 80 cols are OK or
new args after column 80 are OK, then sure. Until then,
if you want it, you should do it.