[U-Boot] [U-Boot,8/8] rockchip: rk3128: add sdram driver

Philipp Tomsich philipp.tomsich at theobroma-systems.com
Fri Oct 6 10:34:25 UTC 2017



On Wed, 27 Sep 2017, Kever Yang wrote:

> RK3128 support up to 2GB DDR3 sdram, one channel, 32bit data width.
>
> This patch is only used for U-Boot, but not for SPL which will
> comes later, maybe after we merge all the common code into a common
> file.
>
> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
> ---
>
> drivers/ram/rockchip/Makefile       |  1 +
> drivers/ram/rockchip/sdram_rk3128.c | 60 +++++++++++++++++++++++++++++++++++++
> 2 files changed, 61 insertions(+)
> create mode 100644 drivers/ram/rockchip/sdram_rk3128.c
>
> diff --git a/drivers/ram/rockchip/Makefile b/drivers/ram/rockchip/Makefile
> index 45b5fe7..1a1e557 100644
> --- a/drivers/ram/rockchip/Makefile
> +++ b/drivers/ram/rockchip/Makefile
> @@ -5,6 +5,7 @@
> #
>
> obj-$(CONFIG_ROCKCHIP_RK3368) = dmc-rk3368.o
> +obj-$(CONFIG_ROCKCHIP_RK3128) = sdram_rk3128.o
> obj-$(CONFIG_ROCKCHIP_RK3188) = sdram_rk3188.o
> obj-$(CONFIG_ROCKCHIP_RK322X) = sdram_rk322x.o
> obj-$(CONFIG_ROCKCHIP_RK3288) = sdram_rk3288.o
> diff --git a/drivers/ram/rockchip/sdram_rk3128.c b/drivers/ram/rockchip/sdram_rk3128.c
> new file mode 100644
> index 0000000..04ad2bb
> --- /dev/null
> +++ b/drivers/ram/rockchip/sdram_rk3128.c
> @@ -0,0 +1,60 @@
> +/*
> + * (C) Copyright 2017 Rockchip Electronics Co., Ltd.
> + *
> + * SPDX-License-Identifier:     GPL-2.0
> + */
> +
> +#include <common.h>
> +#include <dm.h>
> +#include <ram.h>
> +#include <syscon.h>
> +#include <asm/arch/clock.h>
> +#include <asm/arch/grf_rk3128.h>
> +#include <asm/arch/sdram_common.h>
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +struct dram_info {
> +	struct ram_info info;
> +	struct rk3128_grf *grf;
> +};
> +
> +static int rk3128_dmc_probe(struct udevice *dev)
> +{
> +	struct dram_info *priv = dev_get_priv(dev);
> +
> +	priv->grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF);
> +	printf("%s: grf=%p\n", __func__, priv->grf);
> +	priv->info.base = CONFIG_SYS_SDRAM_BASE;
> +	priv->info.size = rockchip_sdram_size(
> +				(phys_addr_t)&priv->grf->os_reg[1]);

Can we make this into a common driver for devices that are not supported 
by SPL yet (or for people that want to use the miniloader)?

To make this chip-independent, we just need to export the OS_REG via 
syscon or regmap... e.g allowing a regmap_read on the OS_REG region for 
each chip.

> +
> +	return 0;
> +}
> +
> +static int rk3128_dmc_get_info(struct udevice *dev, struct ram_info *info)
> +{
> +	struct dram_info *priv = dev_get_priv(dev);
> +
> +	*info = priv->info;
> +
> +	return 0;
> +}
> +
> +static struct ram_ops rk3128_dmc_ops = {
> +	.get_info = rk3128_dmc_get_info,
> +};
> +
> +
> +static const struct udevice_id rk3128_dmc_ids[] = {
> +	{ .compatible = "rockchip,rk3128-dmc" },
> +	{ }
> +};
> +
> +U_BOOT_DRIVER(dmc_rk3128) = {
> +	.name = "rockchip_rk3128_dmc",
> +	.id = UCLASS_RAM,
> +	.of_match = rk3128_dmc_ids,
> +	.ops = &rk3128_dmc_ops,
> +	.probe = rk3128_dmc_probe,
> +	.priv_auto_alloc_size = sizeof(struct dram_info),
> +};
>


More information about the U-Boot mailing list