[Buildroot] [PATCH v2 4/9] board: Add Sipeed MAIX-bit support

[Damien Le Moal]

On 2021/07/24 7:35, Thomas Petazzoni wrote:
> Hello,
> 
> On Sat, 24 Jul 2021 00:25:27 +0200
> Arnout Vandecappelle <arnout at mind.be> wrote:
> 
>>> +++ b/board/canaan/k210-common/busybox-tiny-initramfs.config  
>>
>>  Wouldn't it be better to start from busybox-minimal.config and just add a
>> fragment that removes the unnecessary options?
> 
> Also, we already have a package/busybox/busybox-minimal.config. I'm not
> sure we want to add zillions of "minimal" Busybox configurations.

OK. In this case, this is all about saving memory and getting that busybox
process as small as possible, given the super low amount of free memory
available after boot.

The toolchain do not allow to compile "shared library" for NOMMU (shared text
segment). This is not supported for RV64. Once that is done, someday, we can use
the default minimal config as forking shells will be cheaper in memory.

> 
>>> +++ b/configs/sipeed_maix_bit_defconfig
>>> @@ -0,0 +1,23 @@
>>> +BR2_riscv=y
>>> +# BR2_COMPILER_PARANOID_UNSAFE_PATH is not set
> 
> This should remain the default.

OK.

> 
>>> +BR2_KERNEL_HEADERS_5_13=y
>>> +# BR2_UCLIBC_INSTALL_UTILS is not set
> 
> This should remain the default as well.

OK.

> 
> Thomas
> 


-- 
Damien Le Moal
Western Digital Research