Fix Arm Compiler support

ARMmbed / mbed-os-example-for-aws

Mbed OS example to connect to AWS IoT Core

Apache License 2.0

15 stars 21 forks source link

Fix Arm Compiler support #7

Closed LDong-Arm closed 4 years ago

LDong-Arm commented 4 years ago

To build & run the example with the Arm Compiler, we need to

Fix build issues in mbed-aws-client. ~Preceding PR: https://github.com/ARMmbed/mbed-aws-client/pull/4~ (merged)
Reduce stack memory size so all the threads can fit into DISCO_L475VG_IOT01A RAM (128KB, as opposed to K64F's 256KB). Without this PR, the memory usage is borderline too much probably, failing with ARMC6 and working with GCC_ARM. Out-of-memory happened when mbed-aws-client allocates the IoTClock SystemTimer thread's stack memory on DISCO_L475VG_IOT01A.

Fixes: #3

LDong-Arm commented 4 years ago

@ithinuel @ashok-rao @evedon @MarceloSalazar This is one of the two PRs to fix the AWS example with the Arm Compiler, would you please have a look? Thanks. (I don't have permission to add reviewers.)

ashok-rao commented 4 years ago

Thanks @LDong-Arm .. what about the README changes? Are you planning to them here or in a different PR? Thanks.

LDong-Arm commented 4 years ago

Thanks @LDong-Arm .. what about the README changes? Are you planning to them here or in a different PR? Thanks.

I'm working on it and would prefer a different PR - I'm adding some information I found necessary for users, so it will be slightly beyond the scope of this PR.

LDong-Arm commented 4 years ago

Just added a bit of extra improvements.

ithinuel commented 4 years ago

@LDong-Arm Does the log work without https://github.com/ARMmbed/mbed-aws-client/pull/3 ?

LDong-Arm commented 4 years ago

@LDong-Arm Does the log work without ARMmbed/mbed-aws-client#3 ?

It doesn't. I'm waiting for mbed-aws-client PRs to be merged so I can update the .lib file here.

LDong-Arm commented 4 years ago

@ithinuel I've updated mbed-aws-client.lib, could you merge this too?