Add OTA library and PAL #1412
Add OTA library and PAL #1412
Conversation
Submodule JSMN to v1.1.0
submodule tinycbor to v0.5.3
| * @brief Test that otaPal_WriteBlock will return correct result code. | ||
| */ | ||
| void test_OTAPAL_WriteBlock_FseekError( void ) | ||
| { |
There was a problem hiding this comment.
Can this be implemented? This seems like an important test to have.
There was a problem hiding this comment.
Will be addressed in further PR from @yanjos-dev
| { | ||
| } | ||
|
|
||
| void test_OTAPAL_CloseFile_ValidSignature( void ) | ||
| { | ||
| OtaPalMainStatus_t result; | ||
| int16_t bytes_written = 0; |
There was a problem hiding this comment.
Can this be implemented? This seems like an important test to have.
There was a problem hiding this comment.
Will be addressed in further PR from @yanjos-dev
demos/lexicon.txt
Outdated
| @@ -370,6 +396,7 @@ pkcs | |||
| pki | |||
| pkparse | |||
| pkwrite | |||
| plaformimagestate | |||
There was a problem hiding this comment.
spelling: platformimagestage
There was a problem hiding this comment.
it is platform image state, we have valid, invalid, pending commit states I believe.
There was a problem hiding this comment.
right, but the t is missing from "platform",
| set(OPENSSL_USE_STATIC_LIBS TRUE) | ||
| find_package(OpenSSL REQUIRED) | ||
|
|
||
| set(THREADS_PREFER_PTHREAD_FLAG ON) | ||
| find_package(Threads REQUIRED) | ||
|
|
||
| find_library(LIB_RT rt REQUIRED) |
There was a problem hiding this comment.
What are these for? If this is being introduced in this PR, can I assume it is only required by the OTA PAL? In that case would it make sense to move this to the OTA PAL specific CMakeLists.txt instead of this one?
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.