On Fri, Mar 05, 2021 at 03:06:08AM -0500, Denys Dmytriyenko wrote:
> On Fri, Mar 05, 2021 at 12:45:53AM -0600, Nishanth Menon wrote:
> > On 00:41-20210305, Nishanth Menon wrote:
> > > "evm" prefix messes up the pattern matchup for J7200 and falls back into
> > > default, as a result, we attempt to pull [1] instead of [2].
> > >
> > > [1]
> > > https://software-dl.ti.com/jacinto7/esd/processor-sdk-rtos-jacinto7/firmware/07_03_00_12/coresdk_rtos_j7200_07_03_00_12_firmware.tar.xz
> > > [2]
> > > https://software-dl.ti.com/jacinto7/esd/processor-sdk-rtos-jacinto7/firmware/07_03_00_10/coresdk_rtos_j7200_07_03_00_12_firmware.tar.xz
> >
> > ^^ above link should have been:
> > https://software-dl.ti.com/jacinto7/esd/processor-sdk-rtos-jacinto7/firmware/07_03_00_10/coresdk_rtos_j7200_07_03_00_10_firmware.tar.xz
> >
> > if you need me to send a v2 with fixups, i can.. or if that is the only
> > comment, then might be helpful to get a fixup while applying..
>
> I'm failing to see the issue here...
>
> Of course, using "_j7200" instead of "_j7200-evm" is more preferred to cover
Actually, that is wishful thinking - there's NO "j7200" SOC_FAMILY set up at
all! So this patch is completely incorrect, as there's no such override as
_j7200, only _j7200-evm. Something else is going on...
> other variants, like potential "j7200-hs-evm", but when building for
> j7200-evm
> should result in the same output. I would need to reproduce it - any another
> pointers?
>
>
> > > Fixes: 77cdfa9f6560 ("ti-rtos: Update metadata and versions for RTOS
> > > firmware")
> > > Signed-off-by: Nishanth Menon <[email protected]>
> > > ---
> > > recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
> > > b/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
> > > index ed62b876e4db..dffd51abda66 100644
> > > --- a/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
> > > +++ b/recipes-ti/ti-rtos-bin/ti-rtos-firmware.bb
> > > @@ -31,7 +31,7 @@ DEFAULT_RTOS_VERSION_DOT_am65xx = "07.03.00.10"
> > >
> > > DEFAULT_RTOS_SOC = "undefined"
> > > DEFAULT_RTOS_SOC_j7 = "j721e"
> > > -DEFAULT_RTOS_SOC_j7200-evm = "j7200"
> > > +DEFAULT_RTOS_SOC_j7200 = "j7200"
> > > DEFAULT_RTOS_SOC_am65xx = "am65xx"
> > > DEFAULT_RTOS_SOC_am64xx = "am64x"
> > >
> > > @@ -47,7 +47,7 @@ DEFAULT_FIRMWARE_URL_k3 =
> > > "${CORESDK_RTOS_WEBLINK}/${DEFAULT_FIRMWARE_FILE}"
> > >
> > > DEFAULT_FIRMWARE_SHA256SUM = "unknown"
> > > DEFAULT_FIRMWARE_SHA256SUM_j7 =
> > > "69cc16c30c895fe813002bd7bdb3d30e6ca622d86b008a6bf6e2a24b86550969"
> > > -DEFAULT_FIRMWARE_SHA256SUM_j7200-evm =
> > > "a16865d928b2b00bfca64e0c41b02dcfc0ac8ec1ce125fbfd3581a1c610847f8"
> > > +DEFAULT_FIRMWARE_SHA256SUM_j7200 =
> > > "a16865d928b2b00bfca64e0c41b02dcfc0ac8ec1ce125fbfd3581a1c610847f8"
> > > DEFAULT_FIRMWARE_SHA256SUM_am65xx =
> > > "aa121c10d3099dd61ef898717245ecee25b501170607c9a95afd741e05783a5d"
> > > DEFAULT_FIRMWARE_SHA256SUM_am64xx =
> > > "9e9efeef0b1dbb101de97e58cfda8bf052a697f2ea635fdf6ca5158fffdc212b"
> > >
> > > --
> > > 2.25.1.377.g2d2118b814c1
> > >
> >
> > --
> > Regards,
> > Nishanth Menon
> > Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5
> > 849D 1736 249D
> >
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13488):
https://lists.yoctoproject.org/g/meta-ti/message/13488
Mute This Topic: https://lists.yoctoproject.org/mt/81097409/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
