Backport 27263 ([personalize] pad UJSON messages to fixed sizes) #29247
+517
−115
Conversation
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
pamaury
force-pushed
the
backport_27263
branch
from
9ee40fc to
5f6f278
Compare
cfrantz
approved these changes
Feb 5, 2026
pamaury
force-pushed
the
backport_27263
branch
from
5f6f278 to
7b56a33
Compare
In manufacturing ATE environments, constant sized UJSON strings are preferred. CRCs are variable in length. Therefore, we enable setting the CRC to a fixed value of 0 and skipping the check on the host side. Signed-off-by: Tim Trippel <[email protected]> (cherry picked from commit 9a66cc2)
This adds a function to opentitanlib to enable padding a UJSON string with whitespace before the final "}" to enable testing always constant sized host-->device UJSON transmissions. These will be used during ATE provisioning flows. Signed-off-by: Tim Trippel <[email protected]> (cherry picked from commit 16b435f)
This adds UJSON serialization functions to enable transmitting fixed size UJSON strings from the device --> host. This will be useful for provisioning flows in ATE environments. Signed-off-by: Tim Trippel <[email protected]> (cherry picked from commit 3b354ff)
To facilitate bringup of the personalization flow in an ATE environment fixed sized UJSON messages are required. This updates the personalization flow to pad all UJSON messages between the host and device with whitespace before each message's trailing "}" character to ensure fixed size data transmissions. Signed-off-by: Tim Trippel <[email protected]> (cherry picked from commit b05b4f7)
pamaury
force-pushed
the
backport_27263
branch
from
7b56a33 to
d546c2f
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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.
Backport #27263. Depends on #29238, only review last 3 commits.