Re: [Buildroot] [PATCH 1/1] configs/orangepi_5_plus: new board

From: Julien Olivain <ju.o@free.fr>
To: Sergey Matyukevich <geomatsi@gmail.com>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1] configs/orangepi_5_plus: new board
Date: Sun, 26 Jan 2025 16:41:39 +0100	[thread overview]
Message-ID: <d0942ad2ff729c48c1b1c93bb71c8c5e@free.fr> (raw)
In-Reply-To: <Z5YeKmQHSNJE1J7Z@curiosity>

Hi Sergey,

On 26/01/2025 12:36, Sergey Matyukevich wrote:
> On Sat, Jan 25, 2025 at 10:10:59PM +0100, Julien Olivain wrote:
>> On 06/01/2025 20:07, Sergey Matyukevich wrote:
>> > This patch adds basic support for the new OrangePi 5 Plus board:
>> > - http://www.orangepi.org/html/hardWare/computerAndMicrocontrollers/details/Orange-Pi-5-plus-32GB.html
>> >
>> > Brief summary of the board features:
>> > - RK3588 Rockchip SoC
>> >   - Quad-Core ARM Cortex-A76 Processor 2.4GHz
>> >   - Quad-Core ARM Cortex-A55 Processor 1.8GHz
>> >   - Mali G610 GPU
>> >   - NPU 6 Tops INT4/INT8/INT16/FP16
>> > - LPDDR4x: 8GB/16GB/32GB
>> > - 2x USB 2.0
>> > - 2x USB 3.0
>> > - 2x HDMI out
>> > - PCIe M.2 M-key socket (e.g. for NVMe)
>> > - PCIe M.2 E-key socket (e.g. for WiFi/BT)
>> > - 40-pin expansion interface (GPIO / UART / I2C / SPI / PWM / CAN / I2S)
>> > - and much much more ...
>> >
>> > BSP includes the following components:
>> > - mainline ATF v2.12
>> > - mainline U-Boot v2024.10
>> > - mainline Linux kernel v6.12.x
>> >
>> > Unlike all the other boards from board/orangepi directory, this board
>> > is not based on an Allwinner SoC. It is based on Rockchip RK3588 SoC.
>> > So custom scripts and configs are used instead of common options from
>> > the board/orangepi/common directory. Suggested scripts and configs are
>> > based on those of Radxa Rock5B board.
>> >
>> > Signed-off-by: Sergey Matyukevich <geomatsi@gmail.com>
>> 
>> Applied to master, thanks.
> 
> Thanks !
> 
> Unfortunately, your minor cleanup 'switch to human readable 
> genimage.cfg
> partition type uuid' breaks the build:
> 
> : >>>   Executing post-image script support/scripts/genimage.sh
> : ...
> : ERROR: hdimage(sdcard.img): part uboot-env has invalid type shortcut: 
> u-boot-env
> 
> Latest available genimage release does not yet provide the GUID 
> shortcut
> for 'u-boot-env'. So I sent a fixup to keep using the GUID value 
> directly:
> 
> - 
> https://lore.kernel.org/buildroot/20250126113141.3258643-1-geomatsi@gmail.com/T/#u

Sorry for my non-improvement ;)  and thanks for the fix!

> Regards,
> Sergey

Best regards,

Julien.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot