Add STM32 F411RE Nucleo 64 BG96 demo project#13
Open
chinglee-iot wants to merge 21 commits intoFreeRTOS:mainfrom
Open
Add STM32 F411RE Nucleo 64 BG96 demo project#13chinglee-iot wants to merge 21 commits intoFreeRTOS:mainfrom
chinglee-iot wants to merge 21 commits intoFreeRTOS:mainfrom
Conversation
Contributor
chinglee-iot
commented
Jul 13, 2022
chinglee-iot
commented
- Add STM32 F411RE Nucleo 64 demo project
- Add README.md document to describe how to integrate FreeRTOS libraries to existing code base from scratch.
FreeRTOS -> v10.4.3 backoff_algorithm -> v1.0.0 coreMQTT -> v1.1.0 ThirdParty/mbedtls -> v2.25.0
* FreeRTOS V10.4.3-LTS-Patch-2 * FreeRTOS-Cellular-Interface v1.2.0-16-gdbbfbed * backoff_algorithm v1.2.0 * MbedTLS v2.28.0
Adjust the folder structure to close to FreeRTOS * Move demo tasks and config files to project folder * Update the README.md to state that this repository is for community supported demos
ActoryOu
reviewed
Jul 14, 2022
| */ | ||
| #define CELLULAR_IP_ADDRESS_MAX_SIZE ( 64U ) | ||
|
|
||
| #define CELLULAR_CONFIG_DISABLE_FLOW_CONTROL |
Member
There was a problem hiding this comment.
Suggested change
| #define CELLULAR_CONFIG_DISABLE_FLOW_CONTROL | |
| /* STM32 F411RE doesn't support timer, so we need to disable flow control. */ | |
| #define CELLULAR_CONFIG_DISABLE_FLOW_CONTROL |
Contributor
Author
There was a problem hiding this comment.
The reason to disable flow control is that the CTS/RTS pins are not connected together. I will add comments in the config.h file.
ActoryOu
reviewed
Jul 14, 2022
Comment on lines
+63
to
+66
| /* Use 1NCE service to onboard device. */ | ||
| #ifdef USE_1NCE_ZERO_TOUCH_PROVISIONING | ||
| #include "1nce_zero_touch_provisioning.h" | ||
| #endif |
Member
There was a problem hiding this comment.
Should we remove these lines which are dedicated for 1NCE?
Contributor
Author
There was a problem hiding this comment.
This will be removed in new commit.
ActoryOu
reviewed
Jul 14, 2022
Comment on lines
+488
to
+500
| #ifdef USE_1NCE_ZERO_TOUCH_PROVISIONING | ||
| uint8_t status = nce_onboard( &pThingName, | ||
| &pEndpoint, | ||
| &pExampleTopic, | ||
| &pRootCA, | ||
| &pClientCert, | ||
| &pPrvKey ); | ||
| configASSERT( status == EXIT_SUCCESS ); | ||
| #else | ||
| pThingName = democonfigCLIENT_IDENTIFIER; | ||
| pEndpoint = democonfigMQTT_BROKER_ENDPOINT; | ||
| pExampleTopic = mqttexampleTOPIC; | ||
| #endif /* ifdef USE_1NCE_ZERO_TOUCH_PROVISIONING */ |
Contributor
Author
There was a problem hiding this comment.
This will be removed in new commit.
ActoryOu
reviewed
Jul 15, 2022
* User should reference the 1nce demo for if they are interesting in 1nce * Add comment in cellular_config.h
…iot/Lab-Project-FreeRTOS-Cellular-Demo into add-stm32-demo-project
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.