[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87h7odld3x.fsf@manicouagan.localdomain>
Date: Tue, 22 Dec 2020 18:55:46 -0300
From: Thiago Jung Bauermann <bauerman@...ux.ibm.com>
To: Rob Herring <robh@...nel.org>
Cc: Lakshmi Ramasubramanian <nramas@...ux.microsoft.com>,
takahiro.akashi@...aro.org, will@...nel.org,
catalin.marinas@....com, mpe@...erman.id.au, zohar@...ux.ibm.com,
james.morse@....com, sashal@...nel.org, benh@...nel.crashing.org,
paulus@...ba.org, frowand.list@...il.com,
vincenzo.frascino@....com, mark.rutland@....com,
dmitry.kasatkin@...il.com, jmorris@...ei.org, serge@...lyn.com,
pasha.tatashin@...een.com, allison@...utok.net,
masahiroy@...nel.org, bhsharma@...hat.com, mbrugger@...e.com,
hsinyi@...omium.org, tao.li@...o.com, christophe.leroy@....fr,
linux-integrity@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
prsriva@...ux.microsoft.com, balajib@...ux.microsoft.com
Subject: Re: [RFC PATCH 4/4] powerpc: Use common of_kexec_setup_new_fdt()
Rob Herring <robh@...nel.org> writes:
> Signed-off-by: Rob Herring <robh@...nel.org>
> ---
>
> After the IMA changes, delete_fdt_mem_rsv() can also be removed.
>
> arch/powerpc/kexec/file_load.c | 125 ++-------------------------------
> 1 file changed, 6 insertions(+), 119 deletions(-)
Reviewed-by: Thiago Jung Bauermann <bauerman@...ux.ibm.com>
Shouldn't this series also be Cc'd to the linuxppc-dev mailing list?
I just noticed that the ARM64 IMA kexec series hasn't been copying the
linuxppc-dev mailing list, so perhaps this is why this series isn't
doing that, either.
> diff --git a/arch/powerpc/kexec/file_load.c b/arch/powerpc/kexec/file_load.c
> index e452b11df631..956bcb2d1ec2 100644
> --- a/arch/powerpc/kexec/file_load.c
> +++ b/arch/powerpc/kexec/file_load.c
> @@ -16,6 +16,7 @@
>
> #include <linux/slab.h>
> #include <linux/kexec.h>
> +#include <linux/of.h>
> #include <linux/of_fdt.h>
> #include <linux/libfdt.h>
> #include <asm/setup.h>
It's possible to remove the <linux/of_fdt.h> include now.
--
Thiago Jung Bauermann
IBM Linux Technology Center
Powered by blists - more mailing lists