Commit 9481e08
committed
feat(evse_manager)!: restructure charging state machine to simplify pause/resume and remove replug
Remove WaitingForEnergy, Replug states and related session events
(ChargingResumed, ReplugStarted, ReplugFinished, WaitingForEnergy).
Consolidate pause handling into ChargingPausedEVSE with reason tracking
(Error, NoEnergy, UserPause) via new PauseChargingEVSEReasonEnum.
Key changes in Charger state machine:
- Replace WaitingForEnergy state with direct transitions through
ChargingPausedEVSE, which now reports multiple concurrent reasons
- Remove evse_replug command from BSP interface and all implementations
- Remove EvseReplugStarted/Finished events from board_support_common
- Add configurable hlc_charge_loop_without_energy_timeout_s for ISO
sessions to handle no-energy scenarios gracefully in charge loop
- Add dc_ramp_ampere_per_second config for DC current ramping
- Change zero_power_ignore_pause default to true for better EV compatibility
- Introduce StoppingCharging as proper transitional state that waits
for contactor open before moving to paused or finished states
- Use atomic flags (flag_authorized, flag_transaction_active,
flag_ev_plugged_in, flag_paused_by_evse) for thread-safe state
BREAKING CHANGE: Removes WaitingForEnergy, ChargingResumed,
ReplugStarted, ReplugFinished session events and evse_replug BSP
command. Consumers must handle ChargingPausedEVSE with reason
tracking instead.
Smoke tests:
- Add pause/resume session commands to test controller interface
- Add comprehensive smoke tests for pause/resume and no-energy scenarios
- Remove deprecated startup_tests.py in favor of new probe-based tests
Signed-off-by: Piet Gömpel <pietgoempel@gmail.com>
Signed-off-by: Cornelius Claussen <cc@pionix.de>
Signed-off-by: Sebastian Lukas <sebastian.lukas@pionix.de>1 parent cd96d4d commit 9481e08
File tree
69 files changed
+1556
-945
lines changed- applications
- pionix_chargebridge
- include/charge_bridge/everest_api
- src/everest_api
- utils/everest-testing/src/everest/testing/core_utils/controller
- config
- docs/source/reference/EVerest_API
- interfaces
- lib/everest/everest_api_types
- include/everest_api_types
- evse_board_support
- evse_manager
- private_include/everest_api_types/evse_manager
- src/everest_api_types
- evse_board_support
- evse_manager
- tests
- modules
- API
- API
- RpcApi
- data
- docs
- helpers
- tests/helpers
- types
- json_rpc_api
- evse_board_support_API/main
- evse_manager_consumer_API
- EVSE
- Auth
- lib
- tests
- EvseManager
- energy_grid
- evse
- tests
- OCPP201
- OCPP
- HardwareDrivers/EVSE
- AdAcEvse22KwzKitBSP/board_support
- MicroMegaWattBSP/board_support
- PhyVersoBSP
- connector_1
- connector_2
- TIDA010939/board_support
- YetiDriver
- board_support
- docs
- Simulation/YetiSimulator/board_support
- tests/core_tests
- types
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
69 files changed
+1556
-945
lines changedLines changed: 0 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
76 | | - | |
77 | 76 | | |
78 | 77 | | |
79 | 78 | | |
| |||
Lines changed: 0 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
119 | 119 | | |
120 | 120 | | |
121 | 121 | | |
122 | | - | |
123 | | - | |
124 | 122 | | |
125 | 123 | | |
126 | 124 | | |
| |||
393 | 391 | | |
394 | 392 | | |
395 | 393 | | |
396 | | - | |
397 | | - | |
398 | | - | |
399 | 394 | | |
400 | 395 | | |
401 | 396 | | |
| |||
Lines changed: 32 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
56 | | - | |
| 56 | + | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
60 | 60 | | |
61 | | - | |
| 61 | + | |
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
73 | 103 | | |
74 | 104 | | |
75 | 105 | | |
| |||
Lines changed: 30 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
49 | 79 | | |
50 | 80 | | |
51 | 81 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| 52 | + | |
| 53 | + | |
52 | 54 | | |
53 | 55 | | |
54 | 56 | | |
| |||
161 | 163 | | |
162 | 164 | | |
163 | 165 | | |
164 | | - | |
| 166 | + | |
165 | 167 | | |
166 | 168 | | |
167 | 169 | | |
| |||
170 | 172 | | |
171 | 173 | | |
172 | 174 | | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
4 | 13 | | |
5 | 14 | | |
6 | 15 | | |
| |||
35 | 44 | | |
36 | 45 | | |
37 | 46 | | |
38 | | - | |
39 | | - | |
| 47 | + | |
| 48 | + | |
40 | 49 | | |
41 | 50 | | |
42 | 51 | | |
| |||
47 | 56 | | |
48 | 57 | | |
49 | 58 | | |
50 | | - | |
| 59 | + | |
51 | 60 | | |
52 | 61 | | |
53 | 62 | | |
| |||
108 | 117 | | |
109 | 118 | | |
110 | 119 | | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
118 | | - | |
119 | | - | |
| 120 | + | |
120 | 121 | | |
121 | 122 | | |
122 | 123 | | |
| |||
128 | 129 | | |
129 | 130 | | |
130 | 131 | | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
Lines changed: 2 additions & 34 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
62 | 62 | | |
63 | 63 | | |
64 | 64 | | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
| 65 | + | |
| 66 | + | |
72 | 67 | | |
73 | 68 | | |
74 | 69 | | |
| |||
205 | 200 | | |
206 | 201 | | |
207 | 202 | | |
208 | | - | |
209 | | - | |
210 | | - | |
211 | | - | |
212 | | - | |
213 | | - | |
214 | 203 | | |
215 | 204 | | |
216 | 205 | | |
| |||
384 | 373 | | |
385 | 374 | | |
386 | 375 | | |
387 | | - | |
388 | | - | |
389 | | - | |
390 | | - | |
391 | | - | |
392 | | - | |
393 | | - | |
394 | | - | |
395 | | - | |
396 | | - | |
397 | | - | |
398 | | - | |
399 | | - | |
400 | | - | |
401 | | - | |
402 | | - | |
403 | | - | |
404 | 376 | | |
405 | 377 | | |
406 | 378 | | |
| |||
624 | 596 | | |
625 | 597 | | |
626 | 598 | | |
627 | | - | |
628 | | - | |
629 | 599 | | |
630 | 600 | | |
631 | 601 | | |
| |||
636 | 606 | | |
637 | 607 | | |
638 | 608 | | |
639 | | - | |
640 | | - | |
641 | 609 | | |
642 | 610 | | |
643 | 611 | | |
| |||
Lines changed: 1 addition & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
871 | 871 | | |
872 | 872 | | |
873 | 873 | | |
874 | | - | |
875 | 874 | | |
876 | 875 | | |
877 | 876 | | |
| |||
887 | 886 | | |
888 | 887 | | |
889 | 888 | | |
890 | | - | |
891 | 889 | | |
892 | 890 | | |
893 | 891 | | |
| |||
1320 | 1318 | | |
1321 | 1319 | | |
1322 | 1320 | | |
1323 | | - | |
| 1321 | + | |
1324 | 1322 | | |
1325 | 1323 | | |
1326 | 1324 | | |
| |||
1347 | 1345 | | |
1348 | 1346 | | |
1349 | 1347 | | |
1350 | | - | |
1351 | 1348 | | |
1352 | | - | |
1353 | 1349 | | |
1354 | 1350 | | |
1355 | 1351 | | |
1356 | 1352 | | |
1357 | 1353 | | |
1358 | 1354 | | |
1359 | | - | |
1360 | | - | |
1361 | 1355 | | |
1362 | 1356 | | |
1363 | 1357 | | |
| |||
1374 | 1368 | | |
1375 | 1369 | | |
1376 | 1370 | | |
1377 | | - | |
1378 | | - | |
1379 | 1371 | | |
1380 | 1372 | | |
1381 | 1373 | | |
1382 | 1374 | | |
1383 | 1375 | | |
1384 | 1376 | | |
1385 | | - | |
1386 | | - | |
1387 | 1377 | | |
1388 | 1378 | | |
1389 | 1379 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | | - | |
73 | | - | |
74 | 62 | | |
75 | 63 | | |
76 | 64 | | |
| |||
Lines changed: 0 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
22 | | - | |
23 | 21 | | |
24 | 22 | | |
25 | 23 | | |
| |||
0 commit comments