[PATCH] Added support for Digilent Cora Z7 board

mail archive of the barebox mailing list
 help / color / mirror / Atom feed

* [PATCH] Added support for Digilent Cora Z7 board
@ 2025-05-16 17:18 Johannes Roith
  2025-05-16 20:05 ` Ahmad Fatoum
  0 siblings, 1 reply; 2+ messages in thread
From: Johannes Roith @ 2025-05-16 17:18 UTC (permalink / raw)
  To: s.hauer; +Cc: barebox, Johannes Roith

This patch adds support for the Digilent Cora Z7 board to barebox.

This patch includes a PBL initializing the DDR memory and the most important
hardware and barebox proper which is loaded from the mmc but uses network
boot as the default boot source. mmc boot is also working.

This patch only brings support for booting the PS side, loading a bitstream
on the PL side is nit supported.

Signed-off-by: Johannes Roith <johannes@gnu-linux.rocks>
---
 arch/arm/boards/Makefile                      |   1 +
 arch/arm/boards/digilent-cora/Makefile        |   4 +
 arch/arm/boards/digilent-cora/board.c         |  33 ++
 arch/arm/boards/digilent-cora/cora.zynqcfg    |   4 +
 .../env/nv/linux.bootargs.console             |   1 +
 arch/arm/boards/digilent-cora/lowlevel.c      | 296 ++++++++++++++++++
 arch/arm/configs/zynq_defconfig               |   1 +
 arch/arm/dts/Makefile                         |   1 +
 arch/arm/dts/zynq-cora-linux.dts              |  62 ++++
 arch/arm/dts/zynq-cora.dts                    |  29 ++
 arch/arm/mach-zynq/Kconfig                    |   5 +
 images/Makefile.zynq                          |   6 +
 12 files changed, 443 insertions(+)
 create mode 100644 arch/arm/boards/digilent-cora/Makefile
 create mode 100644 arch/arm/boards/digilent-cora/board.c
 create mode 100644 arch/arm/boards/digilent-cora/cora.zynqcfg
 create mode 100644 arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console
 create mode 100644 arch/arm/boards/digilent-cora/lowlevel.c
 create mode 100644 arch/arm/dts/zynq-cora-linux.dts
 create mode 100644 arch/arm/dts/zynq-cora.dts

diff --git a/arch/arm/boards/Makefile b/arch/arm/boards/Makefile
index 908497cd8b..fcf4d393a1 100644
--- a/arch/arm/boards/Makefile
+++ b/arch/arm/boards/Makefile
@@ -154,6 +154,7 @@ obj-$(CONFIG_MACH_USI_TOPKICK)			+= usi-topkick/
 obj-$(CONFIG_MACH_VERSATILEPB)			+= versatile/
 obj-$(CONFIG_MACH_VEXPRESS)			+= vexpress/
 obj-$(CONFIG_MACH_ZEDBOARD)			+= avnet-zedboard/
+obj-$(CONFIG_MACH_CORA_Z7)			+= digilent-cora/
 obj-$(CONFIG_MACH_VARISCITE_MX6)		+= variscite-mx6/
 obj-$(CONFIG_MACH_VARISCITE_SOM_MX7)		+= variscite-som-mx7/
 obj-$(CONFIG_MACH_VSCOM_BALTOS)			+= vscom-baltos/
diff --git a/arch/arm/boards/digilent-cora/Makefile b/arch/arm/boards/digilent-cora/Makefile
new file mode 100644
index 0000000000..d653aa1f4b
--- /dev/null
+++ b/arch/arm/boards/digilent-cora/Makefile
@@ -0,0 +1,4 @@
+# SPDX-License-Identifier: GPL-2.0-only
+
+obj-y += board.o
+lwl-y += lowlevel.o
diff --git a/arch/arm/boards/digilent-cora/board.c b/arch/arm/boards/digilent-cora/board.c
new file mode 100644
index 0000000000..1e70137352
--- /dev/null
+++ b/arch/arm/boards/digilent-cora/board.c
@@ -0,0 +1,33 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+// SPDX-FileCopyrightText: 2025 Johannes Roith <johannes@gnu-linux.rocks>
+
+#include <asm/armlinux.h>
+#include <bbu.h>
+#include <common.h>
+#include <environment.h>
+#include <asm/mach-types.h>
+#include <init.h>
+#include <mach/zynq/zynq7000-regs.h>
+#include <linux/sizes.h>
+#include <deep-probe.h>
+
+static int cora_probe(struct device *dev)
+{
+	barebox_set_hostname("cora");
+
+	return bbu_register_std_file_update("SD", BBU_HANDLER_FLAG_DEFAULT,
+				     "/boot/BOOT.bin", filetype_zynq_image);
+}
+
+static const struct of_device_id cora_of_match[] = {
+	{ .compatible = "xlnx,zynq-cora" },
+	{ /* sentinel */ },
+};
+BAREBOX_DEEP_PROBE_ENABLE(cora_of_match);
+
+static struct driver cora_board_driver = {
+	.name = "board-digilent-cora",
+	.probe = cora_probe,
+	.of_compatible = cora_of_match,
+};
+coredevice_platform_driver(cora_board_driver);
diff --git a/arch/arm/boards/digilent-cora/cora.zynqcfg b/arch/arm/boards/digilent-cora/cora.zynqcfg
new file mode 100644
index 0000000000..8e68bbf618
--- /dev/null
+++ b/arch/arm/boards/digilent-cora/cora.zynqcfg
@@ -0,0 +1,4 @@
+#include <zynq/zynq7000-header-regs.h>
+
+/* stop */
+wm 32 0xFFFFFFFF			0x00000000
diff --git a/arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console b/arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console
new file mode 100644
index 0000000000..903e90f870
--- /dev/null
+++ b/arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console
@@ -0,0 +1 @@
+console=ttyPS1,115200 root=/dev/mmcblk0p2 rootwait
diff --git a/arch/arm/boards/digilent-cora/lowlevel.c b/arch/arm/boards/digilent-cora/lowlevel.c
new file mode 100644
index 0000000000..adaf80a858
--- /dev/null
+++ b/arch/arm/boards/digilent-cora/lowlevel.c
@@ -0,0 +1,296 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+// SPDX-FileCopyrightText: 2025 Johannes Roith <johannes@gnu-linux.rocks>
+
+#include <common.h>
+#include <io.h>
+#include <asm/barebox-arm.h>
+#include <asm/barebox-arm-head.h>
+#include <mach/zynq/init.h>
+#include <mach/zynq/zynq7000-regs.h>
+#include <serial/cadence.h>
+
+#define DCI_DONE	(1 << 13)
+#define PLL_ARM_LOCK	(1 << 0)
+#define PLL_DDR_LOCK	(1 << 1)
+#define PLL_IO_LOCK	(1 << 2)
+#define DDR_NORMAL_OP   (1 << 0)
+
+extern char __dtb_z_zynq_cora_start[];
+
+static void cora_z7_ps7_init(void)
+{
+	/*
+	 * Read OCM mapping configuration, if only the upper 64 KByte are
+	 * mapped to the high address, it's very likely that we just got control
+	 * from the BootROM. If the mapping is changed something other than the
+	 * BootROM was running before us. Skip PS7 init to avoid cutting the
+	 * branch we are sitting on in that case.
+	 */
+	if ((readl(0xf8000910) & 0xf) != 0x8)
+		return;
+	/* Do something!!! */
+	/* open sesame */
+	writel(0x0000DF0D, ZYNQ_SLCR_UNLOCK);
+
+	/* ps7_clock_init_data */
+	writel(0x1F000200, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_CLK_CTRL);
+	writel(0x00203401, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DCI_CLK_CTRL);
+	writel(0x00001401, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_UART_CLK_CTRL);
+	writel(0x00000401, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_SDIO_CLK_CTRL);
+	writel(0x00000A03, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DBG_CLK_CTRL);
+	writel(0x00000501, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_PCAP_CLK_CTRL);
+	writel(0x00000000, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_TOPSW_CLK_CTRL);
+	writel(0x00400500, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_FPGA0_CLK_CTRL);
+	writel(0x00100700, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_FPGA1_CLK_CTRL);
+	writel(0x00101400, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_FPGA2_CLK_CTRL);
+	writel(0x00101400, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_FPGA3_CLK_CTRL);
+	/* 6:2:1 mode */
+	writel(0x00000001, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_CLK_621_TRUE);
+	writel(0x015C044D, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_APER_CLK_CTRL);
+
+	/* configure the PLLs */
+	/* ARM PLL */
+	writel(0x0001A008, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_PLL_CTRL);
+	writel(0x001772c0, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_PLL_CFG);
+	writel(0x0001A010, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_PLL_CTRL);
+	writel(0x0001A011, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_PLL_CTRL);
+	writel(0x0001A010, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_PLL_CTRL);
+
+	while (!(readl(ZYNQ_CLOCK_CTRL_BASE + ZYNQ_PLL_STATUS) & PLL_ARM_LOCK))
+		;
+	writel(0x0001A000, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_ARM_PLL_CTRL);
+
+	/* DDR PLL */
+	/* set to bypass mode */
+	writel(0x0001A018, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_PLL_CTRL);
+	/* assert reset */
+	writel(0x0001A019, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_PLL_CTRL);
+	/* set feedback divs */
+	writel(0x00015019, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_PLL_CTRL);
+	writel(0x001db2c0, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_PLL_CFG);
+	/* set ddr2xclk and ddr3xclk: 3,2 */
+	writel(0x0C200003, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_CLK_CTRL);
+	/* deassert reset */
+	writel(0x00015018, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_PLL_CTRL);
+	/* wait pll lock */
+	while (!(readl(ZYNQ_CLOCK_CTRL_BASE + ZYNQ_PLL_STATUS) & PLL_DDR_LOCK))
+		;
+	/* remove bypass mode */
+	writel(0x00015008, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_DDR_PLL_CTRL);
+
+	/* IO PLL */
+	writel(0x0001a008, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_IO_PLL_CTRL);
+	writel(0x001f42C0, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_IO_PLL_CFG);
+	writel(0x00014010, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_IO_PLL_CTRL);
+	writel(0x00014011, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_IO_PLL_CTRL);
+	writel(0x00014010, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_IO_PLL_CTRL);
+
+	while (!(readl(ZYNQ_CLOCK_CTRL_BASE + ZYNQ_PLL_STATUS) & PLL_IO_LOCK))
+		;
+	writel(0x00014000, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_IO_PLL_CTRL);
+
+	/*
+	 * INP_TYPE[1:2] = {off, vref, diff, lvcmos}
+	 * DCI_UPDATE[3], TERM_EN[4]
+	 * DCI_TYPE[5:6] = {off, drive, res, term}
+	 * IBUF_DISABLE_MODE[7] = {ibuf, ibuf_disable}
+	 * TERM_DISABLE_MODE[8] = {always, dynamic}
+	 * OUTPUT_EN[9:10] = {ibuf, res, res, obuf}
+	 * PULLUP_EN[11]
+	 */
+	writel(0x00000600, ZYNQ_DDRIOB_ADDR0);
+	writel(0x00000600, ZYNQ_DDRIOB_ADDR1);
+	writel(0x00000672, ZYNQ_DDRIOB_DATA0);
+	writel(0x00000800, ZYNQ_DDRIOB_DATA1);
+	writel(0x00000674, ZYNQ_DDRIOB_DIFF0);
+	writel(0x00000800, ZYNQ_DDRIOB_DIFF1);
+	writel(0x00000600, ZYNQ_DDRIOB_CLOCK);
+	/*
+	 * Drive_P[0:6], Drive_N[7:13]
+	 * Slew_P[14:18], Slew_N[19:23]
+	 * GTL[24:26], RTerm[27:31]
+	 */
+	writel(0x0018c61C, ZYNQ_DDRIOB_DRIVE_SLEW_ADDR);
+	writel(0x00F9861C, ZYNQ_DDRIOB_DRIVE_SLEW_DATA);
+	writel(0x00F9861C, ZYNQ_DDRIOB_DRIVE_SLEW_DIFF);
+	writel(0x00f9861C, ZYNQ_DDRIOB_DRIVE_SLEW_CLOCK);
+	/*
+	 * VREF_INT_EN[0]
+	 * VREF_SEL[1:4] = {0001=0.6V, 0100=0.75V, 1000=0.9V}
+	 * VREF_EXT_EN[5:6] = {dis/dis, dis/en, en/dis, en/en}
+	 * RES[7:8], REFIO_EN[9]
+	 */
+	/* FIXME: Xilinx sets this to internal, but Zedboard should support
+	   external VRef, too */
+	writel(0x00000220, ZYNQ_DDRIOB_DDR_CTRL);
+	/*
+	 * RESET[0], ENABLE[1]
+	 * NREF_OPT1[6:7], NREF_OPT2[8:10], NREF_OPT4[11:13]
+	 * PREF_OPT1[14:15], PREF_OPT2[17:19], UPDATE_CONTROL[20]
+	 */
+	writel(0x00000021, ZYNQ_DDRIOB_DCI_CTRL);
+	writel(0x00000020, ZYNQ_DDRIOB_DCI_CTRL);
+	writel(0x00000823, ZYNQ_DDRIOB_DCI_CTRL);
+
+	while (!(readl(ZYNQ_DDRIOB_DCI_STATUS) & DCI_DONE))
+		;
+
+	writel(0x00000000, 0xF8007000);
+
+	/* ps7_ddr_init_data */
+	writel(0x00000084, 0XF8006000);
+	writel(0x0000107f, 0XF8006004);
+	writel(0x03c0780f, 0XF8006008);
+	writel(0x02001001, 0XF800600C);
+	writel(0x00014001, 0XF8006010);
+	writel(0x0004279a, 0XF8006014);
+	writel(0x44e354d2, 0XF8006018);
+	writel(0x720238e5, 0XF800601C);
+	writel(0x270872d0, 0XF8006020);
+	writel(0x00000000, 0XF8006024);
+	writel(0x00002007, 0XF8006028);
+	writel(0x00000008, 0XF800602C);
+	writel(0x00040930, 0XF8006030);
+	writel(0x00011674, 0XF8006034);
+	writel(0x00000000, 0XF8006038);
+	writel(0x00000666, 0XF800603C);
+	writel(0xffff0000, 0XF8006040);
+	writel(0x0f555555, 0XF8006044);
+	writel(0x0003c008, 0XF8006048);
+	writel(0x77010800, 0XF8006050);
+	writel(0x00000000, 0XF8006058);
+	writel(0x00005003, 0XF800605C);
+	writel(0x0000003e, 0XF8006060);
+	writel(0x00020000, 0XF8006064);
+	writel(0x00284141, 0XF8006068);
+	writel(0x00001610, 0XF800606C);
+	writel(0x00466111, 0XF8006078);
+	writel(0x00032222, 0XF800607C);
+	writel(0x10200802, 0XF80060A4);
+	writel(0x0670c845, 0XF80060A8);
+	writel(0x000001fe, 0XF80060AC);
+	writel(0x1cffffff, 0XF80060B0);
+	writel(0x00000200, 0XF80060B4);
+	writel(0x00200066, 0XF80060B8);
+	writel(0x00000000, 0XF80060C4);
+	writel(0x00000000, 0XF80060C8);
+	writel(0x00000000, 0XF80060DC);
+	writel(0x00000000, 0XF80060F0);
+	writel(0x00000008, 0XF80060F4);
+	writel(0x00000000, 0XF8006114);
+	writel(0x40000001, 0XF8006118);
+	writel(0x40000001, 0XF800611C);
+	writel(0x40000000, 0XF8006120);
+	writel(0x40000000, 0XF8006124);
+	writel(0x00026c05, 0XF800612C);
+	writel(0x00026007, 0XF8006130);
+	writel(0x0001d800, 0XF8006134);
+	writel(0x0001e000, 0XF8006138);
+	writel(0x00000035, 0XF8006140);
+	writel(0x00000035, 0XF8006144);
+	writel(0x00000035, 0XF8006148);
+	writel(0x00000035, 0XF800614C);
+	writel(0x00000085, 0XF8006154);
+	writel(0x00000087, 0XF8006158);
+	writel(0x0000007f, 0XF800615C);
+	writel(0x0000007c, 0XF8006160);
+	writel(0x000000f0, 0XF8006168);
+	writel(0x000000ed, 0XF800616C);
+	writel(0x000000cb, 0XF8006170);
+	writel(0x000000cd, 0XF8006174);
+	writel(0x000000c5, 0XF800617C);
+	writel(0x000000c7, 0XF8006180);
+	writel(0x000000bf, 0XF8006184);
+	writel(0x000000bc, 0XF8006188);
+	writel(0x00040080, 0XF8006190);
+	writel(0x0001fc82, 0XF8006194);
+	writel(0x00000000, 0XF8006204);
+	writel(0x000003ff, 0XF8006208);
+	writel(0x000003ff, 0XF800620C);
+	writel(0x000003ff, 0XF8006210);
+	writel(0x000003ff, 0XF8006214);
+	writel(0x000003ff, 0XF8006218);
+	writel(0x000003ff, 0XF800621C);
+	writel(0x000003ff, 0XF8006220);
+	writel(0x000003ff, 0XF8006224);
+	writel(0x00000000, 0XF80062A8);
+	writel(0x00000000, 0XF80062AC);
+	writel(0x00005125, 0XF80062B0);
+	writel(0x000012a6, 0XF80062B4);
+	writel(0x00000085, 0XF8006000);
+
+	/* Wait for DDR init to finish */
+	while ((readl(0xF8006054) &  0x00000007U) != DDR_NORMAL_OP)
+		;
+
+	/* pinmux: UART0 RxD: MIO_PIN_14, TxD: MIO_PIN_15 */
+	writel(0x000006E1, ZYNQ_MIO_BASE + 0x38);
+	writel(0x000006E0, ZYNQ_MIO_BASE + 0x3C);
+
+	/* set UART Clock to 50 MHz */
+	writel(0x00001403, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_UART_CLK_CTRL);
+
+	/* GEM0 */
+	writel(0x00000001, 0xf8000138);
+	writel(0x00100801, 0xf8000140);
+	writel(0x00000302, 0xf8000740);
+	writel(0x00000302, 0xf8000744);
+	writel(0x00000302, 0xf8000748);
+	writel(0x00000302, 0xf800074C);
+	writel(0x00000302, 0xf8000750);
+	writel(0x00000302, 0xf8000754);
+	writel(0x00001303, 0xf8000758);
+	writel(0x00001303, 0xf800075C);
+	writel(0x00001303, 0xf8000760);
+	writel(0x00001303, 0xf8000764);
+	writel(0x00001303, 0xf8000768);
+	writel(0x00001303, 0xf800076C);
+	writel(0x00000280, 0xf80007D0);
+	writel(0x00000280, 0xf80007D4);
+
+	writel(0x00000001, 0xf8000B00);
+
+	/* Set SD Detect and Write Protect Pin to 47 */
+	writel(0x002f002f, 0xF8000830);
+
+	/* FPGA Clock Control */
+	writel(0x00400500, 0xf8000170);
+	writel(0x00101400, 0xf8000180);
+	writel(0x00101400, 0xf8000190);
+	writel(0x00101400, 0xf80001a0);
+
+	/* PCAP Clock Control */
+	writel(0x00000501, 0xf8000168);
+
+	/* lock up. secure, secure */
+	writel(0x0000767B, ZYNQ_SLCR_LOCK);
+}
+
+static void cora_z7_pbl_console_init(void)
+{
+	cadence_uart_init((void *)ZYNQ_UART0_BASE_ADDR);
+	pbl_set_putc(cadence_uart_putc, (void *)ZYNQ_UART0_BASE_ADDR);
+
+	pr_debug("\nDigilent Cora Z7 PBL\n");
+}
+
+ENTRY_FUNCTION_WITHSTACK(start_digilent_cora, 0xfffff000, r0, r1, r2)
+{
+	/* MIO_07 in GPIO Mode 3.3V VIO, can be uncomented because it is the default value */
+	writel(0x0000DF0D, ZYNQ_SLCR_UNLOCK);
+	writel(0x00000600, 0xF800071C);
+	writel(0x0000767B, ZYNQ_SLCR_LOCK);
+
+	zynq_cpu_lowlevel_init();
+
+	cora_z7_ps7_init();
+
+	relocate_to_current_adr();
+	setup_c();
+	barrier();
+
+	if (IS_ENABLED(CONFIG_PBL_CONSOLE))
+		cora_z7_pbl_console_init();
+
+	barebox_arm_entry(0, SZ_512M, __dtb_z_zynq_cora_start);
+}
diff --git a/arch/arm/configs/zynq_defconfig b/arch/arm/configs/zynq_defconfig
index 1a1378d3e0..71bc5eaba7 100644
--- a/arch/arm/configs/zynq_defconfig
+++ b/arch/arm/configs/zynq_defconfig
@@ -1,5 +1,6 @@
 CONFIG_ARCH_ZYNQ=y
 CONFIG_MACH_ZEDBOARD=y
+CONFIG_MACH_CORA_Z7=y
 CONFIG_AEABI=y
 CONFIG_ARM_UNWIND=y
 CONFIG_IMAGE_COMPRESSION_XZKERN=y
diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
index 3044c9bf12..8ba6385bd6 100644
--- a/arch/arm/dts/Makefile
+++ b/arch/arm/dts/Makefile
@@ -240,6 +240,7 @@ lwl-$(CONFIG_MACH_LS1046ARDB) += fsl-ls1046a-rdb.dtb.o
 lwl-$(CONFIG_MACH_TQMLS1046A) += fsl-ls1046a-tqmls1046a-mbls10xxa.dtb.o
 lwl-$(CONFIG_MACH_LS1021AIOT) += fsl-ls1021a-iot.dtb.o
 lwl-$(CONFIG_MACH_ZEDBOARD) += zynq-zed.dtb.o
+lwl-$(CONFIG_MACH_CORA_Z7) += zynq-cora.dtb.o
 lwl-$(CONFIG_MACH_MNT_REFORM) += imx8mq-mnt-reform2.dtb.o
 lwl-$(CONFIG_MACH_VARISCITE_DT8MCUSTOMBOARD_IMX8MP) += imx8mp-var-dart-dt8mcustomboard.dtb.o
 lwl-$(CONFIG_MACH_TQMA93XX) += imx93-tqma9352-mba93xxca.dtb.o \
diff --git a/arch/arm/dts/zynq-cora-linux.dts b/arch/arm/dts/zynq-cora-linux.dts
new file mode 100644
index 0000000000..a75837011f
--- /dev/null
+++ b/arch/arm/dts/zynq-cora-linux.dts
@@ -0,0 +1,62 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ *  Copyright (C) 2011 - 2014 Xilinx
+ *  Copyright (C) 2012 National Instruments Corp.
+ */
+/dts-v1/;
+#include <arm/xilinx/zynq-7000.dtsi>
+
+/ {
+	model = "Digilent Cora Z7";
+	compatible = "digilent,zynq-cora", "xlnx,zynq-cora", "xlnx,zynq-7000";
+
+	aliases {
+		ethernet0 = &gem0;
+		serial0 = &uart0;
+		mmc0 = &sdhci0;
+	};
+
+	memory@0 {
+		device_type = "memory";
+		reg = <0x0 0x20000000>;
+	};
+
+	chosen {
+		bootargs = "";
+		stdout-path = "serial0:115200n8";
+	};
+
+	usb_phy0: phy0 {
+		compatible = "usb-nop-xceiv";
+		#phy-cells = <0>;
+	};
+};
+
+&clkc {
+	ps-clk-frequency = <50000000>;
+};
+
+&gem0 {
+	status = "okay";
+	phy-mode = "rgmii-id";
+	phy-handle = <&ethernet_phy>;
+
+	ethernet_phy: ethernet-phy@0 {
+		reg = <0>;
+		device_type = "ethernet-phy";
+	};
+};
+
+&sdhci0 {
+	status = "okay";
+};
+
+&uart0 {
+	status = "okay";
+};
+
+&usb0 {
+	status = "okay";
+	dr_mode = "host";
+	usb-phy = <&usb_phy0>;
+};
diff --git a/arch/arm/dts/zynq-cora.dts b/arch/arm/dts/zynq-cora.dts
new file mode 100644
index 0000000000..4375e840a6
--- /dev/null
+++ b/arch/arm/dts/zynq-cora.dts
@@ -0,0 +1,29 @@
+#include "zynq-cora-linux.dts"
+#include "zynq-7000.dtsi"
+
+/ {
+	chosen {
+		stdout-path = &uart0;
+
+		environment-sd {
+			compatible = "barebox,environment";
+			device-path = &sdhci0, "partname:0";
+			file-path = "barebox.env";
+		};
+	};
+};
+
+&clkc {
+	ps-clk-frequency = <50000000>;
+};
+
+&sdhci0 {
+	bootph-all;
+	bus-width = <4>;
+	status = "okay";
+};
+
+&uart0 {
+	bootph-all;
+	status = "okay";
+};
diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
index 1d6218d407..936e41f1fc 100644
--- a/arch/arm/mach-zynq/Kconfig
+++ b/arch/arm/mach-zynq/Kconfig
@@ -5,6 +5,7 @@ if ARCH_ZYNQ
 config ZYNQ_DEBUG_LL_UART_BASE
 	hex
 	default 0xe0001000 if MACH_ZEDBOARD
+	default 0xe0000000 if MACH_CORA_Z7
 
 config ARCH_ZYNQ7000
 	bool
@@ -22,6 +23,10 @@ config MACH_ZEDBOARD
 	bool "Avnet Zynq-7000 ZedBoard"
 	select ARCH_ZYNQ7000
 
+config MACH_CORA_Z7
+	bool "Digilent Cora Z7"
+	select ARCH_ZYNQ7000
+
 endmenu
 
 endif
diff --git a/images/Makefile.zynq b/images/Makefile.zynq
index ac9ce8157b..b2a584bb15 100644
--- a/images/Makefile.zynq
+++ b/images/Makefile.zynq
@@ -22,3 +22,7 @@ $(obj)/%.zynqimg: $(obj)/% FORCE
 CFG_start_avnet_zedboard.pblb.zynqimg = $(board)/avnet-zedboard/zedboard.zynqcfg
 FILE_barebox-avnet-zedboard.img = start_avnet_zedboard.pblb.zynqimg
 image-$(CONFIG_MACH_ZEDBOARD) += barebox-avnet-zedboard.img
+
+CFG_start_digilent_cora.pblb.zynqimg = $(board)/digilent-cora/cora.zynqcfg
+FILE_barebox-digilent-cora.img = start_digilent_cora.pblb.zynqimg
+image-$(CONFIG_MACH_CORA_Z7) += barebox-digilent-cora.img
-- 
2.34.1




^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Added support for Digilent Cora Z7 board
  2025-05-16 17:18 [PATCH] Added support for Digilent Cora Z7 board Johannes Roith
@ 2025-05-16 20:05 ` Ahmad Fatoum
  0 siblings, 0 replies; 2+ messages in thread
