On Thu, Feb 22, 2024 at 05:03:17PM -0700, Rob Herring wrote: > On Fri, Feb 16, 2024 at 05:05:54PM -0800, Stephen Boyd wrote: > > Call this function unconditionally so that we can populate an empty DTB > > on platforms that don't boot with a firmware provided or builtin DTB. > > When ACPI is in use, unflatten_device_tree() ignores the > > 'initial_boot_params' pointer so the live DT on those systems won't be > > whatever that's pointing to. Similarly, when kexec copies the DT data > > the previous kernel to the new one on ACPI systems, > > of_kexec_alloc_and_setup_fdt() will ignore the live DT (the empty root > > one) and copy the 'initial_boot_params' data. > > > > Cc: Rob Herring <robh...@kernel.org> > > Cc: Frank Rowand <frowand.l...@gmail.com> > > Cc: Catalin Marinas <catalin.mari...@arm.com> > > Cc: Will Deacon <w...@kernel.org> > > Cc: Mark Rutland <mark.rutl...@arm.com> > > Cc: <linux-arm-ker...@lists.infradead.org> > > Signed-off-by: Stephen Boyd <sb...@kernel.org> > > --- > > arch/arm64/kernel/setup.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > Catalin, Will, Can I get an ack on this so I can take the series via the > DT tree.
Mark had strong pretty strong objections to this in version one: https://lore.kernel.org/all/ZaZtbU9hre3YhZam@FVFF77S0Q05N/ and this patch looks the same now as it did then. Did something else change? Will