On 20/04/16 17:04, Jeremy Linton wrote:
On 04/20/2016 10:54 AM, Sudeep Holla wrote:
On 20/04/16 16:44, Jeremy Linton wrote:
On 04/20/2016 10:35 AM, Sudeep Holla wrote:
[...]
Yes I got bitten by that and I failed to notice it :). I have fixed it
locally and tested correctly now.
DWordIo ( // IO window
ResourceProducer,
MinFixed,
MaxFixed,
PosDecode,
EntireRange,
0x00000000, // Granularity
0x00000000, // Min Base Address
0x007fffff, // Max Base Address
0x5f800000, // Translate
0x00800000, // Length
,
,
,
This is exactly what I have now after fixing the length and works fine,
but ...
TypeTranslation
... this is TypeStatic which I assume is default when not specified.
I tried TypeTranslation too and it works fine. It adds DenseTranslation
by default. Do you want me to add TypeTranslation?
That is your call, but I believe that TypeTranslation is correct, if you
read table 6-214 in ACPI6 (I/O Resource Flag (resource Type=1)
Definitions). That table to be the clearest statement of intent for the
field and the one I would refer to.
Agreed, I didn't see that table.
--
Regards,
Sudeep
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel
