From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47320) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZZX7K-0007UO-4P for qemu-devel@nongnu.org; Wed, 09 Sep 2015 00:36:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZZX7G-00070s-UK for qemu-devel@nongnu.org; Wed, 09 Sep 2015 00:36:18 -0400 Received: from szxga01-in.huawei.com ([58.251.152.64]:40998) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZZX7G-0006zD-4Z for qemu-devel@nongnu.org; Wed, 09 Sep 2015 00:36:14 -0400 Message-ID: <55EFB4C9.7080300@huawei.com> Date: Wed, 9 Sep 2015 12:25:45 +0800 From: Shannon Zhao MIME-Version: 1.0 References: <1441635826-4866-1-git-send-email-leif.lindholm@linaro.org> <55EE5383.40008@huawei.com> <20150908130438.GL10728@bivouac.eciton.net> In-Reply-To: <20150908130438.GL10728@bivouac.eciton.net> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] ACPI/arm-virt: add DBG2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Leif Lindholm Cc: peter.maydell@linaro.org, drjones@redhat.com, qemu-devel@nongnu.org, al.stone@linaro.org, shannon.zhao@linaro.org On 2015/9/8 21:04, Leif Lindholm wrote: > On Tue, Sep 08, 2015 at 11:18:27AM +0800, Shannon Zhao wrote: >> On 2015/9/7 22:23, Leif Lindholm wrote: >>> The Debug Port Table 2 (DBG2) is mandated by the ARM Server Base Boot >>> Requirements specification. Add the DBG2 table definitions, and set up >>> an entry in the ARM virt machine for the pl011 UART. >> >> Looking at Documentation/arm64/acpi_object_usage.txt in Linux kernel, it >> says >> " >> DBG2 Signature Reserved (signature == "DBG2") >> == DeBuG port table 2 == >> Microsoft only table, will not be supported. >> " >> It seems that Linux kernel doesn't support or need it, but Windows >> requires it. So does it need to test this on Windows? > > No, it can be tested under Linux with the set I just sent out: > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-September/368614.html So you need to change Documentation/arm64/acpi_object_usage.txt as well. But that is kernel side. > But I needed this set for QEMU for the kernel people to have something > to test :) > > That said, I would be overjoyed if someone _could_ test it on Windows. > > / > Leif > >>> Leif Lindholm (2): >>> ACPI: Add definitions for the DBG2 table >>> hw/arm/virt-acpi-build: Add DBG2 table >>> >>> hw/arm/virt-acpi-build.c | 37 ++++++++++++++++++++++++++++++++++++- >>> include/hw/acpi/acpi-defs.h | 37 +++++++++++++++++++++++++++++++++++-- >>> 2 files changed, 71 insertions(+), 3 deletions(-) >>> >> >> -- >> Shannon >> > > -- Shannon