I see the moon mode lighting up (uses PHASE PWM value of 1), but the voltage values seem off a bit. Readings are generally higher, but in the low range, 3.1V reads as 3.3V. I enabled full n.n blinking to test it with real batteries.
I use 4 tables: for 19.1K and 22K R1's, and then with a diode before R1 or after R1. With these 2 variables you get 4 possible sets. Mine I've tested extensively. These should be the only variables effecting it, though I'm not familiar of what the impact is of a bleeder resistor.
These Convoy drivers use a 19.1K resistor (labeled 28C) and have R1 after the diode. So my table for this configuration seems to match up better on this driver. Here's your table:
#define ADC_44 194
#define ADC_43 189
#define ADC_42 184
#define ADC_41 178
#define ADC_40 173
#define ADC_39 168
#define ADC_38 163
#define ADC_37 158
#define ADC_36 152
#define ADC_35 147
#define ADC_34 142
#define ADC_33 137
#define ADC_32 131
#define ADC_31 126
#define ADC_30 121
#define ADC_29 116
#define ADC_28 111
#define ADC_27 105
#define ADC_26 100
#define ADC_25 95
#define ADC_24 90
#define ADC_23 84
#define ADC_22 79
Here's my table for R1=19.1K after the diode:
#define ADC_44 192
#define ADC_43 188
#define ADC_42 184
#define ADC_41 180
#define ADC_40 175
#define ADC_39 171
#define ADC_38 167
#define ADC_37 163
#define ADC_36 159
#define ADC_35 154
#define ADC_34 150
#define ADC_33 146
#define ADC_32 141
#define ADC_31 136
#define ADC_30 132
#define ADC_29 128
#define ADC_28 123
#define ADC_27 119
#define ADC_26 116
#define ADC_25 111
#define ADC_24 107
#define ADC_23 103
#define ADC_22 99
Notice they are close/exact at 4.2V, some difference at 3.6V, and by 3.1V, they differ quite a bit, and that's where I'm seeing the biggest difference in your table on the Convoy driver: a 3.1V cell blinks out at 3.3V.
Ohhh - also ran into a compile problem when enabling full n.n blink out. I made this change:
#define BLINK_SPEED (187) // 2017-03-17 TE: this was 750/4
It didn't like the 750/4 because it did the math as a 16 bit value, then truncated to a 8 bit value. Also made this change:
_delay_4ms(252); // 2017-03-17 TE: This was BLINK_SPEED*3/2
because of the same reason. Not sure if it's the Atmel VS 7.0 compiler, and just happening under Windows, or not. Had to disable bike strobes, SOS, and remove some mode sets to get it tot fit in 1024 bytes, but it works!
Edit: More info. I tested moon mode across 2 of the driver versions: 6 7135's (2.1A) and 8 7135's (2.8A) and on 3 different cells at 4.1V, 3.6V, and 3.1V and all combos worked fine.
Also -- I had to slow down the blink rate of the n.nV blinking in half because for some weird reason, I just can't count blinks going that fast - I can't. I tried many times, and just can't tell the difference between 6 blinks and 7, and it's even a little hard to see the difference between 2 and 3 blinks. Once slowed down, no problems counting then at all.