On Fri, 2008-03-21 at 19:00 +0800, Ke Wei wrote: > and remove some unused members from struct. This patch (and the following ones) look great. I'm afraid I can't apply them, though ... it looks like your mail tool has stripped some leading spaces: > @@ -37,11 +37,13 @@ > #include <linux/dma-mapping.h> > #include <linux/ctype.h> > #include <scsi/libsas.h> > +#include <scsi/scsi_tcq.h> > +#include <scsi/sas_ata.h> > #include <asm/io.h> And also added line breaks: > + PHY_MODE6_MUCNT_EN = (1U << 24), /* u Count Enable */ > + PHY_MODE6_SEL_MUCNT_LEN = (1U << 22), /* Training Length > Select */ > + PHY_MODE6_SELMUPI = (1U << 20), /* Phase Multi Select (init) */ If you need to add the patches as an attachment to get them through your mailer undamaged, that's fine ... the tools we have will cope with that. Thanks, 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