A nit, but it is slightly confusing that the documentation (including this comment) uses the string "SOC_DBG_{BLANK,PRE_PROD,PROD}" for the enumerated values of the debug states, but the enumeration in lc_ctrl_state.hjson
is just {BLANK,PRE_PROD,PROD}. It would be good to align these two at some point, and I would suggest adding "SOC_DBG_" to the enumerations in lc_ctrl_state.hjson.
Originally posted by @hcallahan-lowrisc in #28403 (comment)