search

rusefi / rusefi-hardware

15 stars 19 forks source link

testEcuDigitalOutput seems broken around high, low and banks #251

Closed rusefillc closed 1 year ago

rusefillc commented 1 year ago

We have one or more problem with ECU output testing on the stim firmware side

hellen112-17 has 19 low side outputs. channels 17-19 are failing test

rusefillc commented 1 year ago

See https://github.com/rusefi/rusefi-hardware/commit/f8e6445c91802053dcae5e18d64f7fe8df928fed which improves naming in a suspicious area

15/40 LS 16 GOOD channel 16
               Setting ADDR 0
               sending line=17 value=1
                          Processing BENCH_TEST_IO_META_INFO ID=770005/l=8 66 0 20 10 0 0 0 0
       CAN ECU says: total=32 outputs of which low side=16
                          Processing BENCH_TEST_BOARD_STATUS ID=770003/l=8 F0 21 0 4 ED 0 49 0
                          Processing BENCH_TEST_BUTTON_COUNTERS ID=770004/l=8 E1 0 0 0 0 0 0 0
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
                      ADC says HIGH 17@0 1.090v
               sending line=17 value=0
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
                      ADC says LOW 17@1 0.002v
               sending line=17 value=1
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
                          Processing BENCH_TEST_IO_META_INFO ID=770005/l=8 66 0 20 10 0 0 0 0
       CAN ECU says: total=32 outputs of which low side=16
                          Processing BENCH_TEST_BOARD_STATUS ID=770003/l=8 F0 21 0 4 ED 0 49 0
                          Processing BENCH_TEST_BUTTON_COUNTERS ID=770004/l=8 E1 0 0 0 0 0 0 0
               sending line=17 value=0
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
16/40 IGN 1 GOOD channel 17
               Setting ADDR 1
               sending line=18 value=1
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
                      ADC says HIGH 18@0 1.090v
               sending line=18 value=0
                          Processing BENCH_TEST_IO_META_INFO ID=770005/l=8 66 0 20 10 0 0 0 0
       CAN ECU says: total=32 outputs of which low side=16
                          Processing BENCH_TEST_BOARD_STATUS ID=770003/l=8 F0 21 0 4 ED 0 49 0
                          Processing BENCH_TEST_BUTTON_COUNTERS ID=770004/l=8 E1 0 0 0 0 0 0 0
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
                      ADC says LOW 18@1 0.002v
               sending line=18 value=1
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
               sending line=18 value=0
                          Processing BENCH_TEST_EVENT_COUNTERS ID=770000/l=8 E0 E0 E0 E0 0 0 F0 0
                          Processing BENCH_TEST_RAW_ANALOG_1 ID=770001/l=8 1B 29 35 2C 20 73 44 42
                          Processing BENCH_TEST_RAW_ANALOG_2 ID=770006/l=8 0 0 45 72 0 0 0 0
                          Processing BENCH_TEST_IO_META_INFO ID=770005/l=8 66 0 20 10 0 0 0 0
       CAN ECU says: total=32 outputs of which low side=16
                          Processing BENCH_TEST_BOARD_STATUS ID=770003/l=8 F0 21 0 4 EE 0 49 0
                          Processing BENCH_TEST_BUTTON_COUNTERS ID=770004/l=8 E1 0 0 0 0 0 0 0