[PATCH] board: rockchip: Add Hardkernel ODROID-M1S
Jonas Karlman
jonas at kwiboo.se
Thu Jan 25 12:50:56 CET 2024
Hi Dongjin,
On 2024-01-25 08:02, Dongjin Kim wrote:
> Hardkernel ODROID-M1S is a single board computer with a RK3566 SoC,
> a slightly modified version of the RK3566 SoC.
>
> Features tested on a ODROID-M1S 8GB v1.0 2023-08-10:
> - SD-card boot
> - eMMC boot
> - PCIe/NVMe
This board does not yet have a device tree in the linux maintainer tree.
Please first submit the device tree to linux before adding the board to
U-Boot.
The generic-rk3568_defconfig can be used to boot this board from SD-card
and eMMC until a fully reviewed device tree can be picked from linux.
>
> Signed-off-by: Dongjin Kim <tobetter at gmail.com>
> ---
> arch/arm/dts/rk3566-odroid-m1s-u-boot.dtsi | 28 +
> arch/arm/dts/rk3566-odroid-m1s.dts | 665 +++++++++++++++++++++
> arch/arm/mach-rockchip/rk3568/Kconfig | 6 +
> board/hardkernel/odroid_m1s/Kconfig | 15 +
> board/hardkernel/odroid_m1s/MAINTAINERS | 9 +
> board/hardkernel/odroid_m1s/Makefile | 7 +
> board/hardkernel/odroid_m1s/board.c | 80 +++
> configs/odroid-m1s-rk3566_defconfig | 109 ++++
> doc/board/rockchip/rockchip.rst | 1 +
> include/configs/odroid_m1s.h | 12 +
> 10 files changed, 932 insertions(+)
> create mode 100644 arch/arm/dts/rk3566-odroid-m1s-u-boot.dtsi
> create mode 100644 arch/arm/dts/rk3566-odroid-m1s.dts
> create mode 100644 board/hardkernel/odroid_m1s/Kconfig
> create mode 100644 board/hardkernel/odroid_m1s/MAINTAINERS
> create mode 100644 board/hardkernel/odroid_m1s/Makefile
> create mode 100644 board/hardkernel/odroid_m1s/board.c
> create mode 100644 configs/odroid-m1s-rk3566_defconfig
> create mode 100644 include/configs/odroid_m1s.h
>
> diff --git a/arch/arm/dts/rk3566-odroid-m1s-u-boot.dtsi b/arch/arm/dts/rk3566-odroid-m1s-u-boot.dtsi
> new file mode 100644
> index 00000000000..33a1d142b8c
> --- /dev/null
> +++ b/arch/arm/dts/rk3566-odroid-m1s-u-boot.dtsi
> @@ -0,0 +1,28 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +
> +#include "rk356x-u-boot.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/leds/common.h>
These includes is not in use in this file, please remove.
> +
> +/ {
> + chosen {
> + stdout-path = &uart2;
> + u-boot,spl-boot-order = &sdmmc0, &sdhci;
Is there a reason why this differ from other rk356x boards and the
default from rk356x-u-boot.dtsi cannot be used?
> + };
> +};
> +
> +&sdhci {
> + cap-mmc-highspeed;
> + mmc-ddr-1_8v;
This mode cause eMMC write issues, please drop it, I have an incoming
series that drop this prop for supported rk35xx boards.
> + mmc-hs200-1_8v;
> + mmc-hs400-1_8v;
> + mmc-hs400-enhanced-strobe;
Based on my testing HS400 modes is only supported by RK3568 not the
RK3566 variant.
> + pinctrl-0 = <&emmc_bus8 &emmc_clk &emmc_cmd &emmc_datastrobe>;
> +};
Because this device tree is not yet in linux, hopefully correct props
can be applied to device tree and u-boot do not need any overrides.
> +
> +&uart2 {
> + bootph-all;
> + clock-frequency = <24000000>;
> + status = "okay";
> +};
> diff --git a/arch/arm/dts/rk3566-odroid-m1s.dts b/arch/arm/dts/rk3566-odroid-m1s.dts
> new file mode 100644
> index 00000000000..73e29d80c71
> --- /dev/null
> +++ b/arch/arm/dts/rk3566-odroid-m1s.dts
[snip]
> diff --git a/arch/arm/mach-rockchip/rk3568/Kconfig b/arch/arm/mach-rockchip/rk3568/Kconfig
> index baa51349f4b..2f03b983ee5 100644
> --- a/arch/arm/mach-rockchip/rk3568/Kconfig
> +++ b/arch/arm/mach-rockchip/rk3568/Kconfig
> @@ -22,6 +22,11 @@ config TARGET_ODROID_M1_RK3568
> help
> Hardkernel ODROID-M1 single board computer with a RK3568B2 SoC.
>
> +config TARGET_ODROID_M1S_RK3566
> + bool "ODROID-M1S"
> + help
> + Hardkernel ODROID-M1S single board computer with a RK3566 SoC.
> +
> config TARGET_QUARTZ64_RK3566
> bool "Pine64 Quartz64"
> help
> @@ -44,6 +49,7 @@ config SYS_MALLOC_F_LEN
> source "board/rockchip/evb_rk3568/Kconfig"
> source "board/anbernic/rgxx3_rk3566/Kconfig"
> source "board/hardkernel/odroid_m1/Kconfig"
> +source "board/hardkernel/odroid_m1s/Kconfig"
> source "board/pine64/quartz64_rk3566/Kconfig"
>
> endif
> diff --git a/board/hardkernel/odroid_m1s/Kconfig b/board/hardkernel/odroid_m1s/Kconfig
> new file mode 100644
> index 00000000000..0acea61dac4
> --- /dev/null
> +++ b/board/hardkernel/odroid_m1s/Kconfig
> @@ -0,0 +1,15 @@
> +if TARGET_ODROID_M1S_RK3566
> +
> +config SYS_BOARD
> + default "odroid_m1s"
> +
> +config SYS_VENDOR
> + default "hardkernel"
> +
> +config SYS_CONFIG_NAME
> + default "odroid_m1s"
> +
> +config BOARD_SPECIFIC_OPTIONS # dummy
> + def_bool y
> +
> +endif
> diff --git a/board/hardkernel/odroid_m1s/MAINTAINERS b/board/hardkernel/odroid_m1s/MAINTAINERS
> new file mode 100644
> index 00000000000..20fc277ccc6
> --- /dev/null
> +++ b/board/hardkernel/odroid_m1s/MAINTAINERS
> @@ -0,0 +1,9 @@
> +ODROID-M1S
> +M: Dongjin Kim <tobetter at gmail.com>
I have this board so feel free to add me as a reviewer :-)
R: Jonas Karlman <jonas at kwiboo.se>
> +S: Maintained
> +F: arch/arm/dts/rk3566-odroid-m1s-u-boot.dtsi
> +F: arch/arm/dts/rk3566-odroid-m1s.dts
> +F: board/hardkernel/odroid_m1s/
> +F: board/hardkernel/odroid_m1s/board.c
> +F: configs/odroid-m1s-rk3566_defconfig
> +F: include/configs/odroid_m1s.h
> diff --git a/board/hardkernel/odroid_m1s/Makefile b/board/hardkernel/odroid_m1s/Makefile
> new file mode 100644
> index 00000000000..6ca49c49b71
> --- /dev/null
> +++ b/board/hardkernel/odroid_m1s/Makefile
> @@ -0,0 +1,7 @@
> +#
> +# Copyright (c) 2024 Hardkernel Co,. Ltd
> +#
> +# SPDX-License-Identifier: GPL-2.0+
> +#
> +
> +obj-y += board.o
> diff --git a/board/hardkernel/odroid_m1s/board.c b/board/hardkernel/odroid_m1s/board.c
> new file mode 100644
> index 00000000000..de7b160a51a
> --- /dev/null
> +++ b/board/hardkernel/odroid_m1s/board.c
> @@ -0,0 +1,80 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * (C) Copyright 2024 Hardkernel Co., Ltd
> + */
> +
> +#include <asm/unaligned.h>
> +#include <mmc.h>
> +#include <net.h>
> +#include <asm/arch-rockchip/misc.h>
> +
> +#ifdef CONFIG_MISC_INIT_R
> +
> +/* Read first block 512 bytes from the first BOOT partition of eMMC
> + * that stores device identifcation sting in UUID type 1, this string
> + * is written in factory to give device seranl number and MAC address.
> + */
> +static int odroid_setup_macaddr(void)
> +{
> + struct mmc *mmc;
> + struct blk_desc *desc;
> + unsigned long mac_addr;
> + unsigned long count;
> + int ret;
> + u8 buf[512];
> +
> + mmc = find_mmc_device(0);
> + if (!mmc)
> + return -ENODEV;
> +
> + desc = mmc_get_blk_desc(mmc);
> +
> + // Switch to the first BOOT partition
> + ret = blk_select_hwpart_devnum(UCLASS_MMC, 0, 1);
> + if (ret)
> + return -EIO;
> +
> + count = blk_dread(desc, 0, 1, (void *)buf);
> +
> + // Switch back to USER partition
> + ret = blk_dselect_hwpart(desc, 0);
> + if (ret || count != 1)
> + return -EIO;
> +
> + *(char *)(buf + 36) = 0;
> +
> + // Serial number
> + env_set("serial#", (char *)buf);
> +
> + // MAC address
> + mac_addr = cpu_to_be64(simple_strtoul((char *)buf + 24, NULL, 16)) >> 16;
> +
> + eth_env_set_enetaddr("ethaddr", (unsigned char *)&mac_addr);
> + eth_env_set_enetaddr("eth1addr", (unsigned char *)&mac_addr);
> +
> + return 0;
> +}
> +
> +int misc_init_r(void)
> +{
> + const u32 cpuid_offset = CFG_CPUID_OFFSET;
> + const u32 cpuid_length = 0x10;
> + u8 cpuid[cpuid_length];
> + int ret;
> +
> + ret = odroid_setup_macaddr();
> + if (ret) {
> + ret = rockchip_setup_macaddr();
> + if (ret)
> + return ret;
> + }
> +
> + ret = rockchip_cpuid_from_efuse(cpuid_offset, cpuid_length, cpuid);
> + if (ret)
> + return ret;
> +
> + ret = rockchip_cpuid_set(cpuid, cpuid_length);
> +
> + return ret;
> +}
> +#endif
I get build errors without "#include <common.h>":
In file included from include/linux/byteorder/little_endian.h:14,
from ./arch/arm/include/asm/byteorder.h:29,
from include/asm-generic/unaligned.h:5,
from ./arch/arm/include/asm/unaligned.h:2,
from board/hardkernel/odroid_m1s/board.c:6:
board/hardkernel/odroid_m1s/board.c: In function ‘odroid_setup_macaddr’:
CC drivers/adc/adc-uclass.o
board/hardkernel/odroid_m1s/board.c:50:32: warning: implicit declaration of function ‘simple_strtoul’ [-Wimplicit-function-declaration]
50 | mac_addr = cpu_to_be64(simple_strtoul((char *)buf + 24, NULL, 16)) >> 16;
| ^~~~~~~~~~~~~~
include/linux/byteorder/swab.h:93:31: note: in definition of macro ‘__swab64’
93 | (__builtin_constant_p((__u64)(x)) ? \
| ^
include/linux/byteorder/generic.h:92:21: note: in expansion of macro ‘__cpu_to_be64’
92 | #define cpu_to_be64 __cpu_to_be64
| ^~~~~~~~~~~~~
board/hardkernel/odroid_m1s/board.c:50:20: note: in expansion of macro ‘cpu_to_be64’
50 | mac_addr = cpu_to_be64(simple_strtoul((char *)buf + 24, NULL, 16)) >> 16;
| ^~~~~~~~~~~
board/hardkernel/odroid_m1s/board.c: In function ‘misc_init_r’:
board/hardkernel/odroid_m1s/board.c:60:34: error: ‘CFG_CPUID_OFFSET’ undeclared (first use in this function)
60 | const u32 cpuid_offset = CFG_CPUID_OFFSET;
| ^~~~~~~~~~~~~~~~
board/hardkernel/odroid_m1s/board.c:60:34: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [scripts/Makefile.build:257: board/hardkernel/odroid_m1s/board.o] Error 1
make[1]: *** [Makefile:1859: board/hardkernel/odroid_m1s] Error 2
> diff --git a/configs/odroid-m1s-rk3566_defconfig b/configs/odroid-m1s-rk3566_defconfig
> new file mode 100644
> index 00000000000..d70a10dc080
> --- /dev/null
> +++ b/configs/odroid-m1s-rk3566_defconfig
> @@ -0,0 +1,109 @@
> +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_SF_DEFAULT_SPEED=24000000
> +CONFIG_SF_DEFAULT_MODE=0x1000
> +CONFIG_DEFAULT_DEVICE_TREE="rk3566-odroid-m1s"
> +CONFIG_ROCKCHIP_RK3568=y
> +CONFIG_SPL_ROCKCHIP_COMMON_BOARD=y
> +CONFIG_ROCKCHIP_SPI_IMAGE=y
> +CONFIG_SPL_SERIAL=y
> +CONFIG_SPL_STACK_R_ADDR=0x600000
> +CONFIG_TARGET_ODROID_M1S_RK3566=y
> +CONFIG_SPL_STACK=0x400000
> +CONFIG_DEBUG_UART_BASE=0xFE660000
> +CONFIG_DEBUG_UART_CLOCK=24000000
> +CONFIG_SPL_SPI_FLASH_SUPPORT=y
> +CONFIG_SPL_SPI=y
> +CONFIG_SYS_LOAD_ADDR=0xc00800
> +CONFIG_PCI=y
> +CONFIG_DEBUG_UART=y
> +CONFIG_AHCI=y
> +CONFIG_FIT=y
> +CONFIG_FIT_VERBOSE=y
> +CONFIG_SPL_FIT_SIGNATURE=y
> +CONFIG_SPL_LOAD_FIT=y
> +CONFIG_LEGACY_IMAGE_FORMAT=y
> +CONFIG_DEFAULT_FDT_FILE="rockchip/rk3566-odroid-m1s.dtb"
> +# CONFIG_DISPLAY_CPUINFO is not set
> +CONFIG_DISPLAY_BOARDINFO_LATE=y
> +CONFIG_SPL_MAX_SIZE=0x40000
> +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_SPI_LOAD=y
> +CONFIG_SYS_SPI_U_BOOT_OFFS=0x100000
> +CONFIG_SPL_ATF=y
> +CONFIG_CMD_GPIO=y
> +CONFIG_CMD_GPT=y
> +CONFIG_CMD_I2C=y
> +CONFIG_CMD_MMC=y
> +CONFIG_CMD_PCI=y
> +CONFIG_CMD_USB=y
> +# CONFIG_CMD_SETEXPR is not set
> +CONFIG_CMD_INI=y
> +CONFIG_CMD_PMIC=y
> +CONFIG_CMD_REGULATOR=y
> +CONFIG_CMD_CRAMFS=y
> +# CONFIG_SPL_DOS_PARTITION is not set
> +CONFIG_SPL_OF_CONTROL=y
> +CONFIG_OF_LIVE=y
> +CONFIG_OF_SPL_REMOVE_PROPS="clock-names interrupt-parent assigned-clocks assigned-clock-rates assigned-clock-parents"
> +CONFIG_SPL_DM_SEQ_ALIAS=y
> +CONFIG_SPL_REGMAP=y
> +CONFIG_SPL_SYSCON=y
> +CONFIG_AHCI_PCI=y
> +CONFIG_DWC_AHCI=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_SF_DEFAULT_BUS=4
> +CONFIG_PHY_REALTEK=y
> +CONFIG_DWC_ETH_QOS=y
> +CONFIG_DWC_ETH_QOS_ROCKCHIP=y
> +CONFIG_NVME_PCI=y
> +CONFIG_PCIE_DW_ROCKCHIP=y
> +CONFIG_PHY_ROCKCHIP_INNO_USB2=y
> +CONFIG_PHY_ROCKCHIP_NANENG_COMBOPHY=y
> +CONFIG_PHY_ROCKCHIP_PCIE=y
> +CONFIG_SPL_PINCTRL=y
> +CONFIG_DM_PMIC=y
> +CONFIG_PMIC_RK8XX=y
> +CONFIG_REGULATOR_RK8XX=y
> +CONFIG_PWM_ROCKCHIP=y
> +CONFIG_SPL_RAM=y
> +CONFIG_SCSI=y
> +CONFIG_DM_SCSI=y
DM_SCSI have been migrated to SCSI and can be dropped.
> +CONFIG_BAUDRATE=1500000
> +CONFIG_DEBUG_UART_SHIFT=2
> +CONFIG_SYS_NS16550_MEM32=y
> +CONFIG_SYSRESET=y
> +CONFIG_USB=y
> +CONFIG_USB_XHCI_HCD=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_USB_EHCI_GENERIC=y
> +CONFIG_USB_OHCI_HCD=y
> +CONFIG_USB_OHCI_GENERIC=y
> +CONFIG_USB_DWC3=y
> +CONFIG_USB_DWC3_GENERIC=y
> +CONFIG_FS_CRAMFS=y
> +CONFIG_ERRNO_STR=y
> +CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR=0x0800
This does not correspont to SPL_PAD_TO, so the generated
u-boot-rockchip.bin have FIT at wrong offset and I get the following
error on my board:
U-Boot SPL 2024.01 (Jan 25 2024 - 11:43:22 +0000)
Trying to boot from MMC2
mmc_load_image_raw_sector: mmc block read error
Trying to boot from MMC1
mmc_load_image_raw_sector: mmc block read error
SPL: failed to boot from all boot devices
### ERROR ### Please RESET the board ###
Please drop this and accept the default 0x4000 value.
Regards,
Jonas
> diff --git a/doc/board/rockchip/rockchip.rst b/doc/board/rockchip/rockchip.rst
> index 18d0b6f0891..3ce9966fc13 100644
> --- a/doc/board/rockchip/rockchip.rst
> +++ b/doc/board/rockchip/rockchip.rst
> @@ -93,6 +93,7 @@ List of mainline supported Rockchip boards:
>
> * rk3566
> - Anbernic RGxx3 (anbernic-rgxx3-rk3566)
> + - Hardkernel ODROID-M1S (odroid-m1s-rk3566)
> - Pine64 Quartz64-A Board (quartz64-a-rk3566)
> - Pine64 Quartz64-B Board (quartz64-b-rk3566)
> - Pine64 SOQuartz on Blade (soquartz-blade-rk3566)
> diff --git a/include/configs/odroid_m1s.h b/include/configs/odroid_m1s.h
> new file mode 100644
> index 00000000000..b2718613886
> --- /dev/null
> +++ b/include/configs/odroid_m1s.h
> @@ -0,0 +1,12 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +
> +#ifndef __ODROID_M1S_H
> +#define __ODROID_M1S_H
> +
> +#include <configs/rk3568_common.h>
> +
> +#define ROCKCHIP_DEVICE_SETTINGS \
> + "stdout=serial\0" \
> + "stderr=serial\0"
> +
> +#endif
More information about the U-Boot
mailing list