On 2/2/22 07:52, Sami Mujawar wrote:
>> +AmlCodeGenRdMemory32Fixed (
>> +  BOOLEAN                 IsReadWrite,
>> +  UINT32                  Address,
>> +  UINT32                  RangeLength,
>> +  AML_OBJECT_NODE_HANDLE  NameOpNode,
>> +  AML_DATA_NODE_HANDLE    *NewMemNode
>> +  )
>> +{
>> +  EFI_STATUS                                     Status;
>> +  AML_DATA_NODE                                  *MemNode;
>> +  EFI_ACPI_32_BIT_FIXED_MEMORY_RANGE_DESCRIPTOR  RangeDesc;
>> +
>> +  RangeDesc.Header.Header.Byte = ACPI_32_BIT_FIXED_MEMORY_RANGE_DESCRIPTOR;
>> +  RangeDesc.Header.Length      = 0x09;
> [SAMI] Would it be ok if I change the above line to
> RangeDesc.Header.Length     = sizeof 
> (EFI_ACPI_32_BIT_FIXED_MEMORY_RANGE_DESCRIPTOR) -
>                                     sizeof (ACPI_LARGE_RESOURCE_HEADER);
> before merging?
> [/SAMI]

Yes, that would be good.


Thanks.

Rebecca Cran