[PATCH 1/3] rockchip: efuse: add support for RK3288 non-secure efuse
Kever Yang
kever.yang at rock-chips.com
Fri Oct 30 11:21:48 CET 2020
Hi Jonas,
Could you help to format the coding style as required so that we
can merge this patch to the mainline?
Thanks,
- Kever
On 2020/10/14 上午4:21, Jonas Karlman wrote:
> From: Francis Fan <francis.fan at rock-chips.com>
>
> Extend rockchip efuse driver with support for RK3288 non-secure efuse.
>
> Signed-off-by: Francis Fan <francis.fan at rock-chips.com>
> Signed-off-by: Cody Xie <cody.xie at rock-chips.com>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
> ---
> drivers/misc/rockchip-efuse.c | 91 +++++++++++++++++++++++++++++++++--
> 1 file changed, 87 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/misc/rockchip-efuse.c b/drivers/misc/rockchip-efuse.c
> index 46ce6305fe..20423544d9 100644
> --- a/drivers/misc/rockchip-efuse.c
> +++ b/drivers/misc/rockchip-efuse.c
> @@ -27,6 +27,17 @@
> #define RK3399_STROBE BIT(1)
> #define RK3399_CSB BIT(0)
>
> +#define RK3288_A_SHIFT 6
> +#define RK3288_A_MASK 0x3ff
> +#define RK3288_NFUSES 32
> +#define RK3288_BYTES_PER_FUSE 1
> +#define RK3288_PGENB BIT(3)
> +#define RK3288_LOAD BIT(2)
> +#define RK3288_STROBE BIT(1)
> +#define RK3288_CSB BIT(0)
> +
> +typedef int (*EFUSE_READ)(struct udevice *dev, int offset, void *buf, int size);
> +
> struct rockchip_efuse_regs {
> u32 ctrl; /* 0x00 efuse control register */
> u32 dout; /* 0x04 efuse data out register */
> @@ -53,7 +64,7 @@ static int dump_efuses(struct cmd_tbl *cmdtp, int flag,
> */
>
> struct udevice *dev;
> - u8 fuses[128];
> + u8 fuses[128] = {0};
> int ret;
>
> /* retrieve the device */
> @@ -77,7 +88,7 @@ static int dump_efuses(struct cmd_tbl *cmdtp, int flag,
> }
>
> U_BOOT_CMD(
> - rk3399_dump_efuses, 1, 1, dump_efuses,
> + rockchip_dump_efuses, 1, 1, dump_efuses,
> "Dump the content of the efuses",
> ""
> );
> @@ -127,10 +138,59 @@ static int rockchip_rk3399_efuse_read(struct udevice *dev, int offset,
> return 0;
> }
>
> +static int rockchip_rk3288_efuse_read(struct udevice *dev, int offset,
> + void *buf, int size)
> +{
> + struct rockchip_efuse_platdata *plat = dev_get_platdata(dev);
> + struct rockchip_efuse_regs *efuse =
> + (struct rockchip_efuse_regs *)plat->base;
> + u8 *buffer = buf;
> + int max_size = RK3288_NFUSES * RK3288_BYTES_PER_FUSE;
> +
> + if (size > (max_size - offset))
> + size = max_size - offset;
> +
> + /* Switch to read mode */
> + writel(RK3288_LOAD | RK3288_PGENB, &efuse->ctrl);
> + udelay(1);
> +
> + while (size--) {
> + writel(readl(&efuse->ctrl) &
> + (~(RK3288_A_MASK << RK3288_A_SHIFT)),
> + &efuse->ctrl);
> + /* set addr */
> + writel(readl(&efuse->ctrl) |
> + ((offset++ & RK3288_A_MASK) << RK3288_A_SHIFT),
> + &efuse->ctrl);
> + udelay(1);
> + /* strobe low to high */
> + writel(readl(&efuse->ctrl) |
> + RK3288_STROBE, &efuse->ctrl);
> + ndelay(60);
> + /* read data */
> + *buffer++ = readl(&efuse->dout);
> + /* reset strobe to low */
> + writel(readl(&efuse->ctrl) &
> + (~RK3288_STROBE), &efuse->ctrl);
> + udelay(1);
> + }
> +
> + /* Switch to standby mode */
> + writel(RK3288_PGENB | RK3288_CSB, &efuse->ctrl);
> +
> + return 0;
> +}
> +
> static int rockchip_efuse_read(struct udevice *dev, int offset,
> void *buf, int size)
> {
> - return rockchip_rk3399_efuse_read(dev, offset, buf, size);
> + EFUSE_READ efuse_read = NULL;
> +
> + efuse_read = (EFUSE_READ)dev_get_driver_data(dev);
> + if (!efuse_read)
> + return -ENOSYS;
> +
> + return (*efuse_read)(dev, offset, buf, size);
> }
>
> static const struct misc_ops rockchip_efuse_ops = {
> @@ -146,7 +206,30 @@ static int rockchip_efuse_ofdata_to_platdata(struct udevice *dev)
> }
>
> static const struct udevice_id rockchip_efuse_ids[] = {
> - { .compatible = "rockchip,rk3399-efuse" },
> + {
> + .compatible = "rockchip,rk3066a-efuse",
> + .data = (ulong)&rockchip_rk3288_efuse_read,
> + },
> + {
> + .compatible = "rockchip,rk3188-efuse",
> + .data = (ulong)&rockchip_rk3288_efuse_read,
> + },
> + {
> + .compatible = "rockchip,rk3228-efuse",
> + .data = (ulong)&rockchip_rk3288_efuse_read,
> + },
> + {
> + .compatible = "rockchip,rk3288-efuse",
> + .data = (ulong)&rockchip_rk3288_efuse_read,
> + },
> + {
> + .compatible = "rockchip,rk3368-efuse",
> + .data = (ulong)&rockchip_rk3288_efuse_read,
> + },
> + {
> + .compatible = "rockchip,rk3399-efuse",
> + .data = (ulong)&rockchip_rk3399_efuse_read,
> + },
> {}
> };
>
More information about the U-Boot
mailing list