Hi Bin,
On Wed, May 8, 2019 at 7:20 PM Bin Meng bmeng.cn@gmail.com wrote:
Hi Padmarao,
On Thu, May 2, 2019 at 2:28 PM Padmarao Begari padmarao.begari@microchip.com wrote:
This patch adds Microchip MPFS Icicle board support. For now, NS16550 serial driver is only enabled. The Microchip MPFS Icicle defconfig by default builds U-Boot for M-Mode with SMP support.
Signed-off-by: Padmarao Begari padmarao.begari@microchip.com
Changes in v2
Please move the changelog below ---, otherwise it will show up in the commit message.
Ok
Also please add a space after "riscv:" in the commit title.
Ok
- Fix some typos
- Rename target board to TARGET_MICROCHIP_ICICLE
- select CONFIG_BOARD_EARLY_INIT_F in BOARD_SPECIFIC_OPTIONS
- Remove #ifdef CONFIG_BOARD_EARLY_INIT_F
arch/riscv/Kconfig | 4 ++ board/microchip/mpfs_icicle/Kconfig | 21 +++++++++++ board/microchip/mpfs_icicle/MAINTAINERS | 7 ++++ board/microchip/mpfs_icicle/Makefile | 7 ++++ board/microchip/mpfs_icicle/mpfs_icicle.c | 30 +++++++++++++++ configs/microchip_mpfs_icicle_defconfig | 15 ++++++++ include/configs/microchip_mpfs_icicle.h | 63
+++++++++++++++++++++++++++++++
7 files changed, 147 insertions(+) create mode 100644 board/microchip/mpfs_icicle/Kconfig create mode 100644 board/microchip/mpfs_icicle/MAINTAINERS create mode 100644 board/microchip/mpfs_icicle/Makefile create mode 100644 board/microchip/mpfs_icicle/mpfs_icicle.c create mode 100644 configs/microchip_mpfs_icicle_defconfig create mode 100644 include/configs/microchip_mpfs_icicle.h
diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig index ae8ff7b..d83b260 100644 --- a/arch/riscv/Kconfig +++ b/arch/riscv/Kconfig @@ -11,6 +11,9 @@ choice config TARGET_AX25_AE350 bool "Support ax25-ae350"
+config TARGET_MICROCHIP_ICICLE
bool "Support Microchip PolarFire-SoC Icicle Board"config TARGET_QEMU_VIRT bool "Support QEMU Virt Board"
@@ -22,6 +25,7 @@ endchoice # board-specific options below source "board/AndesTech/ax25-ae350/Kconfig" source "board/emulation/qemu-riscv/Kconfig" +source "board/microchip/mpfs_icicle/Kconfig" source "board/sifive/fu540/Kconfig"
# platform-specific options below diff --git a/board/microchip/mpfs_icicle/Kconfig
b/board/microchip/mpfs_icicle/Kconfig
new file mode 100644 index 0000000..9cfa83c --- /dev/null +++ b/board/microchip/mpfs_icicle/Kconfig @@ -0,0 +1,21 @@ +if TARGET_MICROCHIP_ICICLE
+config SYS_BOARD
default "mpfs_icicle"+config SYS_VENDOR
default "microchip"+config SYS_CPU
default "generic"+config SYS_CONFIG_NAME
default "microchip_mpfs_icicle"+config BOARD_SPECIFIC_OPTIONS # dummy
def_bool yselect GENERIC_RISCVimply SMPimply BOARD_EARLY_INIT_FI think this should be "select BOARD_EARLY_INIT_F"
Ok, I will choose "select BOARD_EARLY_INIT_F"
+endif diff --git a/board/microchip/mpfs_icicle/MAINTAINERS
b/board/microchip/mpfs_icicle/MAINTAINERS
new file mode 100644 index 0000000..9987efe --- /dev/null +++ b/board/microchip/mpfs_icicle/MAINTAINERS @@ -0,0 +1,7 @@ +Microchip MPFS icicle +M: Padmarao Begari padmarao.begari@microchip.com +M: Cyril Jean cyril.jean@microchip.com +S: Maintained +F: board/microchip/mpfs-icicle/
should be: mpfs_icicle
Ok
+F: include/configs/microchip-mpfs-icicle.h
should be: microchip_mpfs_icicle.h
Ok
+F: configs/microchip-mpfs-icicle_defconfig
should be: microchip_mpfs_icicle_defconfig
Ok
diff --git a/board/microchip/mpfs_icicle/Makefile
b/board/microchip/mpfs_icicle/Makefile
new file mode 100644 index 0000000..72b0410 --- /dev/null +++ b/board/microchip/mpfs_icicle/Makefile @@ -0,0 +1,7 @@ +# SPDX-License-Identifier: GPL-2.0+ +# +# Copyright (C) 2019 Microchip Technology Inc. +# Padmarao Begari padmarao.begari@microchip.com +#
+obj-y += mpfs_icicle.o diff --git a/board/microchip/mpfs_icicle/mpfs_icicle.c
b/board/microchip/mpfs_icicle/mpfs_icicle.c
new file mode 100644 index 0000000..0ef2431 --- /dev/null +++ b/board/microchip/mpfs_icicle/mpfs_icicle.c @@ -0,0 +1,30 @@ +// SPDX-License-Identifier: GPL-2.0+ +/*
- Copyright (C) 2019 Microchip Technology Inc.
- Padmarao Begari padmarao.begari@microchip.com
- */
+#include <common.h> +#include <dm.h> +#include <asm/io.h>
+#define MPFS_SYSREG_SOFT_RESET ((unsigned int *)0x20002088)
+int board_init(void) +{
/* For now nothing to do here. */return 0;+}
+int board_early_init_f(void) +{
unsigned int val;/* Reset uart peripheral */val = readl(MPFS_SYSREG_SOFT_RESET);val = (val & ~(1u << 5u));writel(val, MPFS_SYSREG_SOFT_RESET);return 0;+} diff --git a/configs/microchip_mpfs_icicle_defconfig
b/configs/microchip_mpfs_icicle_defconfig
new file mode 100644 index 0000000..7073196 --- /dev/null +++ b/configs/microchip_mpfs_icicle_defconfig @@ -0,0 +1,15 @@ +CONFIG_RISCV=y +CONFIG_SYS_TEXT_BASE=0x80000000 +CONFIG_ARCH_RV64I=y +CONFIG_NR_CPUS=5 +CONFIG_TARGET_MICROCHIP_ICICLE=y +CONFIG_BOOTDELAY=3 +CONFIG_DISTRO_DEFAULTS=y +CONFIG_SYS_PROMPT="RISC-V # " +CONFIG_FIT=y +CONFIG_DM=y
This is not necessary as it is already in arch/Kconfig
Ok
+CONFIG_BAUDRATE=57600 +CONFIG_DM_SERIAL=y
This is not necessary too.
Ok
+CONFIG_SYS_NS16550=y +CONFIG_NR_DRAM_BANKS=1 +CONFIG_OF_PRIOR_STAGE=y
So to avoid these unnecessary stuff in defconfig, you can do:
ok
make microchip_mpfs_icicle_defconfig make savedefconfig
then copy the defconfig file to configs/microchip_mpfs_icicle_defconfig
ok
Thanks for your Review
Regards Padmarao
diff --git a/include/configs/microchip_mpfs_icicle.h
b/include/configs/microchip_mpfs_icicle.h
new file mode 100644 index 0000000..82c7fbb --- /dev/null +++ b/include/configs/microchip_mpfs_icicle.h @@ -0,0 +1,63 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ +/*
- Copyright (C) 2019 Microchip Technology Inc.
- Padmarao Begari padmarao.begari@microchip.com
- */
+#ifndef __CONFIG_H +#define __CONFIG_H
+/*
- CPU and Board Configuration Options
- */
+#define CONFIG_BOOTP_SEND_HOSTNAME
+/*
- Miscellaneous configurable options
- */
+#define CONFIG_SYS_CBSIZE 1024 /* Console I/O Buffer Size */
+/*
- Print Buffer Size
- */
+#define CONFIG_SYS_PBSIZE \
(CONFIG_SYS_CBSIZE + sizeof(CONFIG_SYS_PROMPT) + 16)+/*
- max number of command args
- */
+#define CONFIG_SYS_MAXARGS 16
+/*
- Boot Argument Buffer Size
- */
+#define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE
+/*
- Size of malloc() pool
- 512kB is suggested, (CONFIG_ENV_SIZE + 128 * 1024) was not enough
- */
+#define CONFIG_SYS_MALLOC_LEN (512 << 10)
+/*
- Physical Memory Map
- */
+#define PHYS_SDRAM_0 0x80000000 /* SDRAM Bank #1 */ +#define PHYS_SDRAM_0_SIZE 0x40000000 /* 1 GB */ +#define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_0
+/* Init Stack Pointer */ +#define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_SDRAM_BASE +
0x200000)
+#define CONFIG_SYS_LOAD_ADDR 0x80000000 /* SDRAM */
+/*
- memtest works on DRAM
- */
+#define CONFIG_SYS_MEMTEST_START PHYS_SDRAM_0 +#define CONFIG_SYS_MEMTEST_END (PHYS_SDRAM_0 +
PHYS_SDRAM_0_SIZE)
+/* When we use RAM as ENV */ +#define CONFIG_ENV_SIZE 0x2000
+#endif /* __CONFIG_H */
Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot