On 30/08/16 16:15, Jan Beulich wrote:
On 30.08.16 at 15:18, <julien.gr...@arm.com> wrote:
On 30/08/16 14:09, Jan Beulich wrote:
On 30.08.16 at 14:50, <julien.gr...@arm.com> wrote:
If you want to add it, it should be a separate patch and we should go
further by removing the section in the linker script.

While I don't think this needs to be broken out, it certainly can (but
then really there should have been a conditional like the one I
introduce from the beginning, to avoid having dead code in a non-
ACPI enabled binary).

It was an oversight while reviewing the patch adding acpi_device_init
and I thought you added CONFIG_ACPI just for comestic.

So with that, do you then still require me to split this off?

No, I am fine with you keep it here.

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to