From: Andrew Morton ERROR: do not use C99 // comments #102: FILE: drivers/scsi/libsas/sas_ata.c:289: + //uh... hopefully there's no commands left in here? total: 1 errors, 0 warnings, 127 lines checked Your patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: "Darrick J. Wong" Cc: Jeff Garzik Signed-off-by: Andrew Morton --- drivers/scsi/libsas/sas_ata.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/scsi/libsas/sas_ata.c~libsas-convert-ata-bridge-to-use-new-eh-checkpatch-fixes drivers/scsi/libsas/sas_ata.c --- a/drivers/scsi/libsas/sas_ata.c~libsas-convert-ata-bridge-to-use-new-eh-checkpatch-fixes +++ a/drivers/scsi/libsas/sas_ata.c @@ -286,7 +286,7 @@ static int sas_ata_soft_reset(struct ata static void sas_ata_error_handler(struct ata_port *ap) { ata_do_eh(ap, NULL, sas_ata_soft_reset, NULL, NULL); - //uh... hopefully there's no commands left in here? + /* uh... hopefully there's no commands left in here? */ } static void sas_ata_post_internal(struct ata_queued_cmd *qc) _