- the new qla2x00_free_fcport is entirely pointless, please drop that part of the patch (and even if it wasn't pointless it should have been a patch on it's own) - please use struct names and not typedefs for your new structures - pretty much avery single items in your list should be a separate patch. And some of them are actively counterproductive and should be dropped: - Remove direct access of scsi_status field in se_cmd - Remove se_cmd from qlt_do_ctio_completion - Remove se_cmd access in srr code section - Move se_cmd struct outside of qla_tgt_cmd/qla_tgt_mgmt_cmd. -- 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