The current version breaks console output for the secure partition on psoc64 with IAR or ARMCLANG. Appears to work with GNUARM.
It appears that this commit is the culprit: fce78aef Platform: Duplicate the tfm_hal_platform_init
The console starts out good, then during tests only garbage is seen.
Linker script issues?
/Thomas
Looks like it’s linking tfm_hal_platform_init() from platform/ext/common/tfm_platform.c rather than from platform/ext/target/cypress/psoc64/tfm_hal_platform.c.
Chris
From: TF-M tf-m-bounces@lists.trustedfirmware.org On Behalf Of Thomas Törnblom via TF-M Sent: Friday, October 29, 2021 7:41 AM To: tf-m@lists.trustedfirmware.org Subject: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Caution: This e-mail originated outside Infineon Technologies. Do not click on links or open attachments unless you validate it is safehttps://intranet-content.infineon.com/explore/aboutinfineon/rules/informationsecurity/ug/SocialEngineering/Pages/SocialEngineeringElements_en.aspx.
The current version breaks console output for the secure partition on psoc64 with IAR or ARMCLANG. Appears to work with GNUARM.
It appears that this commit is the culprit: fce78aef Platform: Duplicate the tfm_hal_platform_init
The console starts out good, then during tests only garbage is seen.
Linker script issues?
/Thomas --
Thomas Törnblom, Product Engineer IAR Systems AB Box 23051, Strandbodgatan 1 SE-750 23 Uppsala, SWEDEN Mobile: +46 76 180 17 80 Fax: +46 18 16 78 01 E-mail: thomas.tornblom@iar.commailto:thomas.tornblom@iar.com Website: www.iar.comhttp://www.iar.com Twitter: www.twitter.com/iarsystemshttp://www.twitter.com/iarsystems
Thanks for reporting this, would resolve it ASAP from the next Monday.
/Ken
From: TF-M tf-m-bounces@lists.trustedfirmware.org On Behalf Of chris.brand--- via TF-M Sent: Saturday, October 30, 2021 2:52 AM To: tf-m@lists.trustedfirmware.org Subject: Re: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Looks like it’s linking tfm_hal_platform_init() from platform/ext/common/tfm_platform.c rather than from platform/ext/target/cypress/psoc64/tfm_hal_platform.c.
Chris
From: TF-M <tf-m-bounces@lists.trustedfirmware.orgmailto:tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Thomas Törnblom via TF-M Sent: Friday, October 29, 2021 7:41 AM To: tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Subject: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Caution: This e-mail originated outside Infineon Technologies. Do not click on links or open attachments unless you validate it is safehttps://intranet-content.infineon.com/explore/aboutinfineon/rules/informationsecurity/ug/SocialEngineering/Pages/SocialEngineeringElements_en.aspx.
The current version breaks console output for the secure partition on psoc64 with IAR or ARMCLANG. Appears to work with GNUARM.
It appears that this commit is the culprit: fce78aef Platform: Duplicate the tfm_hal_platform_init
The console starts out good, then during tests only garbage is seen.
Linker script issues?
/Thomas --
Thomas Törnblom, Product Engineer IAR Systems AB Box 23051, Strandbodgatan 1 SE-750 23 Uppsala, SWEDEN Mobile: +46 76 180 17 80 Fax: +46 18 16 78 01 E-mail: thomas.tornblom@iar.commailto:thomas.tornblom@iar.com Website: www.iar.comhttp://www.iar.com Twitter: www.twitter.com/iarsystemshttp://www.twitter.com/iarsystems
Hi Thomas, Chris,
I think it is the linking issue. Not sure why.
The last code base should have no this issue as this patchhttps://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/11407/12/platform/ext/common/tfm_platform.c#b13 (merged) has removed the common tfm_hal_platform_init(). Could you rebase to the latest code?
Best Regards, Kevin
From: TF-M tf-m-bounces@lists.trustedfirmware.org On Behalf Of Ken Liu via TF-M Sent: Saturday, October 30, 2021 5:28 PM To: tf-m@lists.trustedfirmware.org Cc: nd nd@arm.com Subject: Re: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Thanks for reporting this, would resolve it ASAP from the next Monday.
/Ken
From: TF-M <tf-m-bounces@lists.trustedfirmware.orgmailto:tf-m-bounces@lists.trustedfirmware.org> On Behalf Of chris.brand--- via TF-M Sent: Saturday, October 30, 2021 2:52 AM To: tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Subject: Re: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Looks like it’s linking tfm_hal_platform_init() from platform/ext/common/tfm_platform.c rather than from platform/ext/target/cypress/psoc64/tfm_hal_platform.c.
Chris
From: TF-M <tf-m-bounces@lists.trustedfirmware.orgmailto:tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Thomas Törnblom via TF-M Sent: Friday, October 29, 2021 7:41 AM To: tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Subject: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Caution: This e-mail originated outside Infineon Technologies. Do not click on links or open attachments unless you validate it is safehttps://intranet-content.infineon.com/explore/aboutinfineon/rules/informationsecurity/ug/SocialEngineering/Pages/SocialEngineeringElements_en.aspx.
The current version breaks console output for the secure partition on psoc64 with IAR or ARMCLANG. Appears to work with GNUARM.
It appears that this commit is the culprit: fce78aef Platform: Duplicate the tfm_hal_platform_init
The console starts out good, then during tests only garbage is seen.
Linker script issues?
/Thomas --
Thomas Törnblom, Product Engineer IAR Systems AB Box 23051, Strandbodgatan 1 SE-750 23 Uppsala, SWEDEN Mobile: +46 76 180 17 80 Fax: +46 18 16 78 01 E-mail: thomas.tornblom@iar.commailto:thomas.tornblom@iar.com Website: www.iar.comhttp://www.iar.com Twitter: www.twitter.com/iarsystemshttp://www.twitter.com/iarsystems
Thanks Kevin,
Works now.
Cheers, /Thomas
Den 2021-11-01 kl. 04:15, skrev Kevin Peng:
Hi Thomas, Chris,
I think it is the linking issue.
Not sure why.
The last code base should have no this issue as this patch https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/11407/12/platform/ext/common/tfm_platform.c#b13 (merged) has removed the common tfm_hal_platform_init().
Could you rebase to the latest code?
Best Regards,
Kevin
*From:* TF-M tf-m-bounces@lists.trustedfirmware.org *On Behalf Of *Ken Liu via TF-M *Sent:* Saturday, October 30, 2021 5:28 PM *To:* tf-m@lists.trustedfirmware.org *Cc:* nd nd@arm.com *Subject:* Re: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Thanks for reporting this, would resolve it ASAP from the next Monday.
/Ken
*From:* TF-M tf-m-bounces@lists.trustedfirmware.org *On Behalf Of *chris.brand--- via TF-M *Sent:* Saturday, October 30, 2021 2:52 AM *To:* tf-m@lists.trustedfirmware.org *Subject:* Re: [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
Looks like it’s linking tfm_hal_platform_init() from platform/ext/common/tfm_platform.c rather than from platform/ext/target/cypress/psoc64/tfm_hal_platform.c.
Chris
*From:* TF-M tf-m-bounces@lists.trustedfirmware.org *On Behalf Of *Thomas Törnblom via TF-M *Sent:* Friday, October 29, 2021 7:41 AM *To:* tf-m@lists.trustedfirmware.org *Subject:* [TF-M] fce78aef breaks psoc64 console output with IARARM and ARMCLANG toolchain
*Caution*: This e-mail originated outside Infineon Technologies. Do not click on links or open attachments unless you validate it is safe https://intranet-content.infineon.com/explore/aboutinfineon/rules/informationsecurity/ug/SocialEngineering/Pages/SocialEngineeringElements_en.aspx.
The current version breaks console output for the secure partition on psoc64 with IAR or ARMCLANG. Appears to work with GNUARM.
It appears that this commit is the culprit: fce78aef Platform: Duplicate the tfm_hal_platform_init
The console starts out good, then during tests only garbage is seen.
Linker script issues?
/Thomas
--
*Thomas Törnblom*, /Product Engineer/ IAR Systems AB Box 23051, Strandbodgatan 1 SE-750 23 Uppsala, SWEDEN Mobile: +46 76 180 17 80 Fax: +46 18 16 78 01 E-mail: thomas.tornblom@iar.com mailto:thomas.tornblom@iar.comWebsite: www.iar.com http://www.iar.com Twitter: www.twitter.com/iarsystems http://www.twitter.com/iarsystems
tf-m@lists.trustedfirmware.org