[PATCH v3 3/5] RISC-V: Initial DTS for Microchip ICICLE board

[Damien Le Moal]

On 2020/12/08 0:55, Vitaly Wool wrote:
> Hi Atish,
> 
>> Add initial DTS for Microchip ICICLE board having only
>> essential devices (clocks, sdhci, ethernet, serial, etc).
>> The device tree is based on the U-Boot patch.
>>
>> https://patchwork.ozlabs.org/project/uboot/patch/20201110103414.10142-6-> padmarao.begari at microchip.com/
>>
>> Signed-off-by: Atish Patra <atish.patra at wdc.com>
>> ---
>>  arch/riscv/boot/dts/Makefile                  |   1 +
>>  arch/riscv/boot/dts/microchip/Makefile        |   2 +
>>  .../microchip/microchip-mpfs-icicle-kit.dts   |  72 ++++
>>  .../boot/dts/microchip/microchip-mpfs.dtsi    | 331 ++++++++++++++++++
> 
> would it be possible to rename dts file to
> microchip_mpfs_icicle_kit.dts? Dashes will not work with
> SOC_BUILTIN_DTB_DECLARE(), and I do want to use built-in device tree
> for XIP on Icicle. Thanks! :)

Please see the patch series I sent for the Canaan Kendryte K210 support: I want
to retire SOC_BUILTIN_DTB_DECLARE() as it is broken: the arch, vendor and imp ID
given by an SOC does not necessarily reflect the fact that the SoC is used on
different boards that need different DT. That makes differentiation of the
builtin DT for different boards using multiple SOC_BUILTIN_DTB_DECLARE()
declaration impossible. In short, SOC_BUILTIN_DTB_DECLARE() does not work.

http://lists.infradead.org/pipermail/linux-riscv/2020-December/003497.html

In the K210 series, I have this patch:

http://lists.infradead.org/pipermail/linux-riscv/2020-December/003498.html

that replaces SOC_BUILTIN_DTB_DECLARE() with a more traditional
BUILTIN_DTB_SOURCE config option. That is a lot more simple and a lot more solid.

> 
> Best regards,
>    Vitaly
> 
> _______________________________________________
> linux-riscv mailing list
> linux-riscv at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-riscv
> 


-- 
Damien Le Moal
Western Digital Research