From 288306c86a9f4392cf3ef6d071a272600c4edd6b Mon Sep 17 00:00:00 2001 From: Florent Kermarrec Date: Thu, 15 Oct 2020 16:20:05 +0200 Subject: [PATCH] software/liblitedram: add initial Build-In Self-Test software. To be used with LiteDRAM's BIST Generator/Checker, ex: from litedram.frontend.bist import LiteDRAMBISTGenerator, LiteDRAMBISTChecker self.submodules.sdram_generator = LiteDRAMBISTGenerator(self.sdram.crossbar.get_port()) self.add_csr("sdram_generator") self.submodules.sdram_checker = LiteDRAMBISTChecker(self.sdram.crossbar.get_port()) self.add_csr("sdram_checker") --- litex/soc/software/bios/cmds/cmd_litedram.c | 15 ++ litex/soc/software/liblitedram/Makefile | 2 +- litex/soc/software/liblitedram/bist.c | 171 ++++++++++++++++++++ litex/soc/software/liblitedram/bist.h | 9 ++ 4 files changed, 196 insertions(+), 1 deletion(-) create mode 100644 litex/soc/software/liblitedram/bist.c create mode 100644 litex/soc/software/liblitedram/bist.h diff --git a/litex/soc/software/bios/cmds/cmd_litedram.c b/litex/soc/software/bios/cmds/cmd_litedram.c index 234814e4d..b39ea7c37 100644 --- a/litex/soc/software/bios/cmds/cmd_litedram.c +++ b/litex/soc/software/bios/cmds/cmd_litedram.c @@ -11,6 +11,7 @@ #include #include +#include #include "../command.h" #include "../helpers.h" @@ -55,6 +56,20 @@ static void sdram_test_handler(int nb_params, char **params) define_command(sdram_test, sdram_test_handler, "Test SDRAM", LITEDRAM_CMDS); #endif +/** + * Command "sdram_bist" + * + * Run SDRAM Build-In Self-Test + * + */ +#if defined(CSR_SDRAM_GENERATOR_BASE) && defined(CSR_SDRAM_CHECKER_BASE) +static void sdram_bist_handler(int nb_params, char **params) +{ + sdram_bist(128, 0); +} +define_command(sdram_bist, sdram_bist_handler, "Run SDRAM Build-In Self-Test", LITEDRAM_CMDS); +#endif + #ifdef CSR_DDRPHY_RDPHASE_ADDR /** * Command "sdram_force_rdphase" diff --git a/litex/soc/software/liblitedram/Makefile b/litex/soc/software/liblitedram/Makefile index c541e3f65..56f4a5445 100644 --- a/litex/soc/software/liblitedram/Makefile +++ b/litex/soc/software/liblitedram/Makefile @@ -1,7 +1,7 @@ include ../include/generated/variables.mak include $(SOC_DIRECTORY)/software/common.mak -OBJECTS = sdram.o +OBJECTS = sdram.o bist.o all: liblitedram.a diff --git a/litex/soc/software/liblitedram/bist.c b/litex/soc/software/liblitedram/bist.c new file mode 100644 index 000000000..214d5f78d --- /dev/null +++ b/litex/soc/software/liblitedram/bist.c @@ -0,0 +1,171 @@ +// This file is Copyright (c) 2018-2020 Florent Kermarrec +// License: BSD + +#include +#if defined(CSR_SDRAM_GENERATOR_BASE) && defined(CSR_SDRAM_CHECKER_BASE) +#include +#include +#include +#include +#include +#include + +#include "bist.h" + +#define SDRAM_TEST_BASE 0x00000000 +#define SDRAM_TEST_DATA_BYTES (CSR_SDRAM_DFII_PI0_RDDATA_SIZE*4) + +uint64_t wr_ticks; +uint64_t wr_length; +uint64_t rd_ticks; +uint64_t rd_length; +uint64_t rd_errors; + +__attribute__((unused)) static void cdelay(int i) +{ +#ifndef CONFIG_SIM_DISABLE_DELAYS + while(i > 0) { + __asm__ volatile(CONFIG_CPU_NOP); + i--; + } +#endif +} + +static uint32_t pseudo_random_bases[128] = { + 0x000e4018,0x0003338d,0x00233429,0x001f589d, + 0x001c922b,0x0011dc60,0x000d1e8f,0x000b20cf, + 0x00360188,0x00041174,0x0003d065,0x000bfe34, + 0x001bfc54,0x001dc7d5,0x00036587,0x00197383, + 0x0035b2d3,0x001c3765,0x00397fae,0x00239bc0, + 0x0000d4f3,0x00146fb7,0x0036183a,0x002b8d54, + 0x00239149,0x0013e6c0,0x001b8f66,0x002b1587, + 0x000d1539,0x000bdf18,0x0030a175,0x000c6133, + 0x002df309,0x002c06bd,0x0021dbd1,0x00058fc8, + 0x003ace6f,0x000ffa4d,0x003073d0,0x000a161f, + 0x002586dd,0x002e4a0e,0x00189ce9,0x0008e72e, + 0x0005dd92,0x001d2bc5,0x00250aaa,0x000a369f, + 0x001dcc17,0x000ced9d,0x0030a7f9,0x002394a3, + 0x003a0959,0x002eb2d2,0x0014d1d9,0x002f6217, + 0x002d7982,0x001ad120,0x00222c54,0x000923b7, + 0x0015e7df,0x001f55f6,0x0014ea5f,0x003b2b57, + 0x003091fe,0x00228da6,0x001c1c59,0x00298218, + 0x000728f9,0x001d5172,0x00041bdc,0x002860c3, + 0x0033595e,0x00224555,0x000878de,0x001b017c, + 0x0028475d,0x001b3758,0x003fe6cf,0x0032a410, + 0x003abba8,0x0012499d,0x0021e797,0x0011df68, + 0x001f917d,0x0021a184,0x0036d6eb,0x00331f8e, + 0x002e55e6,0x001c12b3,0x0011b4da,0x003f2b86, + 0x000ba2eb,0x000607e8,0x000e08fb,0x0013904d, + 0x00147a4a,0x00360956,0x000821ad,0x0031400e, + 0x0030d8e6,0x003be90f,0x00202e56,0x00017835, + 0x000ea9a1,0x00222753,0x002b8ade,0x000e4757, + 0x00259169,0x0037a663,0x00143e83,0x003a139e, + 0x00006a57,0x0021b6bb,0x0016de10,0x000d9ede, + 0x0014ada3,0x000012bd,0x00297df2,0x003e8aa1, + 0x00027e36,0x000e51ae,0x002e7627,0x00275c9f, +}; + +static void sdram_bist_loop(uint32_t loop, uint32_t burst_length, uint32_t random) { + int i; + uint32_t base; + uint32_t length; + length = burst_length*SDRAM_TEST_DATA_BYTES; + + rd_errors = 0; + for(i=0; i<128; i++) { + if (random) + base = SDRAM_TEST_BASE + pseudo_random_bases[(i+loop)%128]*SDRAM_TEST_DATA_BYTES; + else + base = SDRAM_TEST_BASE + ((i+loop)%128)*SDRAM_TEST_DATA_BYTES; + if (i == 0) { + /* Prepare first write */ + sdram_generator_reset_write(1); + sdram_generator_reset_write(0); + sdram_generator_random_write(1); /* Random data */ + sdram_generator_base_write(base); + sdram_generator_length_write(length); + cdelay(100); + } + /* Start write */ + sdram_generator_start_write(1); + /* Prepare next read */ + sdram_checker_reset_write(1); + sdram_checker_reset_write(0); + sdram_checker_random_write(1); /* Random data */ + sdram_checker_base_write(base); + sdram_checker_length_write(length); + cdelay(100); + /* Wait write */ + while(sdram_generator_done_read() == 0); + /* Get write results */ + wr_length += length; + wr_ticks += sdram_generator_ticks_read(); + /* Start read */ + sdram_checker_start_write(1); + if (i != 127) { + if (random) + base = SDRAM_TEST_BASE + pseudo_random_bases[(i+1+loop)%128]*SDRAM_TEST_DATA_BYTES; + else + base = SDRAM_TEST_BASE + ((i+1+loop)%128)*SDRAM_TEST_DATA_BYTES; + /* Prepare next write */ + sdram_generator_reset_write(1); + sdram_generator_reset_write(0); + sdram_generator_random_write(1); /* Random data */ + sdram_generator_base_write(base); + sdram_generator_length_write(length); + cdelay(100); + } + /* Wait read */ + while(sdram_checker_done_read() == 0); + /* Get read results */ + rd_ticks += sdram_checker_ticks_read(); + rd_errors += sdram_checker_errors_read(); + } +} + +static uint64_t compute_speed(uint64_t length, uint64_t ticks) { + uint64_t speed; + speed = (8*length*CONFIG_CLOCK_FREQUENCY)/ticks; + return speed; +} + +void sdram_bist(uint32_t burst_length, uint32_t random) +{ + uint32_t i; + uint64_t total_length; + uint64_t total_errors; + + i = 0; + total_length = 0; + total_errors = 0; + for(;;) { + /* Exit on key pressed */ + if (readchar_nonblock()) + break; + + /* Bist loop */ + sdram_bist_loop(i, burst_length, random); + + /* Results */ + if (i%1000 == 0) + printf("WR-SPEED(Mbps) RD-SPEED(Mbps) TESTED(MiB) ERRORS\n"); + if (i%100 == 100-1) { + printf("%14lld %14lld %11lld %12lld\n", + compute_speed(wr_length, wr_ticks), + compute_speed(rd_length, rd_ticks), + total_length/(1024*1024), + total_errors); + total_length += wr_length; + total_errors += rd_errors; + /* Clear length/ticks/errors */ + wr_length = 0; + wr_ticks = 0; + rd_length = 0; + rd_ticks = 0; + rd_errors = 0; + } + i++; + } +} + +#endif diff --git a/litex/soc/software/liblitedram/bist.h b/litex/soc/software/liblitedram/bist.h new file mode 100644 index 000000000..e1cada233 --- /dev/null +++ b/litex/soc/software/liblitedram/bist.h @@ -0,0 +1,9 @@ +// This file is Copyright (c) 2018-2020 Florent Kermarrec +// License: BSD + +#ifndef __SDRAM_BIST_H +#define __SDRAM_BIST_H + +void sdram_bist(uint32_t burst_length, uint32_t random); + +#endif /* __SDRAM_BIST_H */