ARMmbed / mbed-client-for-azure

Azure IoT Device SDK port for Mbed OS
Apache License 2.0
5 stars 4 forks source link

Apply azure-sdk-for-c for smaller ROM size #8

Closed cyliangtw closed 3 years ago

cyliangtw commented 3 years ago

Current ROM size of mbed-os-example-for-azure seems too big for normal MCU platforms. According to my evaluation based on the same NUMAKER_IOT_M487 platform:

We forked & ported mbed-os-example-for-azure on NUMKAER_IOT_M487 in this repo , it could work but ROM size is too big. So, I cross check with azure-iot-preview based on azure-sdk-for-c, this M487 azure-iot-preview repo is ROM size as small as 131 KB. I used the same toolchain to build these 2 examples for Azure connection and the key factor is azure azure-iot-sdk-c v.s azure-sdk-for-c.

So, is there any plan to apply azure-sdk-for-c to fulfill memory footprint of MCU level ?

ciarmcom commented 3 years ago

@cyliangtw thank you for raising this issue.Please take a look at the following comments:

Could you add some more detail to the description? A good description should be at least 25 words. What target(s) are you using? What toolchain(s) are you using? What version of Mbed OS are you using (tag or sha)? It would help if you could also specify the versions of any tools you are using? How can we reproduce your issue?

NOTE: If there are fields which are not applicable then please just add 'n/a' or 'None'.This indicates to us that at least all the fields have been considered. Please update the issue header with the missing information, the issue will not be mirroredto our internal defect tracking system or investigated until this has been fully resolved.

ciarmcom commented 3 years ago

@cyliangtw it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

cyliangtw commented 3 years ago

@ciarmcom , Sorry for late update We forked & ported mbed-os-example-for-azure on NUMKAER_IOT_M487 in this repo , it could work but ROM size is too big. So, I cross check with azure-iot-preview based on azure-sdk-for-c, this M487 azure-iot-preview repo is ROM size as small as 131 KB. I used the same toolchain to build these 2 examples for Azure connection and the key factor is azure azure-iot-sdk-c v.s azure-sdk-for-c.

ciarmcom commented 3 years ago

@cyliangtw it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

cyliangtw commented 3 years ago

@ciarmcom , I updated this issue header with target, tool chain, OS version & how to reproduce info. .

ciarmcom commented 3 years ago

@cyliangtw it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

ciarmcom commented 3 years ago

@cyliangtw it has been 5 days since the last reminder. Could you please update the issue header as previously requested?

cyliangtw commented 3 years ago

transfer to https://github.com/ARMmbed/mbed-client-for-azure/issues/10