kaidokert / rm32

63%

main: 63%

LAST BUILD ON BRANCH pr23_1

branch: pr23_1

CHANGE BRANCH

x

Reset

Sync Branches

• pr23_1
• main
• pr1
• pr10_1
• pr11_1
• pr12_1
• pr13_1
• pr14_1
• pr15_1
• pr16_1
• pr17_1
• pr18_1
• pr19_1
• pr20_1
• pr21_1
• pr22_1
• pr24_1
• pr25_1
• pr26_1
• pr27_1
• pr28_1
• pr29_1
• pr2_1
• pr2_2
• pr2_3
• pr2_4
• pr2_5
• pr2_6
• pr2_7
• pr2_ref
• pr30_1
• pr31_1
• pr32_1
• pr33_1
• pr34_1
• pr3_1
• pr5_1
• pr7_1
• pr8_1
• pr_codecov1
• pr_codecov2
• system-tick
• tmps

Build # 25205329108

Build Type

push

github

Committed by kaidokert

Commit Message

Make BemfState fields fully private, move logic from isr_logic.rs

Move BEMF logic into BemfState methods in state.rs:
- sync_config() — config sync from SharedComm
- update() — comparator sample processing (moved from bemf.rs)
- zero_cross_detected() — threshold check
- record_zero_cross() — timing + CI computation
- update_timing_from_timer() — commutation timer path
- record_zc_timing() — comparator ISR path
- reset_for_step(), reset_after_commutation() — counter resets
- com_timer_delay() — wait_time accessor

All 11 BemfState fields now private. bemf.rs reduced to tests only.
isr_logic.rs bemf_polling() now calls BemfState methods instead of
manipulating fields directly.

state.rs pub(crate) fields: 37 → 13 (BemfState 11 + DutyState 13 made private).

Coverage Stats

128 of 141 new or added lines in 3 files covered. (90.78%)

1 existing line in 1 file now uncovered.

3541 of 5650 relevant lines covered (62.67%)

779.23 hits per line