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

[Lakshmi Ramasubramanian]

On 12/22/20 1:55 PM, Thiago Jung Bauermann wrote:
> 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.

Thanks for pointing that out Thiago.
I will copy linuxppc-dev mailing list (linuxppc-dev@xxxxxxxxxxxxxxx) 
when I post v14 of my patch set.

 -lakshmi

> > 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.