Mark,
if car_chargestate will still have the value 4 when it comes from 0x04 to 0x63 then charge done seems correct.
I don't quite get the else if part. What does chargestate 21 represent? And if we don't come from 0x04 it has to be an interruption and therefore needs no further check with chargestate or chargesubstate, does it? I guess I'm missing something here.

Patrick


Am 14.04.2013 um 16:06 schrieb Mark Webb-Johnson <mark@webb-johnson.net>:

Patrick,

There shouldn't be any 'charge complete' notifications - only 'charge interrupted':

Current logic is:

          case 0x63: // Charge done/failed (depending where we came from)
            car_doors1 &= ~0x0c;    // Clear charge and pilot bits
            car_chargemode = 0;     // Standard charge mode
            car_charge_b4 = 0;      // Not required
            if (car_chargestate == 4)
              { // Charge done
              }
            else if (car_chargestate != 21)
              { // Charge interrupted
              car_chargestate = 21;    // Charge STOPPED
              car_chargesubstate = 14; // Charge INTERRUPTED
              net_req_notification(NET_NOTIFY_CHARGE);
              }
            break;

PID 0xFDE0 shows 8bit value 0x63 and car_chargestate != 4 and car_chargestate != 21, then we treat that as a charge interrupted and alert appropriately.

Does that seem correct? I'll have to check through the logs you made to double-check.

Regards, Mark.

On 14 Apr, 2013, at 5:53 AM, Patrick Kapsch <patrick.kapsch@mac.com> wrote:

Mark,
Since you fixed push notifications, fully charged notification works for the Tazzari. Charge interrupt notification doesn't.

Regards,
Patrick
_______________________________________________
OvmsDev mailing list
OvmsDev@lists.teslaclub.hk
http://lists.teslaclub.hk/mailman/listinfo/ovmsdev

_______________________________________________
OvmsDev mailing list
OvmsDev@lists.teslaclub.hk
http://lists.teslaclub.hk/mailman/listinfo/ovmsdev