From mboxrd@z Thu Jan  1 00:00:00 1970
From: Masahiro Yamada <yamada.masahiro@socionext.com>
Date: Fri, 21 Oct 2016 10:58:03 +0900
Subject: [U-Boot] [PATCH v2] libfdt: replace ARCH_FIXUP_FDT with
	ARCH_FIXUP_FDT_MEMORY
In-Reply-To: <1476964504.2577.4.camel@synopsys.com>
References: <1476922559-22084-1-git-send-email-yamada.masahiro@socionext.com>
 <1476964504.2577.4.camel@synopsys.com>
Message-ID: <CAK7LNAQ-43U6_WD4p760-Qa+CS_QSQ+7LXww18zHhEDWr0qx4Q@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Alexey,

2016-10-20 20:55 GMT+09:00 Alexey Brodkin <Alexey.Brodkin@synopsys.com>:
> Hello Masahiro-san,
>
> On Thu, 2016-10-20 at 09:15 +0900, Masahiro Yamada wrote:
>> Commit e2f88dfd2d96 ("libfdt: Introduce new ARCH_FIXUP_FDT option")
>> allows us to skip memory setup of DTB, but a problem for ARM is that
>> spin_table_update_dt() and psci_update_dt() are skipped as well if
>> CONFIG_ARCH_FIXUP_FDT is disabled.
>>
>> This commit allows us to skip only fdt_fixup_memory_banks() instead
>> of the whole of arch_fixup_fdt().  It will be useful when we want to
>> use a memory node from a kernel DTB as is, but need some fixups for
>> Spin-Table/PSCI.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>> ---
>>
>> Changes in v2:
>>  - Add empty stub to ARC, PowerPC, Microblaze instead of
>>    a weak function common/image-fdt.c
>
> [snip]
>
>> diff --git a/arch/arc/lib/bootm.c b/arch/arc/lib/bootm.c
>> index 04d9d9c..5798149 100644
>> --- a/arch/arc/lib/bootm.c
>> +++ b/arch/arc/lib/bootm.c
>> @@ -37,6 +37,11 @@ void arch_lmb_reserve(struct lmb *lmb)
>>       lmb_reserve(lmb, sp, (CONFIG_SYS_SDRAM_BASE + gd->ram_size - sp));
>>  }
>>
>> +int arch_fixup_fdt(void *blob)
>> +{
>> +     return 0;
>> +}
>> +
>
> I'm wondering why don't we add weak implementation of arch_fixup_fdt()
> right in say common/image-fdt.c? This will allow us to not add dummy stubs
> for those arches that don't really use it.


I fully agree with you.


I used a weak function in v1:
http://patchwork.ozlabs.org/patch/678049/

But, it was change-request'ed by Simon.


-- 
Best Regards
Masahiro Yamada