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

[Lakshmi Ramasubramanian <nramas@linux.microsoft.com>]

On 12/22/20 1:55 PM, Thiago Jung Bauermann wrote:
> 
> Rob Herring <robh@kernel.org> writes:
> 
>> Signed-off-by: Rob Herring <robh@kernel.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@linux.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.

Thanks for pointing that out Thiago.
I will copy linuxppc-dev mailing list (linuxppc-dev@vger.kernel.org) 
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.
>