colibri-imx8x: extract is_imx8dx() from ram detection

Refactor the detection of QXP vs. DX SoC into its own helper function.

Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
Signed-off-by: Andrejs Cainikovs <andrejs.cainikovs@toradex.com>

diff --git a/board/toradex/colibri-imx8x/colibri-imx8x.c b/board/toradex/colibri-imx8x/colibri-imx8x.c
index 169d4d04b1632c13be5098b28355b16c6fe156e0..88eddbddaef4e762addcc120074a341d206ea797 100644 (file)
--- a/board/toradex/colibri-imx8x/colibri-imx8x.c
+++ b/board/toradex/colibri-imx8x/colibri-imx8x.c
@@ -40,21 +40,25 @@ static void setup_iomux_uart(void)
        imx8_iomux_setup_multiple_pads(uart3_pads, ARRAY_SIZE(uart3_pads));
 }
 
-void board_mem_get_layout(u64 *phys_sdram_1_start,
-                         u64 *phys_sdram_1_size,
-                         u64 *phys_sdram_2_start,
-                         u64 *phys_sdram_2_size)
+static int is_imx8dx(void)
 {
-       u32 is_dualx = 0, val = 0;
-       sc_err_t scierr = sc_misc_otp_fuse_read(-1, 6, &val);
+       u32 val = 0;
+       sc_err_t sc_err = sc_misc_otp_fuse_read(-1, 6, &val);
 
-       if (scierr == SC_ERR_NONE) {
+       if (sc_err == SC_ERR_NONE) {
                /* DX has two A35 cores disabled */
-               is_dualx = (val & 0xf) != 0x0;
+               return (val & 0xf) != 0x0;
        }
+       return false;
+}
 
+void board_mem_get_layout(u64 *phys_sdram_1_start,
+                         u64 *phys_sdram_1_size,
+                         u64 *phys_sdram_2_start,
+                         u64 *phys_sdram_2_size)
+{
        *phys_sdram_1_start = PHYS_SDRAM_1;
-       if (is_dualx)
+       if (is_imx8dx())
                /* Our DX based SKUs only have 1 GB RAM */
                *phys_sdram_1_size = SZ_1G;
        else