* [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm
@ 2022-05-12 13:46 Aparna M
2022-05-12 13:57 ` Devarsh Thakkar
2022-05-12 15:31 ` Ryan Eatmon
0 siblings, 2 replies; 6+ messages in thread
From: Aparna M @ 2022-05-12 13:46 UTC (permalink / raw)
To: meta-ti, praneeth, denis, reatmon; +Cc: devarsht, s-adivi
Add DM firmware ipc_echo_testb_mcu1_0_release_strip.xer5f to deploy
directory for am62xx-evm. This is required for building tispl.bin.
Signed-off-by: Aparna M <a-m1@ti.com>
---
recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 1 +
1 file changed, 1 insertion(+)
diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
index 099473b4..bb88633a 100644
--- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
+++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
@@ -186,6 +186,7 @@ do_deploy_am62xx-evm-k3r5() {
ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_SYMLINK}
ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_BINARY}
install -m 644 ${SYSFW_TISCI} ${DEPLOYDIR}/
+ install -m 644 ${S}/ti-dm/am62xx/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}/
}
addtask deploy before do_build after do_compile
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm
2022-05-12 13:46 [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm Aparna M
@ 2022-05-12 13:57 ` Devarsh Thakkar
2022-05-12 14:05 ` Nishanth Menon
2022-05-12 15:31 ` Ryan Eatmon
1 sibling, 1 reply; 6+ messages in thread
From: Devarsh Thakkar @ 2022-05-12 13:57 UTC (permalink / raw)
To: Aparna M, meta-ti, praneeth, denis, reatmon; +Cc: s-adivi
On 12/05/22 19:16, Aparna M wrote:
> Add DM firmware ipc_echo_testb_mcu1_0_release_strip.xer5f to deploy
> directory for am62xx-evm. This is required for building tispl.bin.
>
> Signed-off-by: Aparna M <a-m1@ti.com>
> ---
> recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> index 099473b4..bb88633a 100644
> --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> @@ -186,6 +186,7 @@ do_deploy_am62xx-evm-k3r5() {
> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_SYMLINK}
> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_BINARY}
> install -m 644 ${SYSFW_TISCI} ${DEPLOYDIR}/
> + install -m 644 ${S}/ti-dm/am62xx/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}/
What is SYSFW_TISCI being used for AM62 ? Any existing macros can't be
used ?
Also is it changing indentation w.r.t previous line ?
> }
>
> addtask deploy before do_build after do_compile
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm
2022-05-12 13:57 ` Devarsh Thakkar
@ 2022-05-12 14:05 ` Nishanth Menon
2022-05-13 8:57 ` M, Aparna
2022-05-13 10:02 ` M, Aparna
0 siblings, 2 replies; 6+ messages in thread
From: Nishanth Menon @ 2022-05-12 14:05 UTC (permalink / raw)
To: devarsht; +Cc: Aparna M, meta-ti, praneeth, denis, reatmon, s-adivi
On 19:27-20220512, Devarsh Thakkar via lists.yoctoproject.org wrote:
>
> On 12/05/22 19:16, Aparna M wrote:
> > Add DM firmware ipc_echo_testb_mcu1_0_release_strip.xer5f to deploy
> > directory for am62xx-evm. This is required for building tispl.bin.
> >
> > Signed-off-by: Aparna M <a-m1@ti.com>
> > ---
> > recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> > index 099473b4..bb88633a 100644
> > --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> > +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> > @@ -186,6 +186,7 @@ do_deploy_am62xx-evm-k3r5() {
> > ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_SYMLINK}
> > ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_BINARY}
> > install -m 644 ${SYSFW_TISCI} ${DEPLOYDIR}/
> > + install -m 644 ${S}/ti-dm/am62xx/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}/
>
> What is SYSFW_TISCI being used for AM62 ? Any existing macros can't be used
> ?
>
> Also is it changing indentation w.r.t previous line ?
This is the DM firmware. yes, it is already packaged as of tispl build, so
what is missing here?
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm
2022-05-12 14:05 ` Nishanth Menon
@ 2022-05-13 8:57 ` M, Aparna
2022-05-13 10:02 ` M, Aparna
1 sibling, 0 replies; 6+ messages in thread
From: M, Aparna @ 2022-05-13 8:57 UTC (permalink / raw)
To: Menon, Nishanth, Thakkar, Devarsh, Eatmon, Ryan
Cc: M, Aparna, meta-ti@lists.yoctoproject.org, Bajjuri, Praneeth,
denis@denix.org, Adivi, Sai Sree Kartheek
On 12/05/22 19:35, Nishanth Menon wrote:
> On 19:27-20220512, Devarsh Thakkar via lists.yoctoproject.org wrote:
>> On 12/05/22 19:16, Aparna M wrote:
>>> Add DM firmware ipc_echo_testb_mcu1_0_release_strip.xer5f to deploy
>>> directory for am62xx-evm. This is required for building tispl.bin.
>>>
>>> Signed-off-by: Aparna M <a-m1@ti.com>
>>> ---
>>> recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> index 099473b4..bb88633a 100644
>>> --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> @@ -186,6 +186,7 @@ do_deploy_am62xx-evm-k3r5() {
>>> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_SYMLINK}
>>> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_BINARY}
>>> install -m 644 ${SYSFW_TISCI} ${DEPLOYDIR}/
>>> + install -m 644 ${S}/ti-dm/am62xx/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}/
>> What is SYSFW_TISCI being used for AM62 ? Any existing macros can't be used
>> ?
>>
>> Also is it changing indentation w.r.t previous line ?
>
> This is the DM firmware. yes, it is already packaged as of tispl build, so
> what is missing here?
The DM firmware is required in the SDK for toplevel makefile build of
u-boot for am62xx. The flow is to add it to deploy and then to
prebuilt-images folder of SDK.
Sent a new patch for the same as per Ryan's suggestion to use
ti-rtos-firmware recipe for this -
https://lists.yoctoproject.org/g/meta-ti/topic/dunfell_patch/91076133
--
Aparna
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm
2022-05-12 14:05 ` Nishanth Menon
2022-05-13 8:57 ` M, Aparna
@ 2022-05-13 10:02 ` M, Aparna
1 sibling, 0 replies; 6+ messages in thread
From: M, Aparna @ 2022-05-13 10:02 UTC (permalink / raw)
To: Menon, Nishanth, Thakkar, Devarsh, Eatmon, Ryan
Cc: meta-ti@lists.yoctoproject.org, Bajjuri, Praneeth,
denis@denix.org, Adivi, Sai Sree Kartheek
On 12/05/22 19:35, Nishanth Menon wrote:
> On 19:27-20220512, Devarsh Thakkar via lists.yoctoproject.org wrote:
>> On 12/05/22 19:16, Aparna M wrote:
>>> Add DM firmware ipc_echo_testb_mcu1_0_release_strip.xer5f to deploy
>>> directory for am62xx-evm. This is required for building tispl.bin.
>>>
>>> Signed-off-by: Aparna M <a-m1@ti.com>
>>> ---
>>> recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> index 099473b4..bb88633a 100644
>>> --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
>>> @@ -186,6 +186,7 @@ do_deploy_am62xx-evm-k3r5() {
>>> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_SYMLINK}
>>> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_BINARY}
>>> install -m 644 ${SYSFW_TISCI} ${DEPLOYDIR}/
>>> + install -m 644 ${S}/ti-dm/am62xx/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}/
>> What is SYSFW_TISCI being used for AM62 ? Any existing macros can't be used
>> ?
>>
>> Also is it changing indentation w.r.t previous line ?
>
> This is the DM firmware. yes, it is already packaged as of tispl build, so
> what is missing here?
The DM firmware is required in the SDK for toplevel makefile build of
u-boot for am62xx. The flow is to add it to deploy and then to
prebuilt-images folder of SDK.
Sent a new patch for the same as per Ryan's suggestion to use
ti-rtos-firmware recipe for this -
https://lists.yoctoproject.org/g/meta-ti/topic/dunfell_patch/91076133
--
Aparna
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm
2022-05-12 13:46 [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm Aparna M
2022-05-12 13:57 ` Devarsh Thakkar
@ 2022-05-12 15:31 ` Ryan Eatmon
1 sibling, 0 replies; 6+ messages in thread
From: Ryan Eatmon @ 2022-05-12 15:31 UTC (permalink / raw)
To: Aparna M, meta-ti, praneeth, denis; +Cc: devarsht, s-adivi
On 5/12/2022 8:46, Aparna M wrote:
> Add DM firmware ipc_echo_testb_mcu1_0_release_strip.xer5f to deploy
> directory for am62xx-evm. This is required for building tispl.bin.
>
> Signed-off-by: Aparna M <a-m1@ti.com>
> ---
> recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> index 099473b4..bb88633a 100644
> --- a/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> +++ b/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb
> @@ -186,6 +186,7 @@ do_deploy_am62xx-evm-k3r5() {
> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_SYMLINK}
> ln -sf ${UBOOT_IMAGE} ${DEPLOYDIR}/${UBOOT_BINARY}
> install -m 644 ${SYSFW_TISCI} ${DEPLOYDIR}/
> + install -m 644 ${S}/ti-dm/am62xx/ipc_echo_testb_mcu1_0_release_strip.xer5f ${DEPLOYDIR}/
> }
>
> addtask deploy before do_build after do_compile
NACK.
I think this is in the wrong recipe. This file is referenced for all of
the other platforms in:
recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
I think it would make more sense to put this change in that recipe for
consistency.
--
Ryan Eatmon reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc. - LCPD - MGTS
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-05-13 10:02 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-05-12 13:46 [meta-ti][dunfell][PATCH] ti-sci-fw: Add DM firmware to deploy directory for am62xx-evm Aparna M
2022-05-12 13:57 ` Devarsh Thakkar
2022-05-12 14:05 ` Nishanth Menon
2022-05-13 8:57 ` M, Aparna
2022-05-13 10:02 ` M, Aparna
2022-05-12 15:31 ` Ryan Eatmon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.