TaskBG.c

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Just use 0x0 and 0xFFFFFFFF

Just use 0x0 and 0xFFFFFFFF

NUM_OF_BATTERY_REGISTERS - 1

NUM_OF_BATTERY_REGISTERS - 1

Put 2 items at end of the list here: BATTERY_END_OF_LIST_TIME_TO_PUBLISH NUM_OF_BATTERY_REGISTERS

Put 2 items at end of the list here:
BATTERY_END_OF_LIST_TIME_TO_PUBLISH
NUM_OF_BATTERY_REGISTERS

Put this one back to NUM_OF_BATTERY_DEVICES.

Put this one back to NUM_OF_BATTERY_DEVICES.

Don't need min/max for this. Just use 0x0 and 0xFFFFFFFF where needed.

Don't need min/max for this. Just use 0x0 and 0xFFFFFFFF where needed.

I don't think we need these min/max #defines. Just use 0x0 for minimums and 0xFFFFFFFF for maximums.

I don't think we need these min/max #defines. Just use 0x0 for minimums and 0xFFFFFFFF for maximums.

Not a float.

Not a float.

Not a float.

Not a float.

Not a float anymore. Use u32Override function and no need to cast the pointer.

Not a float anymore. Use u32Override function and no need to cast the pointer.

Should we publish all registers read?

Should we publish all registers read?

Remove this comment since we have a comment to right.

Remove this comment since we have a comment to right.

Remove comment.

Remove comment.

Set them individually instead of with memset.

Set them individually instead of with memset.

Add blank line before a /// comment.

Add blank line before a /// comment.

device and register is fine. Or deviceSel and registerSel.

device and register is fine. Or deviceSel and registerSel.

slaveAddr should be an enum now and so just "device".

slaveAddr should be an enum now and so just "device".

Add a banner before Dialin test support functions (see other units for example).

Add a banner before Dialin test support functions (see other units for example).

Initialize .override to OVERRIDE_RESET

Initialize .override to OVERRIDE_RESET

Why is this commented out? Do we not need to initialize it?

Why is this commented out? Do we not need to initialize it?

Rename similar to ADDR definitions below. these are not commands.

Rename similar to ADDR definitions below. these are not commands.

Combine charger and pack data into one payload with one broadcast message.

Combine charger and pack data into one payload with one broadcast message.

Move these 3 override messages (should end in OVERRIDE_REQUEST) to TD Dialin section below and assign an ID in the 0x8... range.

Move these 3 override messages (should end in OVERRIDE_REQUEST) to TD Dialin section below and assign an ID in the 0x8... range.

I think we only need one data message for both charger and battery pack data.

I think we only need one data message for both charger and battery pack data.

Remove.

Remove.

uncomment.

uncomment.

Call the init for the battery driver here.

Call the init for the battery driver here.

INVALID_BATTERY

INVALID_BATTERY

Remove repeating entries.

Remove repeating entries.

remove extra blank line.

remove extra blank line.

Remove this comment.

Remove this comment.