remoteproc: stm32: use reset for hold boot
Use the reset function to handle the hold boot bit in RCC with device tree handle. Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com> Change-Id: Id4df826efcbb8c6e22a7d5ad02cfc7459ef27345 Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/u-boot/+/173695 Reviewed-by: CITOOLS <smet-aci-reviews@lists.codex.cro.st.com> Reviewed-by: CIBUILD <smet-aci-builds@lists.codex.cro.st.com> Reviewed-by: Fabien DESSENNE <fabien.dessenne@st.com> Reviewed-by: Arnaud POULIQUEN <arnaud.pouliquen@st.com>
This commit is contained in:
@ -7,31 +7,19 @@
|
||||
#include <dm.h>
|
||||
#include <errno.h>
|
||||
#include <fdtdec.h>
|
||||
#include <regmap.h>
|
||||
#include <remoteproc.h>
|
||||
#include <reset.h>
|
||||
#include <syscon.h>
|
||||
#include <asm/io.h>
|
||||
#include <asm/arch/stm32mp1_smc.h>
|
||||
|
||||
#define RCC_GCR_HOLD_BOOT 0
|
||||
#define RCC_GCR_RELEASE_BOOT 1
|
||||
|
||||
/**
|
||||
* struct stm32_copro_privdata - power processor private data
|
||||
* @reset_ctl: reset controller handle
|
||||
* @hold_boot_regmap: regmap for remote processor reset hold boot
|
||||
* @hold_boot_offset: offset of the register controlling the hold boot setting
|
||||
* @hold_boot_mask: bitmask of the register for the hold boot field
|
||||
* @secured_soc: TZEN flag (register protection)
|
||||
* @hold_boot: hold boot controller handle
|
||||
* @rsc_table_addr: resource table address
|
||||
*/
|
||||
struct stm32_copro_privdata {
|
||||
struct reset_ctl reset_ctl;
|
||||
struct regmap *hold_boot_regmap;
|
||||
uint hold_boot_offset;
|
||||
uint hold_boot_mask;
|
||||
bool secured_soc;
|
||||
struct reset_ctl hold_boot;
|
||||
ulong rsc_table_addr;
|
||||
};
|
||||
|
||||
@ -43,100 +31,27 @@ struct stm32_copro_privdata {
|
||||
static int stm32_copro_probe(struct udevice *dev)
|
||||
{
|
||||
struct stm32_copro_privdata *priv;
|
||||
struct regmap *regmap;
|
||||
const fdt32_t *cell;
|
||||
uint tz_offset, tz_mask, tzen;
|
||||
int len, ret;
|
||||
int ret;
|
||||
|
||||
priv = dev_get_priv(dev);
|
||||
|
||||
regmap = syscon_regmap_lookup_by_phandle(dev, "st,syscfg-holdboot");
|
||||
if (IS_ERR(regmap)) {
|
||||
dev_err(dev, "unable to find holdboot regmap (%ld)\n",
|
||||
PTR_ERR(regmap));
|
||||
return PTR_ERR(regmap);
|
||||
}
|
||||
|
||||
cell = dev_read_prop(dev, "st,syscfg-holdboot", &len);
|
||||
if (len < 3 * sizeof(fdt32_t)) {
|
||||
dev_err(dev, "holdboot offset and mask not available\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
priv->hold_boot_regmap = regmap;
|
||||
priv->hold_boot_offset = fdtdec_get_number(cell + 1, 1);
|
||||
priv->hold_boot_mask = fdtdec_get_number(cell + 2, 1);
|
||||
|
||||
ret = reset_get_by_index(dev, 0, &priv->reset_ctl);
|
||||
ret = reset_get_by_name(dev, "mcu_rst", &priv->reset_ctl);
|
||||
if (ret) {
|
||||
dev_err(dev, "failed to get reset (%d)\n", ret);
|
||||
return ret;
|
||||
}
|
||||
|
||||
regmap = syscon_regmap_lookup_by_phandle(dev, "st,syscfg-tz");
|
||||
if (IS_ERR(regmap)) {
|
||||
dev_dbg(dev, "unable to find tz regmap (%ld)\n",
|
||||
PTR_ERR(regmap));
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
cell = dev_read_prop(dev, "st,syscfg-tz", &len);
|
||||
if (3 * sizeof(fdt32_t) - len > 0) {
|
||||
dev_dbg(dev, "tz offset and mask not available\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
tz_offset = fdtdec_get_number(cell + 1, 1);
|
||||
|
||||
tz_mask = fdtdec_get_number(cell + 2, 1);
|
||||
|
||||
ret = regmap_read(regmap, tz_offset, &tzen);
|
||||
ret = reset_get_by_name(dev, "hold_boot", &priv->hold_boot);
|
||||
if (ret) {
|
||||
dev_dbg(dev, "failed to read soc secure state\n");
|
||||
dev_err(dev, "failed to get hold boot (%d)\n", ret);
|
||||
return ret;
|
||||
}
|
||||
|
||||
priv->secured_soc = !!(tzen & tz_mask);
|
||||
|
||||
dev_dbg(dev, "probed\n");
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
/**
|
||||
* stm32_copro_set_hold_boot() - Hold boot bit management
|
||||
* @dev: corresponding STM32 remote processor device
|
||||
* @hold: hold boot value
|
||||
* @return 0 if all went ok, else corresponding -ve error
|
||||
*/
|
||||
static int stm32_copro_set_hold_boot(struct udevice *dev, bool hold)
|
||||
{
|
||||
struct stm32_copro_privdata *priv;
|
||||
uint val;
|
||||
int ret;
|
||||
|
||||
priv = dev_get_priv(dev);
|
||||
|
||||
val = hold ? RCC_GCR_HOLD_BOOT : RCC_GCR_RELEASE_BOOT;
|
||||
|
||||
if (priv->secured_soc) {
|
||||
return stm32_smc_exec(STM32_SMC_RCC, STM32_SMC_REG_WRITE,
|
||||
priv->hold_boot_offset, val);
|
||||
}
|
||||
|
||||
/*
|
||||
* Note: shall run an SMC call (STM32_SMC_RCC) if platform is secured.
|
||||
* To be updated when the code for this SMC service is available which
|
||||
* is not the case for the time being.
|
||||
*/
|
||||
ret = regmap_update_bits(priv->hold_boot_regmap, priv->hold_boot_offset,
|
||||
priv->hold_boot_mask, val);
|
||||
if (ret)
|
||||
dev_err(dev, "failed to set hold boot\n");
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
/**
|
||||
* stm32_copro_device_to_virt() - Convert device address to virtual address
|
||||
* @dev: corresponding STM32 remote processor device
|
||||
@ -180,9 +95,11 @@ static int stm32_copro_load(struct udevice *dev, ulong addr, ulong size)
|
||||
|
||||
priv = dev_get_priv(dev);
|
||||
|
||||
ret = stm32_copro_set_hold_boot(dev, true);
|
||||
if (ret)
|
||||
ret = reset_assert(&priv->hold_boot);
|
||||
if (ret) {
|
||||
dev_err(dev, "Unable to assert hold boot (ret=%d)\n", ret);
|
||||
return ret;
|
||||
}
|
||||
|
||||
ret = reset_assert(&priv->reset_ctl);
|
||||
if (ret) {
|
||||
@ -211,16 +128,20 @@ static int stm32_copro_start(struct udevice *dev)
|
||||
|
||||
priv = dev_get_priv(dev);
|
||||
|
||||
/* move hold boot from true to false start the copro */
|
||||
ret = stm32_copro_set_hold_boot(dev, false);
|
||||
if (ret)
|
||||
ret = reset_deassert(&priv->hold_boot);
|
||||
if (ret) {
|
||||
dev_err(dev, "Unable to deassert hold boot (ret=%d)\n", ret);
|
||||
return ret;
|
||||
}
|
||||
|
||||
/*
|
||||
* Once copro running, reset hold boot flag to avoid copro
|
||||
* rebooting autonomously
|
||||
*/
|
||||
ret = stm32_copro_set_hold_boot(dev, true);
|
||||
ret = reset_assert(&priv->hold_boot);
|
||||
if (ret)
|
||||
dev_err(dev, "Unable to assert hold boot (ret=%d)\n", ret);
|
||||
|
||||
writel(ret ? TAMP_COPRO_STATE_OFF : TAMP_COPRO_STATE_CRUN,
|
||||
TAMP_COPRO_STATE);
|
||||
if (!ret)
|
||||
@ -242,9 +163,11 @@ static int stm32_copro_reset(struct udevice *dev)
|
||||
|
||||
priv = dev_get_priv(dev);
|
||||
|
||||
ret = stm32_copro_set_hold_boot(dev, true);
|
||||
if (ret)
|
||||
ret = reset_assert(&priv->hold_boot);
|
||||
if (ret) {
|
||||
dev_err(dev, "Unable to assert hold boot (ret=%d)\n", ret);
|
||||
return ret;
|
||||
}
|
||||
|
||||
ret = reset_assert(&priv->reset_ctl);
|
||||
if (ret) {
|
||||
|
||||
Reference in New Issue
Block a user