Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build failure for FUTURE_SEQUANA_M0 with PSOC6_DYNSRM_DISABLE=1 #8710

Closed
orenc17 opened this issue Nov 12, 2018 · 9 comments
Closed

Build failure for FUTURE_SEQUANA_M0 with PSOC6_DYNSRM_DISABLE=1 #8710

orenc17 opened this issue Nov 12, 2018 · 9 comments

Comments

@orenc17
Copy link
Contributor

orenc17 commented Nov 12, 2018

Description

Failure in compilation of FUTURE_SEQUANA_M0 with -DPSOC6_DYNSRM_DISABLE=1 on GCC_ARM
i'm getting undefined reference to DEFAULT_DIVIDER8_RES in targets\TARGET_Cypress\TARGET_PSOC6\psoc6_utils.c line 142

Issue request type

[ ] Question
[ ] Enhancement
[X] Bug

@lrusinowicz

@ciarmcom
Copy link
Member

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-158

@MarceloSalazar MarceloSalazar self-assigned this Nov 13, 2018
@MarceloSalazar
Copy link

discussed with @alzix - this is not a blocker but important to fix at some point soon

@MarceloSalazar MarceloSalazar removed their assignment Dec 4, 2018
@MarceloSalazar
Copy link

@lrusinowicz can you plese look into this?

@orenc17
Copy link
Contributor Author

orenc17 commented Dec 9, 2018

I'm sorry but #9009 does not really fixes the issue as now lines 103 & 105 define the same constant with different values

[Warning] psoc6_utils.c@105,0: "DEFAULT_DIVIDER8_RES" redefined

@lrusinowicz
Copy link
Contributor

@orenc17
Yes, I have missed this warning, nevertheless SRM is working for me now. I'm not seeing any undefined references, so it seems like something specific to your project. Could you point me to your code?

@orenc17
Copy link
Contributor Author

orenc17 commented Dec 17, 2018

@lrusinowicz
The problem now is redefinition.
Try running mbed compile -m FUTURE_SEQUANA_M0_PSA

@lrusinowicz
Copy link
Contributor

@orenc17
I've just found the redefinition is just a merge issue introduced when my change was merged into the master where your change was already applied to this file. So actually it's different issue, but I will submit a patch, likely tomorrow.
Have you tried removing the first definition already? Where is the main() for FUTURE_SEQUANA_M0_PSA?

@orenc17
Copy link
Contributor Author

orenc17 commented Dec 18, 2018

@lrusinowicz

  1. when a redefinition occurs the last definition is being used, so your definition works
  2. the main function is in https://github.com/ARMmbed/mbed-os/blob/master/components/TARGET_PSA/spm/COMPONENT_SPE/spm_main.c

@orenc17 orenc17 closed this as completed Jan 17, 2019
@MarceloSalazar
Copy link

Fixed here: #9155

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants