[U-Boot] [PATCH v3 06/22] ram: rockchip: add common code for sdram driver

Jagan Teki jagan at amarulasolutions.com
Wed Nov 27 09:50:26 UTC 2019


Hi Kever/YouMin,

On Fri, Nov 15, 2019 at 8:43 AM Kever Yang <kever.yang at rock-chips.com> wrote:
>
> There are some function like os_reg setting, capacity detect functions,
> can be used as common code for different Rockchip SoCs, add a
> sdram_common.c for all these functions.
>
> Signed-off-by: YouMin Chen <cym at rock-chips.com>
> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
> ---
>
> Changes in v3: None
> Changes in v2: None
>
>  .../include/asm/arch-rockchip/sdram_common.h  |  31 ++
>  drivers/ram/rockchip/Kconfig                  |   6 +
>  drivers/ram/rockchip/Makefile                 |   1 +
>  drivers/ram/rockchip/sdram_common.c           | 285 ++++++++++++++++++
>  4 files changed, 323 insertions(+)
>  create mode 100644 drivers/ram/rockchip/sdram_common.c
>
> diff --git a/arch/arm/include/asm/arch-rockchip/sdram_common.h b/arch/arm/include/asm/arch-rockchip/sdram_common.h
> index cd3d7f97d8..36d31156be 100644
> --- a/arch/arm/include/asm/arch-rockchip/sdram_common.h
> +++ b/arch/arm/include/asm/arch-rockchip/sdram_common.h
> @@ -6,6 +6,15 @@
>  #ifndef _ASM_ARCH_SDRAM_COMMON_H
>  #define _ASM_ARCH_SDRAM_COMMON_H
>
> +#ifndef MHZ
> +#define MHZ            (1000 * 1000)
> +#endif
> +
> +#define PATTERN                (0x5aa5f00f)
> +
> +#define MIN(a, b)      (((a) > (b)) ? (b) : (a))
> +#define MAX(a, b)      (((a) > (b)) ? (a) : (b))
> +
>  struct sdram_cap_info {
>         unsigned int rank;
>         /* dram column number, 0 means this channel is invalid */
> @@ -125,4 +134,26 @@ void sdram_print_ddr_info(struct sdram_cap_info *cap_info,
>  void sdram_print_stride(unsigned int stride);
>  #endif /* CONFIG_RAM_ROCKCHIP_DEBUG */
>
> +void sdram_org_config(struct sdram_cap_info *cap_info,
> +                     struct sdram_base_params *base,
> +                     u32 *p_os_reg2, u32 *p_os_reg3, u32 channel);
> +
> +int sdram_detect_bw(struct sdram_cap_info *cap_info);
> +int sdram_detect_cs(struct sdram_cap_info *cap_info);
> +int sdram_detect_col(struct sdram_cap_info *cap_info,
> +                    u32 coltmp);
> +int sdram_detect_bank(struct sdram_cap_info *cap_info,
> +                     u32 coltmp, u32 bktmp);
> +int sdram_detect_bg(struct sdram_cap_info *cap_info,
> +                   u32 coltmp);
> +int sdram_detect_dbw(struct sdram_cap_info *cap_info, u32 dram_type);
> +int sdram_detect_row(struct sdram_cap_info *cap_info,
> +                    u32 coltmp, u32 bktmp, u32 rowtmp);
> +int sdram_detect_row_3_4(struct sdram_cap_info *cap_info,
> +                        u32 coltmp, u32 bktmp);
> +int sdram_detect_high_row(struct sdram_cap_info *cap_info);
> +int sdram_detect_cs1_row(struct sdram_cap_info *cap_info, u32 dram_type);
> +u64 sdram_get_cs_cap(struct sdram_cap_info *cap_info, u32 cs, u32 dram_type);
> +void sdram_copy_to_reg(u32 *dest, const u32 *src, u32 n);
> +
>  #endif
> diff --git a/drivers/ram/rockchip/Kconfig b/drivers/ram/rockchip/Kconfig
> index 4f274e01b3..dcc06b3fd3 100644
> --- a/drivers/ram/rockchip/Kconfig
> +++ b/drivers/ram/rockchip/Kconfig
> @@ -5,6 +5,12 @@ config RAM_ROCKCHIP
>         help
>           This enables support for ram drivers Rockchip SoCs.
>
> +config ROCKCHIP_SDRAM_COMMON
> +       bool "Enable rockchip sdram common driver"
> +       depends on TPL_RAM || SPL_RAM
> +       help
> +         This enable sdram common driver
> +
>  if RAM_ROCKCHIP
>
>  config RAM_ROCKCHIP_DEBUG
> diff --git a/drivers/ram/rockchip/Makefile b/drivers/ram/rockchip/Makefile
> index feb1f82d00..2f90f4088e 100644
> --- a/drivers/ram/rockchip/Makefile
> +++ b/drivers/ram/rockchip/Makefile
> @@ -11,3 +11,4 @@ obj-$(CONFIG_ROCKCHIP_RK322X) = sdram_rk322x.o
>  obj-$(CONFIG_ROCKCHIP_RK3288) = sdram_rk3288.o
>  obj-$(CONFIG_ROCKCHIP_RK3328) = sdram_rk3328.o
>  obj-$(CONFIG_RAM_RK3399) += sdram_rk3399.o
> +obj-$(CONFIG_ROCKCHIP_SDRAM_COMMON) += sdram_common.o
> diff --git a/drivers/ram/rockchip/sdram_common.c b/drivers/ram/rockchip/sdram_common.c
> new file mode 100644
> index 0000000000..3f0e1d9a42
> --- /dev/null
> +++ b/drivers/ram/rockchip/sdram_common.c
> @@ -0,0 +1,285 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * (C) Copyright 2018 Rockchip Electronics Co., Ltd.
> + */
> +
> +#include <common.h>
> +#include <debug_uart.h>
> +#include <ram.h>
> +#include <asm/io.h>
> +#include <asm/arch-rockchip/sdram.h>
> +#include <asm/arch-rockchip/sdram_common.h>
> +
> +/* n: Unit bytes */
> +void sdram_copy_to_reg(u32 *dest, const u32 *src, u32 n)
> +{
> +       int i;
> +
> +       for (i = 0; i < n / sizeof(u32); i++) {
> +               writel(*src, dest);
> +               src++;
> +               dest++;
> +       }
> +}
> +
> +void sdram_org_config(struct sdram_cap_info *cap_info,
> +                     struct sdram_base_params *base,
> +                     u32 *p_os_reg2, u32 *p_os_reg3, u32 channel)
> +{
> +       *p_os_reg2 |= SYS_REG_ENC_DDRTYPE(base->dramtype);
> +       *p_os_reg2 |= SYS_REG_ENC_NUM_CH(base->num_channels);
> +
> +       *p_os_reg2 |= SYS_REG_ENC_ROW_3_4(cap_info->row_3_4, channel);
> +       *p_os_reg2 |= SYS_REG_ENC_CHINFO(channel);
> +       *p_os_reg2 |= SYS_REG_ENC_RANK(cap_info->rank, channel);
> +       *p_os_reg2 |= SYS_REG_ENC_COL(cap_info->col, channel);
> +       *p_os_reg2 |= SYS_REG_ENC_BK(cap_info->bk, channel);
> +       *p_os_reg2 |= SYS_REG_ENC_BW(cap_info->bw, channel);
> +       *p_os_reg2 |= SYS_REG_ENC_DBW(cap_info->dbw, channel);
> +
> +       SYS_REG_ENC_CS0_ROW(cap_info->cs0_row, *p_os_reg2, *p_os_reg3, channel);
> +       if (cap_info->cs1_row)
> +               SYS_REG_ENC_CS1_ROW(cap_info->cs1_row, *p_os_reg2,
> +                                   *p_os_reg3, channel);
> +       *p_os_reg3 |= SYS_REG_ENC_CS1_COL(cap_info->col, channel);
> +       *p_os_reg3 |= SYS_REG_ENC_VERSION(DDR_SYS_REG_VERSION);
> +}
> +
> +int sdram_detect_bw(struct sdram_cap_info *cap_info)
> +{
> +       return 0;
> +}
> +
> +int sdram_detect_cs(struct sdram_cap_info *cap_info)
> +{
> +       return 0;
> +}
> +
> +int sdram_detect_col(struct sdram_cap_info *cap_info,
> +                    u32 coltmp)
> +{
> +       void __iomem *test_addr;
> +       u32 col;
> +       u32 bw = cap_info->bw;
> +
> +       for (col = coltmp; col >= 9; col -= 1) {
> +               writel(0, CONFIG_SYS_SDRAM_BASE);
> +               test_addr = (void __iomem *)(CONFIG_SYS_SDRAM_BASE +
> +                               (1ul << (col + bw - 1ul)));
> +               writel(PATTERN, test_addr);
> +               if ((readl(test_addr) == PATTERN) &&
> +                   (readl(CONFIG_SYS_SDRAM_BASE) == 0))
> +                       break;
> +       }
> +       if (col == 8) {
> +               printascii("col error\n");
> +               return -1;
> +       }
> +
> +       cap_info->col = col;
> +
> +       return 0;
> +}
> +
> +int sdram_detect_bank(struct sdram_cap_info *cap_info,
> +                     u32 coltmp, u32 bktmp)
> +{
> +       void __iomem *test_addr;
> +       u32 bk;
> +       u32 bw = cap_info->bw;
> +
> +       test_addr = (void __iomem *)(CONFIG_SYS_SDRAM_BASE +
> +                       (1ul << (coltmp + bktmp + bw - 1ul)));
> +       writel(0, CONFIG_SYS_SDRAM_BASE);
> +       writel(PATTERN, test_addr);
> +       if ((readl(test_addr) == PATTERN) &&
> +           (readl(CONFIG_SYS_SDRAM_BASE) == 0))
> +               bk = 3;
> +       else
> +               bk = 2;
> +
> +       cap_info->bk = bk;
> +
> +       return 0;
> +}
> +
> +/* detect bg for ddr4 */
> +int sdram_detect_bg(struct sdram_cap_info *cap_info,
> +                   u32 coltmp)
> +{
> +       void __iomem *test_addr;
> +       u32 dbw;
> +       u32 bw = cap_info->bw;
> +
> +       test_addr = (void __iomem *)(CONFIG_SYS_SDRAM_BASE +
> +                       (1ul << (coltmp + bw + 1ul)));
> +       writel(0, CONFIG_SYS_SDRAM_BASE);
> +       writel(PATTERN, test_addr);
> +       if ((readl(test_addr) == PATTERN) &&
> +           (readl(CONFIG_SYS_SDRAM_BASE) == 0))
> +               dbw = 0;
> +       else
> +               dbw = 1;
> +
> +       cap_info->dbw = dbw;
> +
> +       return 0;
> +}
> +
> +/* detect dbw for ddr3,lpddr2,lpddr3,lpddr4 */
> +int sdram_detect_dbw(struct sdram_cap_info *cap_info, u32 dram_type)
> +{
> +       u32 row, col, bk, bw, cs_cap, cs;
> +       u32 die_bw_0 = 0, die_bw_1 = 0;
> +
> +       if (dram_type == DDR3 || dram_type == LPDDR4) {
> +               cap_info->dbw = 1;
> +       } else if (dram_type == LPDDR3 || dram_type == LPDDR2) {
> +               row = cap_info->cs0_row;
> +               col = cap_info->col;
> +               bk = cap_info->bk;
> +               cs = cap_info->rank;
> +               bw = cap_info->bw;
> +               cs_cap = (1 << (row + col + bk + bw - 20));
> +               if (bw == 2) {
> +                       if (cs_cap <= 0x2000000) /* 256Mb */
> +                               die_bw_0 = (col < 9) ? 2 : 1;
> +                       else if (cs_cap <= 0x10000000) /* 2Gb */
> +                               die_bw_0 = (col < 10) ? 2 : 1;
> +                       else if (cs_cap <= 0x40000000) /* 8Gb */
> +                               die_bw_0 = (col < 11) ? 2 : 1;
> +                       else
> +                               die_bw_0 = (col < 12) ? 2 : 1;
> +                       if (cs > 1) {
> +                               row = cap_info->cs1_row;
> +                               cs_cap = (1 << (row + col + bk + bw - 20));
> +                               if (cs_cap <= 0x2000000) /* 256Mb */
> +                                       die_bw_0 = (col < 9) ? 2 : 1;
> +                               else if (cs_cap <= 0x10000000) /* 2Gb */
> +                                       die_bw_0 = (col < 10) ? 2 : 1;
> +                               else if (cs_cap <= 0x40000000) /* 8Gb */
> +                                       die_bw_0 = (col < 11) ? 2 : 1;
> +                               else
> +                                       die_bw_0 = (col < 12) ? 2 : 1;
> +                       }
> +               } else {
> +                       die_bw_1 = 1;
> +                       die_bw_0 = 1;
> +               }
> +               cap_info->dbw = (die_bw_0 > die_bw_1) ? die_bw_0 : die_bw_1;
> +       }
> +
> +       return 0;
> +}
> +
> +int sdram_detect_row(struct sdram_cap_info *cap_info,
> +                    u32 coltmp, u32 bktmp, u32 rowtmp)
> +{
> +       u32 row;
> +       u32 bw = cap_info->bw;
> +       void __iomem *test_addr;
> +
> +       for (row = rowtmp; row > 12; row--) {
> +               writel(0, CONFIG_SYS_SDRAM_BASE);
> +               test_addr = (void __iomem *)(CONFIG_SYS_SDRAM_BASE +
> +                               (1ul << (row + bktmp + coltmp + bw - 1ul)));
> +               writel(PATTERN, test_addr);
> +               if ((readl(test_addr) == PATTERN) &&
> +                   (readl(CONFIG_SYS_SDRAM_BASE) == 0))
> +                       break;
> +       }
> +       if (row == 12) {
> +               printascii("row error");
> +               return -1;
> +       }
> +
> +       cap_info->cs0_row = row;
> +
> +       return 0;
> +}
> +
> +int sdram_detect_row_3_4(struct sdram_cap_info *cap_info,
> +                        u32 coltmp, u32 bktmp)
> +{
> +       u32 row_3_4;
> +       u32 bw = cap_info->bw;
> +       u32 row = cap_info->cs0_row;
> +       void __iomem *test_addr, *test_addr1;
> +
> +       test_addr = CONFIG_SYS_SDRAM_BASE;
> +       test_addr1 = (void __iomem *)(CONFIG_SYS_SDRAM_BASE +
> +                       (0x3ul << (row + bktmp + coltmp + bw - 1ul - 1ul)));
> +
> +       writel(0, test_addr);
> +       writel(PATTERN, test_addr1);
> +       if ((readl(test_addr) == 0) && (readl(test_addr1) == PATTERN))
> +               row_3_4 = 0;
> +       else
> +               row_3_4 = 1;
> +
> +       cap_info->row_3_4 = row_3_4;
> +
> +       return 0;
> +}
> +
> +int sdram_detect_high_row(struct sdram_cap_info *cap_info)
> +{
> +       cap_info->cs0_high16bit_row = cap_info->cs0_row;
> +       cap_info->cs1_high16bit_row = cap_info->cs1_row;
> +
> +       return 0;
> +}
> +
> +int sdram_detect_cs1_row(struct sdram_cap_info *cap_info, u32 dram_type)
> +{
> +       void __iomem *test_addr;
> +       u32 row = 0, bktmp, coltmp, bw;
> +       ulong cs0_cap;
> +       u32 byte_mask;
> +
> +       if (cap_info->rank == 2) {
> +               cs0_cap = sdram_get_cs_cap(cap_info, 0, dram_type);
> +
> +               if (dram_type == DDR4) {
> +                       if (cap_info->dbw == 0)
> +                               bktmp = cap_info->bk + 2;
> +                       else
> +                               bktmp = cap_info->bk + 1;
> +               } else {
> +                       bktmp = cap_info->bk;
> +               }
> +               bw = cap_info->bw;
> +               coltmp = cap_info->col;
> +
> +               /*
> +                * because px30 support axi split,min bandwidth
> +                * is 8bit. if cs0 is 32bit, cs1 may 32bit or 16bit
> +                * so we check low 16bit data when detect cs1 row.
> +                * if cs0 is 16bit/8bit, we check low 8bit data.
> +                */
> +               if (bw == 2)
> +                       byte_mask = 0xFFFF;
> +               else
> +                       byte_mask = 0xFF;
> +
> +               /* detect cs1 row */
> +               for (row = cap_info->cs0_row; row > 12; row--) {
> +                       test_addr = (void __iomem *)(CONFIG_SYS_SDRAM_BASE +
> +                                   cs0_cap +
> +                                   (1ul << (row + bktmp + coltmp + bw - 1ul)));
> +                       writel(0, CONFIG_SYS_SDRAM_BASE + cs0_cap);
> +                       writel(PATTERN, test_addr);
> +
> +                       if (((readl(test_addr) & byte_mask) ==
> +                            (PATTERN & byte_mask)) &&
> +                           ((readl(CONFIG_SYS_SDRAM_BASE + cs0_cap) &
> +                             byte_mask) == 0)) {
> +                               break;
> +                       }
> +               }
> +       }
> +
> +       cap_info->cs1_row = row;

I have seen the determined row value is 14 instead of 15 for LPDDR3
parameter. I did fixed cs value to 1 on sdram_get_cs_cap and I have
even tried of creating parameters from the ddr bin from rkbin but
either way result seems same.

Each channel has 1.5GiB total 3GiB memory.

Any idea? each


More information about the U-Boot mailing list