From: Ahmad Fatoum @ 2025-05-16 20:05 UTC (permalink / raw)
  To: Johannes Roith, s.hauer; +Cc: barebox

Hello Johannes,

On 16.05.25 19:18, Johannes Roith wrote:
> This patch adds support for the Digilent Cora Z7 board to barebox.
> 
> This patch includes a PBL initializing the DDR memory and the most important
> hardware and barebox proper which is loaded from the mmc but uses network
> boot as the default boot source.

For your information, the next barebox release will have a different boot.default
value by default: "bootsource net". If your SoC support sets the bootsource
correctly that means that it will look for bootloader spec entries for example
on your SD-Card before falling back to the net boot.

> mmc boot is also working.

How much on-chip SRAM do you have? I am wondering about the absence of
MMC bootstrap code in your PBL. Is your barebox small enough that it fits
into the SRAM?

> This patch only brings support for booting the PS side, loading a bitstream
> on the PL side is nit supported.

Nit: s/nit/not/ ;)

> Signed-off-by: Johannes Roith <johannes@gnu-linux.rocks>
> ---
>  arch/arm/boards/Makefile                      |   1 +
>  arch/arm/boards/digilent-cora/Makefile        |   4 +
>  arch/arm/boards/digilent-cora/board.c         |  33 ++
>  arch/arm/boards/digilent-cora/cora.zynqcfg    |   4 +
>  .../env/nv/linux.bootargs.console             |   1 +
>  arch/arm/boards/digilent-cora/lowlevel.c      | 296 ++++++++++++++++++
>  arch/arm/configs/zynq_defconfig               |   1 +
>  arch/arm/dts/Makefile                         |   1 +
>  arch/arm/dts/zynq-cora-linux.dts              |  62 ++++
>  arch/arm/dts/zynq-cora.dts                    |  29 ++
>  arch/arm/mach-zynq/Kconfig                    |   5 +
>  images/Makefile.zynq                          |   6 +
>  12 files changed, 443 insertions(+)
>  create mode 100644 arch/arm/boards/digilent-cora/Makefile
>  create mode 100644 arch/arm/boards/digilent-cora/board.c
>  create mode 100644 arch/arm/boards/digilent-cora/cora.zynqcfg
>  create mode 100644 arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console
>  create mode 100644 arch/arm/boards/digilent-cora/lowlevel.c
>  create mode 100644 arch/arm/dts/zynq-cora-linux.dts
>  create mode 100644 arch/arm/dts/zynq-cora.dts
> 
> diff --git a/arch/arm/boards/Makefile b/arch/arm/boards/Makefile
> index 908497cd8b..fcf4d393a1 100644
> --- a/arch/arm/boards/Makefile
> +++ b/arch/arm/boards/Makefile
> @@ -154,6 +154,7 @@ obj-$(CONFIG_MACH_USI_TOPKICK)			+= usi-topkick/
>  obj-$(CONFIG_MACH_VERSATILEPB)			+= versatile/
>  obj-$(CONFIG_MACH_VEXPRESS)			+= vexpress/
>  obj-$(CONFIG_MACH_ZEDBOARD)			+= avnet-zedboard/
> +obj-$(CONFIG_MACH_CORA_Z7)			+= digilent-cora/
>  obj-$(CONFIG_MACH_VARISCITE_MX6)		+= variscite-mx6/
>  obj-$(CONFIG_MACH_VARISCITE_SOM_MX7)		+= variscite-som-mx7/
>  obj-$(CONFIG_MACH_VSCOM_BALTOS)			+= vscom-baltos/
> diff --git a/arch/arm/boards/digilent-cora/Makefile b/arch/arm/boards/digilent-cora/Makefile
> new file mode 100644
> index 0000000000..d653aa1f4b
> --- /dev/null
> +++ b/arch/arm/boards/digilent-cora/Makefile
> @@ -0,0 +1,4 @@
> +# SPDX-License-Identifier: GPL-2.0-only
> +
> +obj-y += board.o
> +lwl-y += lowlevel.o
> diff --git a/arch/arm/boards/digilent-cora/board.c b/arch/arm/boards/digilent-cora/board.c
> new file mode 100644
> index 0000000000..1e70137352
> --- /dev/null
> +++ b/arch/arm/boards/digilent-cora/board.c
> @@ -0,0 +1,33 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// SPDX-FileCopyrightText: 2025 Johannes Roith <johannes@gnu-linux.rocks>
> +
> +#include <asm/armlinux.h>
> +#include <bbu.h>
> +#include <common.h>
> +#include <environment.h>
> +#include <asm/mach-types.h>
> +#include <init.h>
> +#include <mach/zynq/zynq7000-regs.h>
> +#include <linux/sizes.h>
> +#include <deep-probe.h>
> +
> +static int cora_probe(struct device *dev)
> +{
> +	barebox_set_hostname("cora");

The name would already be zynq-cora by default. If that's ok with you,
I wouldn't change it.

> diff --git a/arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console b/arch/arm/boards/digilent-cora/env/nv/linux.bootargs.console

The environment is unreferenced. You can use bbenv- to compile in environment
without having to reference it in the config. This should be preferred
over setting it in the Kconfig as that allows you to build a config
with multiple board and only apply the env when appropriate (by
calling defaultenv_append_directory in the board driver).

> +static void cora_z7_ps7_init(void)
> +{
> +	/*
> +	 * Read OCM mapping configuration, if only the upper 64 KByte are
> +	 * mapped to the high address, it's very likely that we just got control
> +	 * from the BootROM. If the mapping is changed something other than the
> +	 * BootROM was running before us. Skip PS7 init to avoid cutting the
> +	 * branch we are sitting on in that case.
> +	 */
> +	if ((readl(0xf8000910) & 0xf) != 0x8)

Please use a macro for the address and wrap it in IOMEM().
Eventually, we will want to enforce addresses to be pointers, so it would
be cool for new code to already use the correct types. If you want type
checking, just add

#include <linux/io.h> at the top of your includes.

> +		return;
> +	/* Do something!!! */
> +	/* open sesame */

Is there some stuff that's generic enough here that we would want it in
mach-zynq instead?

> +	/* pinmux: UART0 RxD: MIO_PIN_14, TxD: MIO_PIN_15 */
> +	writel(0x000006E1, ZYNQ_MIO_BASE + 0x38);
> +	writel(0x000006E0, ZYNQ_MIO_BASE + 0x3C);
> +
> +	/* set UART Clock to 50 MHz */
> +	writel(0x00001403, ZYNQ_CLOCK_CTRL_BASE + ZYNQ_UART_CLK_CTRL);

I'd move that into cora_z7_pbl_console_init if you don't need it
unconditionally. If it's needed outside of PBL_CONSOLE, I think clock
and pinctrl drivers in barebox should already take care of it?

> +
> +	/* GEM0 */
> +	writel(0x00000001, 0xf8000138);
> +	writel(0x00100801, 0xf8000140);
> +	writel(0x00000302, 0xf8000740);
> +	writel(0x00000302, 0xf8000744);
> +	writel(0x00000302, 0xf8000748);
> +	writel(0x00000302, 0xf800074C);
> +	writel(0x00000302, 0xf8000750);
> +	writel(0x00000302, 0xf8000754);
> +	writel(0x00001303, 0xf8000758);
> +	writel(0x00001303, 0xf800075C);
> +	writel(0x00001303, 0xf8000760);
> +	writel(0x00001303, 0xf8000764);
> +	writel(0x00001303, 0xf8000768);
> +	writel(0x00001303, 0xf800076C);
> +	writel(0x00000280, 0xf80007D0);
> +	writel(0x00000280, 0xf80007D4);

What are you doing to the MAC here?

> +ENTRY_FUNCTION_WITHSTACK(start_digilent_cora, 0xfffff000, r0, r1, r2)
> +{
> +	/* MIO_07 in GPIO Mode 3.3V VIO, can be uncomented because it is the default value */
> +	writel(0x0000DF0D, ZYNQ_SLCR_UNLOCK);
> +	writel(0x00000600, 0xF800071C);
> +	writel(0x0000767B, ZYNQ_SLCR_LOCK);
> +
> +	zynq_cpu_lowlevel_init();
> +
> +	cora_z7_ps7_init();
> +
> +	relocate_to_current_adr();
> +	setup_c();
> +	barrier();

You don't need the barrier().

> +
> +	if (IS_ENABLED(CONFIG_PBL_CONSOLE))
> +		cora_z7_pbl_console_init();
> +
> +	barebox_arm_entry(0, SZ_512M, __dtb_z_zynq_cora_start);
> +}
> diff --git a/arch/arm/configs/zynq_defconfig b/arch/arm/configs/zynq_defconfig
> index 1a1378d3e0..71bc5eaba7 100644
> --- a/arch/arm/configs/zynq_defconfig
> +++ b/arch/arm/configs/zynq_defconfig
> @@ -1,5 +1,6 @@
>  CONFIG_ARCH_ZYNQ=y
>  CONFIG_MACH_ZEDBOARD=y
> +CONFIG_MACH_CORA_Z7=y
>  CONFIG_AEABI=y
>  CONFIG_ARM_UNWIND=y
>  CONFIG_IMAGE_COMPRESSION_XZKERN=y
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 3044c9bf12..8ba6385bd6 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -240,6 +240,7 @@ lwl-$(CONFIG_MACH_LS1046ARDB) += fsl-ls1046a-rdb.dtb.o
>  lwl-$(CONFIG_MACH_TQMLS1046A) += fsl-ls1046a-tqmls1046a-mbls10xxa.dtb.o
>  lwl-$(CONFIG_MACH_LS1021AIOT) += fsl-ls1021a-iot.dtb.o
>  lwl-$(CONFIG_MACH_ZEDBOARD) += zynq-zed.dtb.o
> +lwl-$(CONFIG_MACH_CORA_Z7) += zynq-cora.dtb.o
>  lwl-$(CONFIG_MACH_MNT_REFORM) += imx8mq-mnt-reform2.dtb.o
>  lwl-$(CONFIG_MACH_VARISCITE_DT8MCUSTOMBOARD_IMX8MP) += imx8mp-var-dart-dt8mcustomboard.dtb.o
>  lwl-$(CONFIG_MACH_TQMA93XX) += imx93-tqma9352-mba93xxca.dtb.o \
> diff --git a/arch/arm/dts/zynq-cora-linux.dts b/arch/arm/dts/zynq-cora-linux.dts
> new file mode 100644
> index 0000000000..a75837011f
> --- /dev/null
> +++ b/arch/arm/dts/zynq-cora-linux.dts
> @@ -0,0 +1,62 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + *  Copyright (C) 2011 - 2014 Xilinx
> + *  Copyright (C) 2012 National Instruments Corp.
> + */
> +/dts-v1/;
> +#include <arm/xilinx/zynq-7000.dtsi>
> +
> +/ {
> +	model = "Digilent Cora Z7";
> +	compatible = "digilent,zynq-cora", "xlnx,zynq-cora", "xlnx,zynq-7000";
> +
> +	aliases {
> +		ethernet0 = &gem0;
> +		serial0 = &uart0;
> +		mmc0 = &sdhci0;
> +	};
> +
> +	memory@0 {
> +		device_type = "memory";
> +		reg = <0x0 0x20000000>;
> +	};
> +
> +	chosen {
> +		bootargs = "";

This is unnecessary, I think.

> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	usb_phy0: phy0 {

If you upstream this, you'll be probably told that you should use a descriptive
node name.

> +		compatible = "usb-nop-xceiv";
> +		#phy-cells = <0>;
> +	};
> +};
> +
> +&clkc {
> +	ps-clk-frequency = <50000000>;
> +};
> +
> +&gem0 {
> +	status = "okay";
> +	phy-mode = "rgmii-id";
> +	phy-handle = <&ethernet_phy>;
> +
> +	ethernet_phy: ethernet-phy@0 {
> +		reg = <0>;

Is 0 really your PHY address or does it just work by virtue of being the broadcast
address? Just wondering.

> +		device_type = "ethernet-phy";
> +	};
> +};
> +
> +&sdhci0 {
> +	status = "okay";
> +};
> +
> +&uart0 {
> +	status = "okay";
> +};
> +
> +&usb0 {
> +	status = "okay";
> +	dr_mode = "host";
> +	usb-phy = <&usb_phy0>;
> +};
> diff --git a/arch/arm/dts/zynq-cora.dts b/arch/arm/dts/zynq-cora.dts
> new file mode 100644
> index 0000000000..4375e840a6
> --- /dev/null
> +++ b/arch/arm/dts/zynq-cora.dts
> @@ -0,0 +1,29 @@
> +#include "zynq-cora-linux.dts"
> +#include "zynq-7000.dtsi"
> +
> +/ {
> +	chosen {
> +		stdout-path = &uart0;

Why overwrite what's in the Linux DTS?

> +
> +		environment-sd {
> +			compatible = "barebox,environment";
> +			device-path = &sdhci0, "partname:0";
> +			file-path = "barebox.env";
> +		};
> +	};
> +};
> +
> +&clkc {
> +	ps-clk-frequency = <50000000>;

Why duplicate?

> +};
> +
> +&sdhci0 {
> +	bootph-all;

Unparsed by barebox.

> +	bus-width = <4>;

It's a SD-Card, so it can't be more than 4 bit. Just drop it.

> +	status = "okay";
> +};
> +
> +&uart0 {
> +	bootph-all;
> +	status = "okay";
> +};
> diff --git a/arch/arm/mach-zynq/Kconfig b/arch/arm/mach-zynq/Kconfig
> index 1d6218d407..936e41f1fc 100644
> --- a/arch/arm/mach-zynq/Kconfig
> +++ b/arch/arm/mach-zynq/Kconfig
> @@ -5,6 +5,7 @@ if ARCH_ZYNQ
>  config ZYNQ_DEBUG_LL_UART_BASE
>  	hex
>  	default 0xe0001000 if MACH_ZEDBOARD
> +	default 0xe0000000 if MACH_CORA_Z7
>  
>  config ARCH_ZYNQ7000
>  	bool
> @@ -22,6 +23,10 @@ config MACH_ZEDBOARD
>  	bool "Avnet Zynq-7000 ZedBoard"
>  	select ARCH_ZYNQ7000
>  
> +config MACH_CORA_Z7
> +	bool "Digilent Cora Z7"
> +	select ARCH_ZYNQ7000

Can you enable this in zynq_defconfig?

Cheers,
Ahmad

> +
>  endmenu
>  
>  endif
> diff --git a/images/Makefile.zynq b/images/Makefile.zynq
> index ac9ce8157b..b2a584bb15 100644
> --- a/images/Makefile.zynq
> +++ b/images/Makefile.zynq
> @@ -22,3 +22,7 @@ $(obj)/%.zynqimg: $(obj)/% FORCE
>  CFG_start_avnet_zedboard.pblb.zynqimg = $(board)/avnet-zedboard/zedboard.zynqcfg
>  FILE_barebox-avnet-zedboard.img = start_avnet_zedboard.pblb.zynqimg
>  image-$(CONFIG_MACH_ZEDBOARD) += barebox-avnet-zedboard.img
> +
> +CFG_start_digilent_cora.pblb.zynqimg = $(board)/digilent-cora/cora.zynqcfg
> +FILE_barebox-digilent-cora.img = start_digilent_cora.pblb.zynqimg
> +image-$(CONFIG_MACH_CORA_Z7) += barebox-digilent-cora.img


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-05-16 20:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-05-16 17:18 [PATCH] Added support for Digilent Cora Z7 board Johannes Roith
2025-05-16 20:05 ` Ahmad Fatoum

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox