[RFC][PATCH] board: rockchip: add Radxa ROCK5B Rk3588 board

Eugen Hristev eugen.hristev at collabora.com
Mon Feb 6 14:55:29 CET 2023


On 2/6/23 14:14, Jonas Karlman wrote:
> Hi Eugen,
> On 2023-02-06 12:59, Eugen Hristev wrote:
>> ROCK 5B is a Rockchip RK3588 based SBC (Single Board Computer) by Radxa.
>>
>> There are tree variants depending on the DRAM size : 4G, 8G and 16G.
>>
>> Specification:
>>
>>      Rockchip Rk3588 SoC
>>      4x ARM Cortex-A76, 4x ARM Cortex-A55
>>      4/8/16GB memory LPDDR4x
>>      Mali G610MC4 GPU
>>      MIPI CSI 2 multiple lanes connector
>>      eMMC module connector
>>      uSD slot (up to 128GB)
>>      2x USB 2.0, 2x USB 3.0
>>      2x HDMI output, 1x HDMI input
>>      Ethernet port
>>      40-pin IO header including UART, SPI, I2C and 5V DC power in
>>      USB PD over USB Type-C
>>      Size: 85mm x 54mm
>>
>> Signed-off-by: Eugen Hristev <eugen.hristev at collabora.com>
>> ---
>>
>> Hi,
>>
>> This patch is based on top of Jagan's series
>> https://lists.denx.de/pipermail/u-boot/2023-January/506156.html
>>
>> And fixes from Jonas Karlman which I cherrypicked
>> The DT is identical with current linux-next.
>>
>> One thing which is not working as expected is the DRAM size detection,
>> the rock5b has 16 GiB but Uboot only reports 4 GiB.
>>
>> The DRAM driver used is the one from Jagan's series.
>>
>> Jagan, maybe you can tell me how is this driver working on your Edgeble board
>> and how the detection works there for you ?
> 
> With the SDRAM series at [1] it should hopefylly detect the correct ram size.
> I will send a v2 rebased on u-boot master later today, a CONFIG_ to CFG_ change.
> 
> [1] https://patchwork.ozlabs.org/project/uboot/cover/20230116161459.1307591-1-jonas@kwiboo.se/
> 
> Regards,
> Jonas
> 

Hi Jonas,

Great, thanks !
With your patches, the whole 16 GiB is detected correctly.
So even if my patch is pretty much independent, it still depends on your 
patch to detect the right amount of DRAM.

I will add a Tested-by when you send a v2.

Eugen

