vh/forgefirm
1
0
Fork 0
mirror of https://github.com/openglow-org/forgefirm.git synced 2026-09-28 09:11:11 -07:00
Code Issues Packages Projects Releases Wiki Activity

Close the unobserved actions in the cloud item

Browse Source 
They were open because the service has never sent them and cannot be asked
to. That is not a reason to leave them open: the factory application says
what each one does, and it says all four are hand-offs to programs this
machine does not have. BRINGUP's cloud item now records the disposition
rather than the question.
This commit is contained in:
ScottW514
2026-08-20 15:39:40 -04:00
parent aaabfdf9b9
commit 3f4cd2b621
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
docs/BRINGUP.md
+9 -3
View File
@@ -989,13 +989,19 @@ Open items only. Anything closed is in `CAMPAIGN-LOG.md`.
service), the memory guards against a real ceiling rather than a reasoned service), the memory guards against a real ceiling rather than a reasoned
one, a print watched from the app to see the bar actually move, one, a print watched from the app to see the bar actually move,
packaged-path boot with packaged-path boot with
`controller_mode = cloud`, the three unobserved `controller_mode = cloud`, the lid-flash LED, and driving the park and the two lifecycle
actions, the lid-flash LED, and driving the park and the two lifecycle
periods off the header (`CFrh`, `CCwp`, `CCrp`) once a capture confirms periods off the header (`CFrh`, `CCwp`, `CCrp`) once a capture confirms
what they mean; the machine warms up and rests on the factory's measured what they mean; the machine warms up and rests on the factory's measured
timings meanwhile, and logs those keys on every job. (`CCbp`/`CCbt`, read timings meanwhile, and logs those keys on every job. (`CCbp`/`CCbt`, read
earlier as the pause constants, are refuted: the factory's tag table marks earlier as the pause constants, are refuted: the factory's tag table marks
them report-only, so they cannot appear in a pulse header at all.) Not them report-only, so they cannot appear in a pulse header at all.) The
four actions the service has never been seen to send are no longer open:
they were read out of the factory binary instead. `user_image` is a lid
capture and is implemented; `update_check`, `factory_reset` and
`head_firmware_update` each hand off to a program this machine does not
have (a factory updater, a reset script, a head firmware push), so each
is answered on the wire and none is performed, and `focus` is ignored
exactly as the factory ignores it. Not
inducible from the bench: the inducible from the bench: the
cancel-with-a-rejected-`settings`-action case, a malformed frame (needs a cancel-with-a-rejected-`settings`-action case, a malformed frame (needs a
MITM), a body past the memory guard (the service has no such job to send), MITM), a body past the memory guard (the service has no such job to send),