| Message ID | 1572442713-26353-6-git-send-email-fabien.dessenne@st.com |
|---|---|
| State | Accepted |
| Commit | 790d5b367025921c1c520eba280b1bd4313a0aa0 |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | remoteproc: add elf resource table loader | expand |
Hi Fabien, > From: Fabien DESSENNE <fabien.dessenne@st.com> > Sent: mercredi 30 octobre 2019 14:39 > > Since the coprocessor state is tracked in a backup register, there is no more need > for tracking it in an environment variable : remove it. > > Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com> Acked-by: Patrick Delaunay <patrick.delaunay@st.com> Thanks Patrick
On Wed, Oct 30, 2019 at 02:38:32PM +0100, Fabien Dessenne wrote: > Since the coprocessor state is tracked in a backup register, there is > no more need for tracking it in an environment variable : remove it. > > Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com> > Acked-by: Patrick Delaunay <patrick.delaunay@st.com> Applied to u-boot/master, thanks!
diff --git a/board/st/stm32mp1/stm32mp1.c b/board/st/stm32mp1/stm32mp1.c index 126af30..ea9afb6 100644 --- a/board/st/stm32mp1/stm32mp1.c +++ b/board/st/stm32mp1/stm32mp1.c @@ -935,10 +935,8 @@ static void board_copro_image_process(ulong fw_image, size_t fw_size) printf("Load Remote Processor %d with data@addr=0x%08lx %u bytes:%s\n", id, fw_image, fw_size, ret ? " Failed!" : " Success!"); - if (!ret) { + if (!ret) rproc_start(id); - env_set("copro_state", "booted"); - } } U_BOOT_FIT_LOADABLE_HANDLER(IH_TYPE_COPRO, board_copro_image_process);
Since the coprocessor state is tracked in a backup register, there is no more need for tracking it in an environment variable : remove it. Signed-off-by: Fabien Dessenne <fabien.dessenne@st.com> --- board/st/stm32mp1/stm32mp1.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)