-
Notifications
You must be signed in to change notification settings - Fork 3k
LPC546XX: Add support for 220MHz core speed available on LPC54628 #6147
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
Conversation
/morph build |
Build : SUCCESSBuild number : 1197 Triggering tests/morph test |
Exporter Build : FAILUREBuild number : 869 |
453f217
to
668e124
Compare
I have pushed in an update to fix the Exporter BUILD Failure. |
/morph build |
Build : SUCCESSBuild number : 1199 Triggering tests/morph test |
/morph uvisor-test |
@mmahadevan108 Can you cherry pick also If the line is present the wait() function takes longer when adc enabled. What do you think? |
Test : FAILUREBuild number : 999 |
@gorazdko Can you provide details on why this change is required? |
@0xc0170 I am not clear why mbed-ci is failing |
We will review the logs (seems based on the info I am seeing to me that this is CI failure but need to confirm). |
Exporter Build : FAILUREBuild number : 871 |
I missed that you pushed an update we run CI, restarting /morph build |
Build : SUCCESSBuild number : 1205 Triggering tests/morph test |
Test : SUCCESSBuild number : 1001 |
@gorazdko I understand your change. Can you please submit a separate PR for your change. Thanks. |
@mmahadevan108 Ok I will. I will change this at 546xx and 54114 |
@gorazdko Thank you. |
Exporter Build : FAILUREBuild number : 876 |
@0xc0170 I don't see this failure reported by mbed-ci. I am using
mbed-ci is using
Can you help update and try. |
@0xc0170 I see the "release_versions" in targets.json should be placed inside children afterall since now the parent (MCU_LPC546XX) is treated as a target. Will correct this next time. Hope this is not causing problems now. |
@mmahadevan108, it looks like the difference between our build tools is that you're building using v8 of IAR. At the moment, mbed-os and its tools do not support v8. |
@cmonr How do you suggest we proceed? |
@mmahadevan108 Is the LPC54628J512 only supported in IAR 8? |
Here is what I see in IAR release notes: Device | Released |
This has to wait for IAR update 8. If this is the case, exporting should not be enabled |
Signed-off-by: Mahesh Mahadevan <[email protected]>
Signed-off-by: Mahesh Mahadevan <[email protected]>
668e124
to
069c80b
Compare
/morph build |
Build : SUCCESSBuild number : 1221 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 890 |
Test : SUCCESSBuild number : 1020 |
Read the part ID at bootup and increase core speed to 220MHz on LPC54628
Status
**READY