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.

Fixed.

Why? I don't recall commenting on private function prototypes. We comment them in function headers.

Why the has status register been removed?

The copyright section will be automatically updated during the builds.

Can some of these functions be added to the leahi-fwcommon alarm management? Like active alarm or clear alarm. I understand that it might be too difficult to tailor these functions for all of the stacks.

Add more details.