Randy Dunlap wrote: >> - ata_dev_printk(dev, KERN_WARNING, >> -"Drive reports diagnostics failure. This may indicate a drive\n"); >> - ata_dev_printk(dev, KERN_WARNING, >> -"fault or invalid emulation. Contact drive vendor for information.\n"); >> - } > > Looks to me like several of these + lines have indent problems: > following lines (i.e., not first line) of function call should be > indented more than the first line: Putting one more tab would put good part of those lines off the 80col limit. Hmmm... but I agree it looks ugly. I'll add one or two spaces there. >> + ata_dev_printk(dev, KERN_WARNING, >> + "Drive reports diagnostics failure. This may indicate a drive\n" >> + "fault or invalid emulation. Contact drive vendor for information.\n"); Thanks. -- tejun - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html