Daniel, Thanks for viewing the patches. I did run the checkpatch.pl first before I submit the patches. Is there possible you can tell me the errors in the specific patch, I need to find out if it is my e-mail setting problems or something else. Thanks, Bo Yang -----Original Message----- From: Daniel Walker [mailto:dwalker@xxxxxxxxxx] Sent: Sunday, September 20, 2009 11:41 PM To: Yang, Bo Cc: James.Bottomley@xxxxxxxxxxxxxxxxxxxxx; James.Bottomley@xxxxxxx; linux-scsi@xxxxxxxxxxxxxxx; akpm@xxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx Subject: Re: [PATCH 1/12] scsi: megaraid_sas - tape drive support fix Your patches all have numerous serious style problems.. There are far too many to even give a summary here, and they are all different kinds of problems.. You would need to fix all these issues before these patches could move forward. Could you run scripts/checkpatch.pl on you patches and fix all the issues that it reports? Daniel -- 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