I think that's what is usually shown, the first 7 char, but in this case it might have been a typo

Was it a regulatory requirement before to have 7 characters from commit ID or something we implemented?

No need, in a different CR I have a module that pulls the Alarm ID, Reject Reason and Msg Ids directly from the FW. After that's merged, this file won't be needed any more manual update.

For more information check LDT-5205 CR, file: utils\ enum_updater.py

In FW we pre-populated Alarm IDs up to 300 to help with the reservation process. Would this also benefit dialin if we reserve up to 300?

references to Denali has been removed

Shouldn't we use Leahi instead of Denali?

done - as part of this comment, the piston pump module was deleted

done

done

This message ID (0xF0) is no longer valid? and remove empty spaces.

Update the comments = Valve 7, D26.

Update the comments - Valve 4, D21

Why so much white space?

???

Where is 0x5B?