On 08/12/2015 04:35 PM, Timur Tabi wrote:
> 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?
>
Argh. Yes, it should. It will always be false, otherwise.
Will fix for next version. Thanks for catching that.
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Linaro Enterprise Group
al.stone@xxxxxxxxxx
-----------------------------------
--
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
