On 01/11/2013 05:43 PM, wenxiong@xxxxxxxxxxxxxxxxxx wrote: > This is version 2 of ipr patches to support new IBM SAS controllers. > In V2, we have fixed the following suggestions/warning/sparse errors: > > 1.Changed simple_strtoul() to kstrtoul() in ipr_restore_iopoll_weight routine. > 2.Removed the __dev annotations. > 3.Fixed unlock bugs which are caused by my previous patches(reported by sparse). > 4.BUG_ON gcc 4.7 warning. > 5.Fixed sparse error in original ipr driver. James, I've pulled down the series, made sure it builds on x86 with a recent gcc, as well as ran them through sparse and smatch. This should address the issues previously encountered with the patch set. Additionally, it should ensure the series is bisectable. The only thing I noticed was that the subject line has the V2 outside of []. Would you like Wendy to resend the series to fix this in order to make it easier to apply? Acked-by: Brian King <brking@xxxxxxxxxxxxxxxxxx> Thanks, Brian -- Brian King Power Linux I/O IBM Linux Technology Center -- 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