Re: [Linaro-acpi] [PATCH 1/5] ACPI: add in a bad_madt_entry() function to eventually replace the macro

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

On Thu, Aug 6, 2015 at 3:59 PM, Al Stone al.stone@linaro.org 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?

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

Thanks Hanjun