On Thu, Aug 6, 2015 at 3:59 PM, Al Stone <al.stone@xxxxxxxxxx> wrote: > + while (ms->num_types != 0) { > + if (ms->major_version == major && > + ms->minor_version == minor && > + ms->madt_version == madt->header.revision) > + break; > + ms++; > + } > + if (!ms) { Shouldn't this be if (!ms->num_types) { instead? -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project. -- 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