Hi Xinyu,
Thanks for the patch, it FP stuff way cleaner.
Bohdan Hunko
Cypress Semiconductor Ukraine Engineer CSUKR CSS ICW SW FW Mobile: +38099 50 19 714 Bohdan.Hunko@infineon.commailto:Bohdan.Hunko@infineon.com
From: Xinyu Zhang Xinyu.Zhang@arm.com Sent: Monday, 6 June 2022 06:59 To: Hunko Bohdan (CSUKR CSS ICW SW FW) Bohdan.Hunko@infineon.com; tf-m@lists.trustedfirmware.org Cc: nd nd@arm.com Subject: RE: Usage of TFM_SYSTEM_FP cmake variable
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.
Hi Bohdan,
The only purpose of 'TFM_SYSTEM_FP' in ARMCLANG & IAR toolchains is to set compiler flags for the time being. To avoid confusion, I created a patch to deprecate it and added you to the reviewer. Any comments are welcome. 😊 https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/15383
Thanks, Xinyu
From: Bohdan.Hunko@infineon.commailto:Bohdan.Hunko@infineon.com <Bohdan.Hunko@infineon.commailto:Bohdan.Hunko@infineon.com> Sent: Friday, June 3, 2022 8:46 PM To: Xinyu Zhang <Xinyu.Zhang@arm.commailto:Xinyu.Zhang@arm.com>; tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Cc: nd <nd@arm.commailto:nd@arm.com> Subject: RE: Usage of TFM_SYSTEM_FP cmake variable
Hi,
As you said CONFIG_TFM_FP is used for GCC as FP is only supported for GCC for now.
But what is the purpose of TFM_SYSTEM_FP variable then?
Regards, Bohdan Hunko
Cypress Semiconductor Ukraine Engineer CSUKR CSS ICW SW FW Mobile: +38099 50 19 714 Bohdan.Hunko@infineon.commailto:Bohdan.Hunko@infineon.com
From: Xinyu Zhang <Xinyu.Zhang@arm.commailto:Xinyu.Zhang@arm.com> Sent: Monday, 30 May 2022 08:50 To: Hunko Bohdan (CSUKR CSS ICW SW FW) <Bohdan.Hunko@infineon.commailto:Bohdan.Hunko@infineon.com>; tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Cc: nd <nd@arm.commailto:nd@arm.com> Subject: RE: Usage of TFM_SYSTEM_FP cmake variable
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.
Hi Bohdan,
Currently FP is only supported by GCC toolchain. Detailed docs is available here: https://tf-m-user-guide.trustedfirmware.org/integration_guide/tfm_fpu_suppor...
BTW, patches are welcome if you have any ideas or interests. 😉
Thanks, Xinyu
From: Bohdan.Hunko--- via TF-M <tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org> Sent: Thursday, May 26, 2022 9:42 PM To: tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Subject: [TF-M] Usage of TFM_SYSTEM_FP cmake variable
Hi everyone,
I have noticed that GCC toolchain uses CONFIG_TFM_FP to determine FP setting while IAR and Clang use TFM_SYSTEM_FP cmake variable. I was not able to find any docs on this variable, and also there is no assignment of this variable in TFM source code (only read operation from this variable).
Is this intendent behavior?
Regards, Bohdan Hunko
Cypress Semiconductor Ukraine Engineer CSUKR CSS ICW SW FW Mobile: +38099 50 19 714 Bohdan.Hunko@infineon.commailto:Bohdan.Hunko@infineon.com