Fix MISRA C 2012 rule 8.6 errors

[chinglee-iot]

Fix MISRA C 2012 rule 8.6 errors

Description

MISRA C 2012 rule 8.6

An identifier with external linkage shall have exactly one external definition

MISRA violation
The following is an example of violation. xQueueCRSendFromISR is declared in the header file but configUSE_CO_ROUTINES is not defined to 1. Therefore, no definition for xQueueCRSendFromISR.

BaseType_t xQueueCRSendFromISR( QueueHandle_t xQueue,
                                const void * pvItemToQueue,
                                BaseType_t xCoRoutinePreviouslyWoken );

Fix in this PR

• Use compile macro in header file to prevent functions declared without definition from compilation.

For example, in the example above use configUSE_CO_ROUTINES == 1 to ensure that the function is declared only when there is definition.

#if ( configUSE_CO_ROUTINES == 1 )
  BaseType_t xQueueCRSendFromISR( QueueHandle_t xQueue,
                                  const void * pvItemToQueue,
                                  BaseType_t xCoRoutinePreviouslyWoken );
#endif

Test Steps

N/A

Checklist:

• I have tested my changes. No regression in existing tests.
• I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (55094e2) 93.77% compared to head (e37d43e) 93.77%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #862   +/-   ##
=======================================
  Coverage   93.77%   93.77%           
=======================================
  Files           6        6           
  Lines        3183     3183           
  Branches      885      885           
=======================================
  Hits         2985     2985           
  Misses         91       91           
  Partials      107      107           

Flag	Coverage Δ
unittests	93.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication