On 8 June 2015 at 08:19, fu.wei@linaro.org wrote:
From: Fu Wei fu.wei@linaro.org
The old "Platform Timer Offset" value is miscalculated, it should include the byte length of "Platform Timer Offset". The definition of "Platform Timer Offset" is "Offset to the Platform Timer Structure[] array from the start of this table".
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Wei fu.wei@linaro.org
Platforms/ARM/VExpress/AcpiTables/rtsm_ve-aemv8a/gtdt.asl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Platforms/ARM/VExpress/AcpiTables/rtsm_ve-aemv8a/gtdt.asl b/Platforms/ARM/VExpress/AcpiTables/rtsm_ve-aemv8a/gtdt.asl index d304243..0cb24c5 100644 --- a/Platforms/ARM/VExpress/AcpiTables/rtsm_ve-aemv8a/gtdt.asl +++ b/Platforms/ARM/VExpress/AcpiTables/rtsm_ve-aemv8a/gtdt.asl @@ -86,7 +86,7 @@ [0008] CntReadBase Physical address : 0000000000000000
[0004] Platform Timer Count : 00000001 -[0004] Platform Timer Offset : 0000005C +[0004] Platform Timer Offset : 00000060
I will add for info Offset type fields are generated automatically in iasl so this is probably why no-one has seen an issue. It is best practice though to make them correct!
Graeme
/* Memory-mapped GT (Generic Timer) structures */ [0001] Subtable Type : 00 -- 1.9.1
Linaro-uefi mailing list Linaro-uefi@lists.linaro.org https://lists.linaro.org/mailman/listinfo/linaro-uefi