On Thu, Sep 22, 2011 at 9:55 AM, Mark Salyzyn <mark_salyzyn@xxxxxxxxxxxxxx> wrote: > Except for the fact the patch as located in the attachment is still not in compliance, the patch works for me. > > Sincerely -- Mark Salyzyn > >> scsi-misc-2.6/scripts/checkpatch.pl 0001-SCSI-libsas-Allow-expander-T-T-attachments.patch > WARNING: line over 80 characters > #57: FILE: drivers/scsi/libsas/sas_expander.c:1245: > + if (child_phy->routing_attr == SUBTRACTIVE_ROUTING || > > WARNING: line over 80 characters > #59: FILE: drivers/scsi/libsas/sas_expander.c:1247: > + child_ex->t2t_supp && parent_ex->t2t_supp)) { > > WARNING: line over 80 characters > #62: FILE: drivers/scsi/libsas/sas_expander.c:1250: > + sas_print_parent_topology_bug(child, parent_phy, child_phy); > > total: 0 errors, 3 warnings, 82 lines checked > Minor infractions of 80 columns are acceptable when they improve readability. checkpactch clean to me is no errors. -- Dan -- 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