On Mon, Jun 30, 2014 at 05:19:57PM -0700, Adam Radford wrote: > James/linux-scsi, > > The following patch for megaraid_sas fixes the LD/VF affiliation policy parsing code to account for LD targetId's and Hidden LD's (not yet affiliated with any Virtual Functions). Hi Adam, this doesn't address Martins comment about removing the initial != 0 case. And while we're at it there basically is no significant shared code between the PlasmaFW111 and !PlasmaFW111, so split these into two different functions. With those two levels of indentation removed the code might actually become readable, too. Also please remove the "James/linux-scsi," address in the individual patches, that's something that should just go into the cover letter, and make sure to include all "Reviewed-by:" and "Acked-by:" tags when resending already reviewd patches. Thanks, Christoph -- 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
