Removed.

No. Button press/release events will be sent as asynchronous events.

This one is different. Not using standard override payload as it is not an override - it's a set command.

I thought about that but decided to keep the common functions simple by calling the param a maximum.

Fixed.

Fixed.

are we not publishing the buttons state?

extra blank line

initilalizes the Airpump driver? Also, the outputs to mention unit variables initialized instead detailed list.

Needs to be updated per latest testsupport routine?

can we modify the routine to take index, max variables as it is ( without subtracting -1) and adjust the conditions inside the routine ( by removing = in the <= )?

I guess this code is common for all stacks. can we remove TD reference across the functions?

I can update the numbers.

But what is 0x3000?

Some day. I don't even know what it is and if we even used it in Denali. For now, I want to close this out and merge to staging.

Do you think we need it? So we should ask Noe to add it?

Noe didn't include it.

This function is used to calculate the baro sensor's CRC.

I kept these out of common because it's doing some specific things like broadcasting alarm trigger message.

Removed.

Details are in the header file where the group is defined. This is just saying that all of the code below goes into this group.

This one masks off lowest nibble. I will make a #define for it.

Who wrote this function? Do you know what to call it?

Correct.