From 95b57899cd8dde8be6a95eb8abab35a5e15605bd Mon Sep 17 00:00:00 2001 From: Florent Kermarrec Date: Tue, 5 May 2020 16:27:21 +0200 Subject: [PATCH] bios: remove usddrphy debug (we'll use a specific debug firmware to fix the usddrphy corner cases). --- litex/boards/targets/kcu105.py | 1 - litex/soc/software/bios/Makefile | 13 +- .../soc/software/bios/commands/cmd_usddrphy.c | 128 ------------------ litex/soc/software/bios/sdram.c | 91 +------------ litex/soc/software/bios/sdram.h | 7 - 5 files changed, 7 insertions(+), 233 deletions(-) delete mode 100644 litex/soc/software/bios/commands/cmd_usddrphy.c diff --git a/litex/boards/targets/kcu105.py b/litex/boards/targets/kcu105.py index a0c98f86..138f3ffa 100755 --- a/litex/boards/targets/kcu105.py +++ b/litex/boards/targets/kcu105.py @@ -68,7 +68,6 @@ class BaseSoC(SoCCore): iodelay_clk_freq = 200e6, cmd_latency = 1) self.add_csr("ddrphy") - self.add_constant("USDDRPHY_DEBUG") self.add_sdram("sdram", phy = self.ddrphy, module = EDY4016A(sys_clk_freq, "1:4"), diff --git a/litex/soc/software/bios/Makefile b/litex/soc/software/bios/Makefile index 0369522e..9fba33a6 100755 --- a/litex/soc/software/bios/Makefile +++ b/litex/soc/software/bios/Makefile @@ -11,12 +11,12 @@ CFLAGS += -DTFTP_SERVER_PORT=$(TFTP_SERVER_PORT) endif OBJECTS = isr.o \ - sdram.o \ - sdcard.o \ - main.o \ + sdram.o \ + sdcard.o \ + main.o \ boot-helper.o \ - boot.o \ - helpers.o \ + boot.o \ + helpers.o \ cmd_bios.o \ cmd_boot.o \ cmd_dram.o \ @@ -24,7 +24,6 @@ OBJECTS = isr.o \ cmd_mem_access.o \ cmd_sdcard.o \ cmd_spi_flash.o \ - cmd_usddrphy.o ifneq "$(or $(TERM_NO_COMPLETE),$(TERM_MINI))" "" CFLAGS += -DTERM_NO_COMPLETE @@ -70,7 +69,7 @@ bios.elf: $(BIOS_DIRECTORY)/linker.ld $(OBJECTS) $(BP_LIBS) \ -lnet -lbase-nofloat -lcompiler_rt \ $(BP_FLAGS) - + ifneq ($(OS),Windows_NT) chmod -x $@ endif diff --git a/litex/soc/software/bios/commands/cmd_usddrphy.c b/litex/soc/software/bios/commands/cmd_usddrphy.c deleted file mode 100644 index 69c62fad..00000000 --- a/litex/soc/software/bios/commands/cmd_usddrphy.c +++ /dev/null @@ -1,128 +0,0 @@ -// SPDX-License-Identifier: BSD-Source-Code - -#include -#include - -#include - -#include "../command.h" -#include "../helpers.h" -#include "../sdram.h" - -/** - * Command "sdram_cdly" - * - * Set SDRAM clk/cmd delay - * - */ -#ifdef USDDRPHY_DEBUG -static void sdram_cdly(int nb_params, char **params) -{ - unsigned int delay; - char *c; - - if (nb_params < 1) { - printf("sdram_cdly "); - return; - } - - delay = strtoul(params[0], &c, 0); - if (*c != 0) { - printf("Incorrect delay"); - return; - } - - ddrphy_cdly(delay); -} - -define_command(sdram_cdly, sdram_cdly, "Set SDRAM clk/cmd delay", DDR_CMDS); -#endif - -/** - * Command "sdram_cdly" - * - * Run SDRAM calibration - * - */ -#ifdef USDDRPHY_DEBUG -define_command(sdram_cal, sdram_cal, "Run SDRAM calibration", DDR_CMDS); -#endif - -/** - * Command "sdram_mpr" - * - * Read SDRAM MPR - * - */ -#ifdef USDDRPHY_DEBUG -define_command(sdram_mpr, sdram_mpr, "Read SDRAM MPR", DDR_CMDS); -#endif - - -/** - * Command "sdram_mrwr" - * - * Write SDRAM mode registers - * - */ -#ifdef USDDRPHY_DEBUG -static void sdram_mrwr(int nb_params, char **params) -{ - unsigned int reg; - unsigned int value; - char *c; - - if (nb_params < 2) { - printf("sdram_mrwr "); - return; - } - - reg = strtoul(params[0], &c, 0); - if (*c != 0) { - printf("Incorrect register value"); - return; - } - - value = strtoul(params[1], &c, 0); - if (*c != 0) { - printf("Incorrect value"); - return; - } - - sdrsw(); - printf("Writing 0x%04x to SDRAM mode register %d\n", value, reg); - sdrmrwr(reg, value); - sdrhw(); -} - -define_command(sdram_mrwr, sdram_mrwr, "Write SDRAM mode registers", DDR_CMDS); -#endif - -/** - * Command "sdram_cdly_scan" - * - * Enable/disable cdly scan - * - */ -#ifdef USDDRPHY_DEBUG -static void sdram_cdly_scan(int nb_params, char **params) -{ - unsigned int value; - char *c; - - if (nb_params < 1) { - printf("sdram_cdly_scan "); - return; - } - - value = strtoul(params[0], &c, 0); - if (*c != 0) { - printf("Incorrect value"); - return; - } - - sdr_cdly_scan(value); -} - -define_command(sdram_cdly_scan, sdram_cdly_scan, "Enable/disable cdly scan", DDR_CMDS); -#endif diff --git a/litex/soc/software/bios/sdram.c b/litex/soc/software/bios/sdram.c index 1923b49b..9056a2ff 100644 --- a/litex/soc/software/bios/sdram.c +++ b/litex/soc/software/bios/sdram.c @@ -166,7 +166,7 @@ void sdrwr(unsigned int addr) #ifdef CSR_DDRPHY_BASE -#if defined(DDRPHY_CMD_DELAY) || defined(USDDRPHY_DEBUG) +#if defined(DDRPHY_CMD_DELAY) void ddrphy_cdly(unsigned int delay) { printf("Setting clk/cmd delay to %d taps\n", delay); #if CSR_DDRPHY_EN_VTC_ADDR @@ -1048,93 +1048,4 @@ int sdrinit(void) return 1; } -#ifdef USDDRPHY_DEBUG - -#define MPR0_SEL (0 << 0) -#define MPR1_SEL (1 << 0) -#define MPR2_SEL (2 << 0) -#define MPR3_SEL (3 << 0) - -#define MPR_ENABLE (1 << 2) - -#define MPR_READ_SERIAL (0 << 11) -#define MPR_READ_PARALLEL (1 << 11) -#define MPR_READ_STAGGERED (2 << 11) - -void sdrcal(void) -{ -#ifdef CSR_DDRPHY_BASE -#if CSR_DDRPHY_EN_VTC_ADDR - ddrphy_en_vtc_write(0); -#endif - sdrlevel(); -#if CSR_DDRPHY_EN_VTC_ADDR - ddrphy_en_vtc_write(1); -#endif -#endif - sdrhw(); -} - -void sdrmrwr(char reg, int value) { - sdram_dfii_pi0_address_write(value); - sdram_dfii_pi0_baddress_write(reg); - command_p0(DFII_COMMAND_RAS|DFII_COMMAND_CAS|DFII_COMMAND_WE|DFII_COMMAND_CS); -} - -static void sdrmpron(char mpr) -{ - sdrmrwr(3, MPR_READ_SERIAL | MPR_ENABLE | mpr); -} - -static void sdrmproff(void) -{ - sdrmrwr(3, 0); -} - -void sdrmpr(void) -{ - int module, phase; - unsigned char buf[DFII_PIX_DATA_BYTES]; - printf("Read SDRAM MPR...\n"); - - /* rst phy */ - for(module=0; module