Re: [PATCH] STM32H7 BSP: Commented out legacy define which causes nameclashes

2021-01-28 Thread Joel Sherrill
I think this is ok. A BSP should not be defining standard named symbols. Does any code internally use PAGESIZE? Just checking for side-effects. It probably should be in an ifndef RTEMS block with a comment above to make it easier to potentially update this file in the future. --joel On Thu, Jan

[PATCH] STM32H7 BSP: Commented out legacy define which causes nameclashes

2021-01-28 Thread Robin Mueller
--- I had a warning related to nameclash of defines for the arm/stm32h7 BSP. More specifically, the define PAGESIZE which is defined in stm32_hal_legacy.h as well as in limits.h (which is located in arm-rtems6/include) I solved the problem for now by commenting out the define in stm32_hal_legacy.