In the file mss_pdma.h, which is located in ~30 places, it uses the
#ifdef __cplusplus
extern "C" {
#endif
for C++ cross compilation. But it is missing the necessary closing bracket
#ifdef __cplusplus
}
#endif
preventing compilation and throwing an error.
I simply added the closing bracket that is present in all other .c and .h files as far as I have found. This is a minor fix for C++ compilation.
Type of change
Please delete options that are not relevant.
[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update
How Has This Been Tested?
Using the mss pdma sample code, I inserted it into a C++ project in SoftConsole (using the Hello World C++ as a base). I then made the change and compiled it. Such a changed allowed the example code to compile unlike before and work just like the C version.
Test Configuration:
Reference design release:
Hardware:
HSS version:
Bare metal examples version:
Buildroot / Yocto release:
Checklist:
[X] I have reviewed my code
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have tested that my fix is effective or that my feature works
[ ] I have added a maintainers file for any new board support
Description
In the file mss_pdma.h, which is located in ~30 places, it uses the
for C++ cross compilation. But it is missing the necessary closing bracket
preventing compilation and throwing an error.
I simply added the closing bracket that is present in all other
.cand.hfiles as far as I have found. This is a minor fix for C++ compilation.Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Using the mss pdma sample code, I inserted it into a C++ project in SoftConsole (using the Hello World C++ as a base). I then made the change and compiled it. Such a changed allowed the example code to compile unlike before and work just like the C version.
Test Configuration:
Checklist: