-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
net: lwm2m: use path as block context retrieval #57228
net: lwm2m: use path as block context retrieval #57228
Conversation
c7246c4
to
c82922d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit, looks good otherwise.
if (memcmp(path, &lwm2m_block1_context()[i].path, | ||
sizeof(struct lwm2m_obj_path)) == 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still looks wrongly aligned (please align to the opening bracket):
if (memcmp(path, &lwm2m_block1_context()[i].path, | |
sizeof(struct lwm2m_obj_path)) == 0) { | |
if (memcmp(path, &lwm2m_block1_context()[i].path, | |
sizeof(struct lwm2m_obj_path)) == 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
*ctx = NULL; | ||
|
||
for (i = 0; i < NUM_BLOCK1_CONTEXT; i++) { | ||
if (block1_contexts[i].tkl == tkl && | ||
memcmp(token, block1_contexts[i].token, tkl) == 0) { | ||
if (memcmp(path, &lwm2m_block1_context()[i].path, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if (memcmp(path, &lwm2m_block1_context()[i].path, | |
if (memcmp(path, &lwm2m_block1_context[i].path, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
c176f37
to
c8708be
Compare
DNM added : Test scheduled tomorrow with Leshan for FW update in push mode. |
c8708be
to
c2e8786
Compare
Successfully pushed firmware from Leshan |
Replace block context retrieval using object path instead of token. Update block context structure Fix issue zephyrproject-rtos#57165 Signed-off-by: romain pelletant <[email protected]>
c2e8786
to
3ed5fcf
Compare
Missing update in TLV writer found by CI |
Replace block context retrieval using object path instead of token. Changing token between each block is a correct behavior and restore a functional link with Leshan/Californium
Fix issue #57165