On 08/13/2015 06:35 AM, Timur Tabi wrote:

On Thu, Aug 6, 2015 at 3:59 PM, Al Stone [off-list ref] wrote:

quoted

+       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?

I think so, but seems nothing to do with the FADT version,
Al may explain more about it :)

Thanks
Hanjun