[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [RFC PATCH 01/12] libacpi: new DSDT ACPI table for Q35
On Tue, Mar 13, 2018 at 06:10:35AM +1000, Alexey G wrote: > On Mon, 12 Mar 2018 15:38:03 -0400 > Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> wrote: > > >On Tue, Mar 13, 2018 at 04:33:46AM +1000, Alexey Gerasimenko wrote: > >> This patch adds the DSDT table for Q35 (new > >> tools/libacpi/dsdt_q35.asl file). There are not many differences > >> with dsdt.asl (for i440) at the moment, namely: > >> > >> - BDF location of LPC Controller > >> - Minor changes related to FDC detection > >> - Addition of _OSC method to inform OSPM about PCIe features > >> supported > >> > >> As we are still using 4 PCI router links and their corresponding > >> device/register addresses are same (offset 0x60), no need to change > >> PCI routing descriptions. > >> > >> Also, ACPI hotplug is still used to control passed through device hot > >> (un)plug (as it was for i440). > >> > >> Signed-off-by: Alexey Gerasimenko <x1917x@xxxxxxxxx> > >> --- > >> tools/libacpi/dsdt_q35.asl | 551 > >> +++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 551 > >> insertions(+) create mode 100644 tools/libacpi/dsdt_q35.asl > >> > >> diff --git a/tools/libacpi/dsdt_q35.asl b/tools/libacpi/dsdt_q35.asl > >> new file mode 100644 > >> index 0000000000..cd02946a07 > >> --- /dev/null > >> +++ b/tools/libacpi/dsdt_q35.asl > >> @@ -0,0 +1,551 @@ > >> +/****************************************************************************** > >> + * DSDT for Xen with Qemu device model (for Q35 machine) > >> + * > >> + * Copyright (c) 2004, Intel Corporation. > >> + * > >> + * This program is free software; you can redistribute it and/or > >> modify > >> + * it under the terms of the GNU Lesser General Public License as > >> published > >> + * by the Free Software Foundation; version 2.1 only. with the > >> special > >> + * exception on linking described in file LICENSE. > > > >I don't see the 'LICENSE' file in Xen's directory? > > > >Also, your email does not seem to be coming from Intel, so I have to > >ask, where did this file originally come from? > > It's basically Xen's dsdt.asl with some modifications related to Q35. > Currently only few modifications needed, but in the future dsdt.asl and > dsdt_q35.asl will diverge more from each other -- that's the reason why > a separate file was forked instead applying these changes to dsdt.asl > directly, for example, as #ifdef-parts. OK, as such you should make a seperate patch that adds this file (and be completly unmodified) and make sure you CC Intel folks (Kevin, et all) so they can Ack it. Thank you. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |