Re: [TF-M] Change dual-core patch topic name "twincpu" to "dualcpu" - TF-M

11 Nov 2019


      Hi Reed,
Thanks for the input.
We name the dual-core/multi-core feature control option flag as `TFM_MULTI_CORE_TOPOLOGY` for a general model.
But as Chris said, it would require additional work to design and support more complicated scenarios. Thus for the current particular patches for dual-core platform, I'd like to specify the topic a little as "dualcpu".
Best regards,
Hu Ziji
-----Original Message-----
From: Reed Hinkel Reed.Hinkel@arm.com 
Sent: Sunday, November 10, 2019 5:45 AM
To: Christopher Brand chris.brand@cypress.com; Andrej Butok andrey.butok@nxp.com; David Hu (Arm Technology China) David.Hu@arm.com
Subject: Re: [TF-M] Change dual-core patch topic name "twincpu" to "dualcpu"
Makes sense!
All the best!
Reed
On 11/8/19, 11:05 AM, "Christopher Brand" chris.brand@cypress.com wrote:
I think that's actually an argument in favour of "dualcpu" for what we have now, because I suspect there would be significant work needed to support an additional core.
Chris
> -----Original Message-----
    > From: TF-M tf-m-bounces@lists.trustedfirmware.org On Behalf Of Reed
    > Hinkel via TF-M
    > Sent: Friday, November 8, 2019 6:53 AM
    > To: Andrej Butok andrey.butok@nxp.com; David Hu (Arm Technology
    > China) David.Hu@arm.com
    > Cc: tf-m@lists.trustedfirmware.org
    > Subject: Re: [TF-M] Change dual-core patch topic name "twincpu" to
    > "dualcpu"
    >
    > Just food for thought, but in the future some devices may have more than
    > two cores, but still with one core dedicated to RoT functions.
    >
    > -Reed
    >
    > On 11/8/19, 2:32 AM, "TF-M on behalf of Andrej Butok via TF-M" <tf-m-
    > bounces@lists.trustedfirmware.org on behalf of tf-
    > m@lists.trustedfirmware.org> wrote:
    >
    >     OK. We use "dual", so it's better ;)
    >
    >     -----Original Message-----
    >     From: TF-M tf-m-bounces@lists.trustedfirmware.org On Behalf Of David
    > Hu (Arm Technology China) via TF-M
    >     Sent: Friday, November 8, 2019 9:16 AM
    >     To: tf-m@lists.trustedfirmware.org
    >     Cc: nd nd@arm.com
    >     Subject: [TF-M] Change dual-core patch topic name "twincpu" to "dualcpu"
    >
    >     Hi all,
    >
    >     As you know, the current topic for dual-core patches is “twincpu”.
    >     In my very own opinion, keyword “twin” implies that the two cores are the
    > same in the system. However, the two cores can be in different archs or
    > even heterogeneous in a dual-core system.
    >
    >     Thus I’d like to propose to change the topic name to “dualcpu” to
    >
    >       *   Cover larger scope of dual-core topology/system.
    >       *   Align with the description in dual PE-based isolation section in TBSA-M
    > spec
    >
    >     Could you please let me know if you have any concern about that change?
    > Any feedback or suggestion is welcome. 😊
    >     Thank you.
    >
    >     Best regards,
    >     Hu Ziji
    >     --
    >     TF-M mailing list
    >     TF-M@lists.trustedfirmware.org
    >
    > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.t
    > rustedfirmware.org%2Fmailman%2Flistinfo%2Ftf-
    > m&amp;data=02%7C01%7Candrey.butok%40nxp.com%7C9dc8588da6e2436e
    > 754508d76423ef30%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63
    > 7087977806556698&amp;sdata=it6Hfh2Vk9A5FUFudmNn206LYO9Wb8lOattG
    > %2FWsm2%2Bo%3D&amp;reserved=0
    >     --
    >     TF-M mailing list
    >     TF-M@lists.trustedfirmware.org
    >     https://lists.trustedfirmware.org/mailman/listinfo/tf-m
    >
    >
    > IMPORTANT NOTICE: The contents of this email and any attachments are
    > confidential and may also be privileged. If you are not the intended recipient,
    > please notify the sender immediately and do not disclose the contents to any
    > other person, use it for any purpose, or store or copy the information in any
    > medium. Thank you.
    > --
    > TF-M mailing list
    > TF-M@lists.trustedfirmware.org
    > https://lists.trustedfirmware.org/mailman/listinfo/tf-m