On Wed, 2009-03-04 at 12:17 +0530, Chauhan, Vijay wrote:
> Hi Hannes,
>
> Thanks for your comment. I have modified the patch with correct indentation. Resubmitting this patch.
> For other ASC/ASQ codes (ie 04/04 - 04/09), I am currently working on it and will be adding it
> in future if needed.
checkpatch.pl doesn't like you very much:
jejb@mulgrave> ./scripts/checkpatch.pl ~/tmp.mail
ERROR: spaces required around that '=' (ctx:VxW)
#135: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:464:
+ err= SCSI_DH_RETRY;
^
WARNING: line over 80 characters
#154: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:482:
+ "MODE_SELECT failed with sense %02x/%02x/%02x.\n",sense_hdr.sense_key,
ERROR: space required after that ',' (ctx:VxV)
#154: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:482:
+ "MODE_SELECT failed with sense %02x/%02x/%02x.\n",sense_hdr.sense_key,
^
ERROR: space prohibited before that close parenthesis ')'
#155: FILE: drivers/scsi/device_handler/scsi_dh_rdac.c:483:
+ sense_hdr.asc, sense_hdr.ascq );
total: 3 errors, 1 warnings, 54 lines checked
I fixed all of this, but could you try running checkpatch.pl before
submitting, next time.
James
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
