ram: stm32mp1: use the DDR size by default in the test addressBus
authorPatrick Delaunay <patrick.delaunay@st.com>
Thu, 2 Jul 2020 17:56:36 +0000 (19:56 +0200)
committerPatrick Delaunay <patrick.delaunay@st.com>
Tue, 7 Jul 2020 14:01:23 +0000 (16:01 +0200)
The test 4 = "AddressBus [size] [addr]" without parameter
detects alias for any address bit only when:
- size = real size of DDR
- addr = start of DDR = 0xC0000000

These value must be the default value when parameters are absent.

This patch sets bufsize to STM32_DDR_SIZE and get_bufsize() selects
the correct value for bufsize when this parameter is absent =
full size of the DDDR

On EV1 board :
DDR> test 4
running at 0xC0000000 length 0x40000000

On DK2 board
DDR> test 4
running at 0xC0000000 length 0x20000000

Reviewed-by: Patrice Chotard <patrice.chotard@st.com>
Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
drivers/ram/stm32mp1/stm32mp1_tests.c

index fec9fd0..a2a69da 100644 (file)
@@ -442,7 +442,7 @@ static enum test_result test_addressbus(struct stm32mp1_ddrctl *ctl,
        u32 bufsize;
        u32 error;
 
-       if (get_bufsize(string, argc, argv, 0, &bufsize, 4 * 1024, 4))
+       if (get_bufsize(string, argc, argv, 0, &bufsize, STM32_DDR_SIZE, 4))
                return TEST_ERROR;
        if (!is_power_of_2(bufsize)) {
                sprintf(string, "size 0x%x is not a power of 2",
@@ -452,6 +452,7 @@ static enum test_result test_addressbus(struct stm32mp1_ddrctl *ctl,
        if (get_addr(string, argc, argv, 1, &addr))
                return TEST_ERROR;
 
+       printf("running at 0x%08x length 0x%x\n", addr, bufsize);
        error = (u32)addressbus((u32 *)addr, bufsize);
        if (error) {
                sprintf(string, "0x%x: error for address 0x%x",