On Fri, Jun 7, 2013 at 9:01 PM, Lv Zheng <lv.zheng@xxxxxxxxx> wrote: > Sort list based on the C-state, remove invalid/zero entries. > ACPICA BZ 890. Lv Zheng. The commit message should not just say what the code does, it should more importantly answer WHY the change was made, and why anybody should care about the patch. I think the answer here is that we are adding _CST sanity checking -- presumably to deal with buggy AML? What will we see happen after this patch that we did not see before it? Nacked-by: Len Brown <len.brown@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html