>>
>>
>> Thanks,
>>
>> Eugen
>>
>>   arch/arm/dts/rk3588-rock-5b-u-boot.dtsi   | 22 ++++++++
>>   arch/arm/dts/rk3588-rock-5b.dts           | 44 +++++++++++++++
>>   arch/arm/mach-rockchip/rk3588/Kconfig     | 26 +++++++++
>>   board/radxa/rock5b-rk3588/Kconfig         | 15 +++++
>>   board/radxa/rock5b-rk3588/MAINTAINERS     |  6 ++
>>   board/radxa/rock5b-rk3588/Makefile        |  6 ++
>>   board/radxa/rock5b-rk3588/rock5b-rk3588.c |  4 ++
>>   configs/rock5b-rk3588_defconfig           | 69 +++++++++++++++++++++++
>>   doc/board/rockchip/rockchip.rst           |  1 +
>>   include/configs/rock5b-rk3588.h           | 15 +++++
>>   10 files changed, 208 insertions(+)
>>   create mode 100644 arch/arm/dts/rk3588-rock-5b-u-boot.dtsi
>>   create mode 100644 arch/arm/dts/rk3588-rock-5b.dts
>>   create mode 100644 board/radxa/rock5b-rk3588/Kconfig
>>   create mode 100644 board/radxa/rock5b-rk3588/MAINTAINERS
>>   create mode 100644 board/radxa/rock5b-rk3588/Makefile
>>   create mode 100644 board/radxa/rock5b-rk3588/rock5b-rk3588.c
>>   create mode 100644 configs/rock5b-rk3588_defconfig
>>   create mode 100644 include/configs/rock5b-rk3588.h
>>
>> diff --git a/arch/arm/dts/rk3588-rock-5b-u-boot.dtsi b/arch/arm/dts/rk3588-rock-5b-u-boot.dtsi
>> new file mode 100644
>> index 000000000000..2386edf90deb
>> --- /dev/null
>> +++ b/arch/arm/dts/rk3588-rock-5b-u-boot.dtsi
>> @@ -0,0 +1,22 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (c) 2023 Collabora Ltd.
>> + */
>> +
>> +#include "rk3588-u-boot.dtsi"
>> +
>> +/ {
>> +	aliases {
>> +		mmc0 = &sdmmc;
>> +	};
>> +
>> +	chosen {
>> +		u-boot,spl-boot-order = &sdmmc;
>> +	};
>> +};
>> +
>> +&sdmmc {
>> +	bus-width = <4>;
>> +	u-boot,dm-spl;
>> +	status = "okay";
>> +};
>> diff --git a/arch/arm/dts/rk3588-rock-5b.dts b/arch/arm/dts/rk3588-rock-5b.dts
>> new file mode 100644
>> index 000000000000..95805cb0adfa
>> --- /dev/null
>> +++ b/arch/arm/dts/rk3588-rock-5b.dts
>> @@ -0,0 +1,44 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +
>> +/dts-v1/;
>> +
>> +#include "rk3588.dtsi"
>> +
>> +/ {
>> +	model = "Radxa ROCK 5 Model B";
>> +	compatible = "radxa,rock-5b", "rockchip,rk3588";
>> +
>> +	aliases {
>> +		mmc0 = &sdhci;
>> +		serial2 = &uart2;
>> +	};
>> +
>> +	chosen {
>> +		stdout-path = "serial2:1500000n8";
>> +	};
>> +
>> +	vcc5v0_sys: vcc5v0-sys-regulator {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vcc5v0_sys";
>> +		regulator-always-on;
>> +		regulator-boot-on;
>> +		regulator-min-microvolt = <5000000>;
>> +		regulator-max-microvolt = <5000000>;
>> +	};
>> +};
>> +
>> +&sdhci {
>> +	bus-width = <8>;
>> +	no-sdio;
>> +	no-sd;
>> +	non-removable;
>> +	max-frequency = <200000000>;
>> +	mmc-hs400-1_8v;
>> +	mmc-hs400-enhanced-strobe;
>> +	status = "okay";
>> +};
>> +
>> +&uart2 {
>> +	pinctrl-0 = <&uart2m0_xfer>;
>> +	status = "okay";
>> +};
>> diff --git a/arch/arm/mach-rockchip/rk3588/Kconfig b/arch/arm/mach-rockchip/rk3588/Kconfig
>> index def4094e2e44..aee71ca1dab8 100644
>> --- a/arch/arm/mach-rockchip/rk3588/Kconfig
>> +++ b/arch/arm/mach-rockchip/rk3588/Kconfig
>> @@ -13,6 +13,31 @@ config TARGET_RK3588_NEU6
>>   	  IO board and Neu6a needs to mount on top of this IO board in order to
>>   	  create complete Edgeble Neural Compute Module 6(Neu6) IO platform.
>>   
>> +config TARGET_ROCK5B_RK3588
>> +	bool "Radxa ROCK5B RK3588 board"
>> +	select BOARD_LATE_INIT
>> +	help
>> +	  Radxa ROCK5B is a Rockchip RK3588 based SBC (Single Board Computer)
>> +	  by Radxa.
>> +
>> +	  There are tree variants depending on the DRAM size : 4G, 8G and 16G.
>> +
>> +	  Specification:
>> +
>> +	  Rockchip Rk3588 SoC
>> +	  4x ARM Cortex-A76, 4x ARM Cortex-A55
>> +	  4/8/16GB memory LPDDR4x
>> +	  Mali G610MC4 GPU
>> +	  MIPI CSI 2 multiple lanes connector
>> +	  eMMC module connector
>> +	  uSD slot (up to 128GB)
>> +	  2x USB 2.0, 2x USB 3.0
>> +	  2x HDMI output, 1x HDMI input
>> +	  Ethernet port
>> +	  40-pin IO header including UART, SPI, I2C and 5V DC power in
>> +	  USB PD over USB Type-C
>> +	  Size: 85mm x 54mm
>> +
>>   config ROCKCHIP_BOOT_MODE_REG
>>   	default 0xfd588080
>>   
>> @@ -26,5 +51,6 @@ config SYS_MALLOC_F_LEN
>>   	default 0x80000
>>   
>>   source board/edgeble/neural-compute-module-6/Kconfig
>> +source board/radxa/rock5b-rk3588/Kconfig
>>   
>>   endif
>> diff --git a/board/radxa/rock5b-rk3588/Kconfig b/board/radxa/rock5b-rk3588/Kconfig
>> new file mode 100644
>> index 000000000000..8f1444649afe
>> --- /dev/null
>> +++ b/board/radxa/rock5b-rk3588/Kconfig
>> @@ -0,0 +1,15 @@
>> +if TARGET_ROCK5B_RK3588
>> +
>> +config SYS_BOARD
>> +	default "rock5b-rk3588"
>> +
>> +config SYS_VENDOR
>> +	default "radxa"
>> +
>> +config SYS_CONFIG_NAME
>> +	default "rock5b-rk3588"
>> +
>> +config BOARD_SPECIFIC_OPTIONS # dummy
>> +	def_bool y
>> +
>> +endif
>> diff --git a/board/radxa/rock5b-rk3588/MAINTAINERS b/board/radxa/rock5b-rk3588/MAINTAINERS
>> new file mode 100644
>> index 000000000000..693751e583d9
>> --- /dev/null
>> +++ b/board/radxa/rock5b-rk3588/MAINTAINERS
>> @@ -0,0 +1,6 @@
>> +ROCK5B-RK3588
>> +M:	Eugen Hristev <eugen.hristev at collabora.com>
>> +S:	Maintained
>> +F:	board/radxa/rock5b-rk3588
>> +F:	include/configs/rock5b-rk3588
>> +F:	configs/rock5b-rk3588_defconfig
>> diff --git a/board/radxa/rock5b-rk3588/Makefile b/board/radxa/rock5b-rk3588/Makefile
>> new file mode 100644
>> index 000000000000..95d813596da4
>> --- /dev/null
>> +++ b/board/radxa/rock5b-rk3588/Makefile
>> @@ -0,0 +1,6 @@
>> +# SPDX-License-Identifier:     GPL-2.0+
>> +#
>> +# Copyright (c) 2022 Collabora Ltd.
>> +#
>> +
>> +obj-y += rock5b-rk3588.o
>> diff --git a/board/radxa/rock5b-rk3588/rock5b-rk3588.c b/board/radxa/rock5b-rk3588/rock5b-rk3588.c
>> new file mode 100644
>> index 000000000000..b5d74798f3b9
>> --- /dev/null
>> +++ b/board/radxa/rock5b-rk3588/rock5b-rk3588.c
>> @@ -0,0 +1,4 @@
>> +// SPDX-License-Identifier: GPL-2.0+
>> +/*
>> + * Copyright (c) 2023 Collabora Ltd.
>> + */
>> diff --git a/configs/rock5b-rk3588_defconfig b/configs/rock5b-rk3588_defconfig
>> new file mode 100644
>> index 000000000000..ac0ab7ab0f92
>> --- /dev/null
>> +++ b/configs/rock5b-rk3588_defconfig
>> @@ -0,0 +1,69 @@
>> +CONFIG_ARM=y
>> +CONFIG_SKIP_LOWLEVEL_INIT=y
>> +CONFIG_COUNTER_FREQUENCY=24000000
>> +CONFIG_ARCH_ROCKCHIP=y
>> +CONFIG_TEXT_BASE=0x00a00000
>> +CONFIG_SPL_LIBCOMMON_SUPPORT=y
>> +CONFIG_SPL_LIBGENERIC_SUPPORT=y
>> +CONFIG_NR_DRAM_BANKS=2
>> +CONFIG_HAS_CUSTOM_SYS_INIT_SP_ADDR=y
>> +CONFIG_CUSTOM_SYS_INIT_SP_ADDR=0xc00000
>> +CONFIG_DEFAULT_DEVICE_TREE="rk3588-rock-5b"
>> +CONFIG_ROCKCHIP_RK3588=y
>> +CONFIG_SPL_ROCKCHIP_BACK_TO_BROM=y
>> +CONFIG_SPL_ROCKCHIP_COMMON_BOARD=y
>> +CONFIG_SPL_MMC=y
>> +CONFIG_SPL_SERIAL=y
>> +CONFIG_SPL_STACK_R_ADDR=0x600000
>> +CONFIG_TARGET_ROCK5B_RK3588=y
>> +CONFIG_SPL_STACK=0x400000
>> +CONFIG_DEBUG_UART_BASE=0xFEB50000
>> +CONFIG_DEBUG_UART_CLOCK=24000000
>> +CONFIG_SYS_LOAD_ADDR=0xc00800
>> +CONFIG_DEBUG_UART=y
>> +CONFIG_FIT=y
>> +CONFIG_FIT_VERBOSE=y
>> +CONFIG_SPL_LOAD_FIT=y
>> +CONFIG_DEFAULT_FDT_FILE="rockchip/rk3588-rock-5b.dtb"
>> +# CONFIG_DISPLAY_CPUINFO is not set
>> +CONFIG_DISPLAY_BOARDINFO_LATE=y
>> +CONFIG_SPL_MAX_SIZE=0x20000
>> +CONFIG_SPL_PAD_TO=0x7f8000
>> +CONFIG_SPL_HAS_BSS_LINKER_SECTION=y
>> +CONFIG_SPL_BSS_START_ADDR=0x4000000
>> +CONFIG_SPL_BSS_MAX_SIZE=0x4000
>> +# CONFIG_SPL_RAW_IMAGE_SUPPORT is not set
>> +# CONFIG_SPL_SHARES_INIT_SP_ADDR is not set
>> +CONFIG_SPL_STACK_R=y
>> +CONFIG_SPL_ATF=y
>> +CONFIG_CMD_GPT=y
>> +CONFIG_CMD_MMC=y
>> +# CONFIG_CMD_SETEXPR is not set
>> +# CONFIG_SPL_DOS_PARTITION is not set
>> +CONFIG_SPL_OF_CONTROL=y
>> +CONFIG_OF_LIVE=y
>> +CONFIG_NET_RANDOM_ETHADDR=y
>> +CONFIG_SPL_REGMAP=y
>> +CONFIG_SPL_SYSCON=y
>> +CONFIG_SPL_CLK=y
>> +CONFIG_ROCKCHIP_GPIO=y
>> +CONFIG_SYS_I2C_ROCKCHIP=y
>> +CONFIG_MISC=y
>> +CONFIG_SUPPORT_EMMC_RPMB=y
>> +CONFIG_MMC_DW=y
>> +CONFIG_MMC_DW_ROCKCHIP=y
>> +CONFIG_MMC_SDHCI=y
>> +CONFIG_MMC_SDHCI_SDMA=y
>> +CONFIG_MMC_SDHCI_ROCKCHIP=y
>> +CONFIG_ETH_DESIGNWARE=y
>> +CONFIG_GMAC_ROCKCHIP=y
>> +CONFIG_REGULATOR_PWM=y
>> +CONFIG_PWM_ROCKCHIP=y
>> +CONFIG_SPL_RAM=y
>> +CONFIG_DM_RESET=y
>> +CONFIG_BAUDRATE=1500000
>> +CONFIG_DEBUG_UART_SHIFT=2
>> +CONFIG_DEBUG_UART_ANNOUNCE=y
>> +CONFIG_SYSRESET=y
>> +# CONFIG_BINMAN_FDT is not set
>> +CONFIG_ERRNO_STR=y
>> diff --git a/doc/board/rockchip/rockchip.rst b/doc/board/rockchip/rockchip.rst
>> index a140c1a8a432..10b971361dab 100644
>> --- a/doc/board/rockchip/rockchip.rst
>> +++ b/doc/board/rockchip/rockchip.rst
>> @@ -88,6 +88,7 @@ List of mainline supported Rockchip boards:
>>        - Theobroma Systems RK3399-Q7 SoM - Puma (puma_rk3399)
>>   * rk3588
>>        - Edgeble Neural Compute Module 6 SoM - Neu6a (neu6a-io-r1126)
>> +     - Radxa ROCK 5B (rock5b-rk3588)
>>   * rv1108
>>        - Rockchip Evb-rv1108 (evb-rv1108)
>>        - Elgin-R1 (elgin-rv1108)
>> diff --git a/include/configs/rock5b-rk3588.h b/include/configs/rock5b-rk3588.h
>> new file mode 100644
>> index 000000000000..4f75c800060a
>> --- /dev/null
>> +++ b/include/configs/rock5b-rk3588.h
>> @@ -0,0 +1,15 @@
>> +/* SPDX-License-Identifier: GPL-2.0+ */
>> +/*
>> + * Copyright (c) 2022 Collabora Ltd.
>> + */
>> +
>> +#ifndef __ROCK5B_RK3588_H
>> +#define __ROCK5B_RK3588_H
>> +
>> +#define ROCKCHIP_DEVICE_SETTINGS \
>> +		"stdout=serial,vidconsole\0" \
>> +		"stderr=serial,vidconsole\0"
>> +
>> +#include <configs/rk3588_common.h>
>> +
>> +#endif /* __ROCK5B_RK3588_H */
> 



More information about the U-Boot mailing list