Hi,
I'm looking at https://git.trustedfirmware.org/TF-M/trusted-firmware-m.git/commit/?id=df871... which added tfm_hal_memory_symbols.{c|h}, containing tfm_hal_sp_meta_start and tfm_hal_sp_meta_end. The former is used in backend_ipc.c, while the latter is unused. The commit message says "this is the first example of using defined symbols to get memory info" but that commit is over two years old now and there doesn't seem to be a second example. Is there still a plan to move in this direction? If so, can somebody outline what that involves?
Thanks,
Chris Brand
Cypress Semiconductor (Canada), Inc. An Infineon Technologies Company Sr Prin Software Engr CSCA CSS ICW SW PSW 1 Chris.Brand@infineon.commailto:Chris.Brand@infineon.com
International Place 13700 V6V 2X8 Richmond Canada
www.infineon.com<www.cypress.com> www.cypress.comhttp://www.cypress.com Discoverieshttp://www.infineon.com/discoveries Facebookhttp://www.facebook.com/infineon Twitterhttp://www.twitter.com/Infineon LinkedInhttp://www.linkedin.com/company/infineon-technologies
Part of your life. Part of tomorrow.
NOTICE: The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material of Infineon Technologies AG and its affiliated entities which is for the exclusive use of the individual designated above as the recipient. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon, this information by persons or entities other than the intended recipient is prohibited. If you received this in error, please contact immediately the sender by returning e-mail and delete the material from any computer. If you are not the specified recipient, you are hereby notified that all disclosure, reproduction, distribution or action taken on the basis of this message is prohibited.
Hi Chris,
This file was created for importing platform defined symbols, and guess there would be other symbols to be referenced, but unfortunately till now the 'metadata' (will be renamed into 'local storage pointer') is the only item in this file.
Actually the rom loader (rom_loader.c) references platform defined symbols as well, so thinking if we could gather them together into one unified file (data_symbols.c and data_import.h) cfor importing platform-defined symbols. As we are MCU software so referencing symbols directly instead of getter function would be more straightforward. But since it is another bridge between SPM and platform so at least we need to well introduce it in the HAL docs.
This idea is not solid planned so what you think of unifying the external symbol referencing mechanism?
BR
/Ken
From: Chris.Brand--- via TF-M tf-m@lists.trustedfirmware.org Sent: Tuesday, February 28, 2023 7:42 AM To: tf-m@lists.trustedfirmware.org Subject: [TF-M] tfm_hal_memory_symbols
Hi,
I'm looking at https://git.trustedfirmware.org/TF-M/trusted-firmware-m.git/commit/?id=df871... which added tfm_hal_memory_symbols.{c|h}, containing tfm_hal_sp_meta_start and tfm_hal_sp_meta_end. The former is used in backend_ipc.c, while the latter is unused. The commit message says "this is the first example of using defined symbols to get memory info" but that commit is over two years old now and there doesn't seem to be a second example. Is there still a plan to move in this direction? If so, can somebody outline what that involves?
Thanks,
Chris Brand
Cypress Semiconductor (Canada), Inc. An Infineon Technologies Company Sr Prin Software Engr CSCA CSS ICW SW PSW 1 Chris.Brand@infineon.commailto:Chris.Brand@infineon.com
International Place 13700 V6V 2X8 Richmond Canada
www.infineon.com<www.cypress.com> www.cypress.comhttp://www.cypress.com Discoverieshttp://www.infineon.com/discoveries Facebookhttp://www.facebook.com/infineon Twitterhttp://www.twitter.com/Infineon LinkedInhttp://www.linkedin.com/company/infineon-technologies
Part of your life. Part of tomorrow.
NOTICE: The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material of Infineon Technologies AG and its affiliated entities which is for the exclusive use of the individual designated above as the recipient. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon, this information by persons or entities other than the intended recipient is prohibited. If you received this in error, please contact immediately the sender by returning e-mail and delete the material from any computer. If you are not the specified recipient, you are hereby notified that all disclosure, reproduction, distribution or action taken on the basis of this message is prohibited.
Hi,
Created one patch for gathering memory symbols: https://review.trustedfirmware.org/c/TF-M/trusted-firmware-m/+/19788
Feel free to comment.
/Ken
From: Ken Liu Sent: Tuesday, February 28, 2023 1:46 PM To: tf-m@lists.trustedfirmware.org Cc: nd nd@arm.com Subject: RE: tfm_hal_memory_symbols
Hi Chris,
This file was created for importing platform defined symbols, and guess there would be other symbols to be referenced, but unfortunately till now the 'metadata' (will be renamed into 'local storage pointer') is the only item in this file.
Actually the rom loader (rom_loader.c) references platform defined symbols as well, so thinking if we could gather them together into one unified file (data_symbols.c and data_import.h) cfor importing platform-defined symbols. As we are MCU software so referencing symbols directly instead of getter function would be more straightforward. But since it is another bridge between SPM and platform so at least we need to well introduce it in the HAL docs.
This idea is not solid planned so what you think of unifying the external symbol referencing mechanism?
BR
/Ken
From: Chris.Brand--- via TF-M <tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org> Sent: Tuesday, February 28, 2023 7:42 AM To: tf-m@lists.trustedfirmware.orgmailto:tf-m@lists.trustedfirmware.org Subject: [TF-M] tfm_hal_memory_symbols
Hi,
I'm looking at https://git.trustedfirmware.org/TF-M/trusted-firmware-m.git/commit/?id=df871... which added tfm_hal_memory_symbols.{c|h}, containing tfm_hal_sp_meta_start and tfm_hal_sp_meta_end. The former is used in backend_ipc.c, while the latter is unused. The commit message says "this is the first example of using defined symbols to get memory info" but that commit is over two years old now and there doesn't seem to be a second example. Is there still a plan to move in this direction? If so, can somebody outline what that involves?
Thanks,
Chris Brand
Cypress Semiconductor (Canada), Inc. An Infineon Technologies Company Sr Prin Software Engr CSCA CSS ICW SW PSW 1 Chris.Brand@infineon.commailto:Chris.Brand@infineon.com
International Place 13700 V6V 2X8 Richmond Canada
www.infineon.com<www.cypress.com> www.cypress.comhttp://www.cypress.com Discoverieshttp://www.infineon.com/discoveries Facebookhttp://www.facebook.com/infineon Twitterhttp://www.twitter.com/Infineon LinkedInhttp://www.linkedin.com/company/infineon-technologies
Part of your life. Part of tomorrow.
NOTICE: The information transmitted is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material of Infineon Technologies AG and its affiliated entities which is for the exclusive use of the individual designated above as the recipient. Any review, retransmission, dissemination or other use of, or taking of any action in reliance upon, this information by persons or entities other than the intended recipient is prohibited. If you received this in error, please contact immediately the sender by returning e-mail and delete the material from any computer. If you are not the specified recipient, you are hereby notified that all disclosure, reproduction, distribution or action taken on the basis of this message is prohibited.
tf-m@lists.trustedfirmware.org
-
Chris.Brand＠infineon.com -
Ken Liu