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.hris...@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-jo...@kwiboo.se/

Regards,
Jonas

> 
> 
> 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.hris...@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 */

Reply via email to