On 26 July 2016 at 08:03, kwangwoo....@sk.com <kwangwoo....@sk.com> wrote: > Hi Peter, > > Please, check the comments below. Thanks a lot! > >> -----Original Message----- >> From: Peter Maydell [mailto:peter.mayd...@linaro.org] >> This seems to be missing code to write the device tree >> information about whatever this device is? > > Is it OK to just add a memory region which cannot be used without ACPI? > This is unclear to me. If you suggest that it is better way, I'll revise > this patch to add a device tree node. Please, help me to understand.
They don't necessarily both have to be added in the exact same patch if it's less confusing to split it, but the series as a whole should support both DT and ACPI (and your kernel patches also ought to have both DT and ACPI bindings I think). thanks -- PMM
