RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

bsps/powerpc/include/mpc8xx.h: Disable zero length array

Browse Source 
A padding array ends up with a size of 0 which is illegal in newer
versions of C -- at least C17 and later.
This commit is contained in:
Joel Sherrill
2025-08-01 15:08:03 -05:00
parent 47e4832f9a
commit b9b572335e
1 changed files with 15 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
bsps/powerpc/include/mpc8xx.h
View File

@@ -1478,7 +1478,13 @@ typedef struct m8xx_ {
m8xxSPIparms_t spip; m8xxSPIparms_t spip;
uint8_t _rsv5[0xDB0-0xD80-sizeof(m8xxSPIparms_t)]; uint8_t _rsv5[0xDB0-0xD80-sizeof(m8xxSPIparms_t)];
m8xxTimerParms_t tmp; m8xxTimerParms_t tmp;
/*
* This calculation results in a zero length array which is interpreted
* differently starting in C17. No padding is needed and there is a
* static assert below to double check the assumption.
*
uint8_t _rsv6[0xDC0-0xDB0-sizeof(m8xxTimerParms_t)]; uint8_t _rsv6[0xDC0-0xDB0-sizeof(m8xxTimerParms_t)];
*/
m8xxIDMAparms_t idma2p; m8xxIDMAparms_t idma2p;
uint8_t _rsv7[0xE00-0xDC0-sizeof(m8xxIDMAparms_t)]; uint8_t _rsv7[0xE00-0xDC0-sizeof(m8xxIDMAparms_t)];
@@ -1499,6 +1505,15 @@ typedef struct m8xx_ {
extern volatile m8xx_t m8xx; extern volatile m8xx_t m8xx;
/*
* The sizeof(m8xxTimerParms_t) is 16 and the _rsv6 element is not needed and
* is commented out. If the assumption breaks, it needs to be added back.
*/
RTEMS_STATIC_ASSERT(
0xDC0-0xDB0-sizeof(m8xxTimerParms_t) == 0,
_rsv6 size computation
);
#ifdef __cplusplus #ifdef __cplusplus
} }
#endif #endif