Hi Graeme,

Thanks for the comments.  I will wait for Al Stone reply.  My comments are as below

On 19 March 2014 14:38, Graeme Gregory <graeme.gregory@linaro.org> wrote:
Hi Naresh,

Why did we merge these when we decided DBG2 was not suitable for use
on arm? I am confused and I think it will also confuse anyone using
acpi-asl for reference!
I am not sure, Did I miss any discussion about DBG2 with-in Linaro ACPI group.  
I have discussed regarding these patches with Al Stone yesterday, we decided it is safe to support whether or not to use it for early console.

In fact having looked at the tables they are invalid anyway as they
reference a device on PCI.
agreed. It's a template. 

Graeme

On Wed, Mar 19, 2014 at 02:07:30PM +0530, Naresh Bhat wrote:
>    Hi Al Stone,
>    Thanks. I have merged DBG2 template, compilation support patches into
>    acpi-asl git repo[1]https://git.linaro.org/leg/acpi/acpi-asl.git
>    Regards
>    -Naresh
>    On 10 January 2014 03:34, Al Stone <[2]al.stone@linaro.org> wrote:
>
>    On 12/16/2013 03:55 AM, [3]naresh.bhat@linaro.org wrote:
>
>      From: Naresh Bhat <[4]naresh.bhat@linaro.org>
>      The DBG2 table support in iASL compiler will be
>      available in the next release of acpica
>      [5]https://github.com/acpica/acpica/commit/278cbb119f090d3aad0480381
>      126c1ae8e59d66b
>      the following set of patches support the DBG2 table template in
>      acpi-asl tool.
>      Naresh Bhat (4):
>      ACPI / ASL: Add DBG2 table template
>      DBG2: Update platforms manifest
>      DBG2: Update xsdt entries across platforms
>      DBG2: Support DBG2 table compilation
>      platforms/exynos5250-arndale.acpi/dbg2.asl | 106
>      ++++++++++++++++++++
>      .../exynos5250-arndale.manifest | 1 +
>      platforms/exynos5250-arndale.acpi/xsdt.asl | 1 +
>      platforms/foundation-v8.acpi/dbg2.asl | 106 ++++++++++++++++++++
>      .../foundation-v8.acpi/foundation-v8.manifest | 1 +
>      platforms/foundation-v8.acpi/xsdt.asl | 1 +
>      platforms/rtsm_ve-aemv8a.acpi/dbg2.asl | 106 ++++++++++++++++++++
>      .../rtsm_ve-aemv8a.acpi/rtsm_ve-aemv8a.manifest | 1 +
>      platforms/rtsm_ve-aemv8a.acpi/xsdt.asl | 1 +
>      tools/common/build_aml.c | 1 +
>      tools/common/check_aml.c | 1 +
>      tools/common/include/check_aml.h | 1 +
>      12 files changed, 327 insertions(+)
>      create mode 100644 platforms/exynos5250-arndale.acpi/dbg2.asl
>      create mode 100644 platforms/foundation-v8.acpi/dbg2.asl
>      create mode 100644 platforms/rtsm_ve-aemv8a.acpi/dbg2.asl
>
>      Now that DBG2 support has been accepted in ACPICA and published,
>      and that acpica-tools has been updated to the 20131215 version
>      that contains the fixes needed, these patches can be safely
>      merged in.
>      Acked-by: Al Stone <[6]al.stone@linaro.org>
>      --
>      ciao,
>      al
>      -----------------------------------
>      Al Stone
>      Software Engineer
>      Linaro Enterprise Group
>      [7]al.stone@linaro.org
>      -----------------------------------
>
> References
>
>    1. https://git.linaro.org/leg/acpi/acpi-asl.git
>    2. mailto:al.stone@linaro.org
>    3. mailto:naresh.bhat@linaro.org
>    4. mailto:naresh.bhat@linaro.org
>    5. https://github.com/acpica/acpica/commit/278cbb119f090d3aad0480381126c1ae8e59d66b
>    6. mailto:al.stone@linaro.org
>    7. mailto:al.stone@linaro.org

> _______________________________________________
> Linaro-acpi mailing list
> Linaro-acpi@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/linaro-acpi