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.