Re: [RFC PATCH 4/4] powerpc: Use common of_kexec_setup_new_fdt()

From: Thiago Jung Bauermann
Date: Tue Dec 22 2020 - 16:57:27 EST



Rob Herring <robh@xxxxxxxxxx> writes:

> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
>
> 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@xxxxxxxxxxxxx>

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