<div dir="auto">I think this is ok. A BSP should not be defining standard named symbols.<div dir="auto"><br></div><div dir="auto">Does any code internally use PAGESIZE? Just checking for side-effects.</div><div dir="auto"><br></div><div dir="auto">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.</div><div dir="auto"><br></div><div dir="auto">--joel</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 28, 2021, 6:25 AM Robin Mueller <<a href="mailto:robin.mueller.m@gmail.com">robin.mueller.m@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">---<br>
<br>
I had a warning related to nameclash of defines for the arm/stm32h7 BSP.<br>
More specifically, the define PAGESIZE which is defined in stm32_hal_legacy.h<br>
as well as in limits.h (which is located in arm-rtems6/include)<br>
<br>
I solved the problem for now by commenting out the define in stm32_hal_legacy.h<br>
<br>
The BSP still built without issues. I searched the stm32h7 BSP folder in the sources<br>
to check whether the legacy define is actually used somewhere and it is not.<br>
<br>
 bsps/arm/stm32h7/include/Legacy/stm32_hal_legacy.h | 4 +++-<br>
 1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/bsps/arm/stm32h7/include/Legacy/stm32_hal_legacy.h b/bsps/arm/stm32h7/include/Legacy/stm32_hal_legacy.h<br>
index c311c1618e..c4fa5d7151 100644<br>
--- a/bsps/arm/stm32h7/include/Legacy/stm32_hal_legacy.h<br>
+++ b/bsps/arm/stm32h7/include/Legacy/stm32_hal_legacy.h<br>
@@ -420,7 +420,9 @@<br>
 #define TYPEPROGRAMDATA_FASTBYTE      FLASH_TYPEPROGRAMDATA_FASTBYTE<br>
 #define TYPEPROGRAMDATA_FASTHALFWORD  FLASH_TYPEPROGRAMDATA_FASTHALFWORD<br>
 #define TYPEPROGRAMDATA_FASTWORD      FLASH_TYPEPROGRAMDATA_FASTWORD<br>
-#define PAGESIZE                      FLASH_PAGE_SIZE<br>
+/* Commented out for the RTEMS BSP because there can be nameclashes with another <br>
+similar defininition in limits.h */<br>
+//#define PAGESIZE                      FLASH_PAGE_SIZE<br>
 #define TYPEPROGRAM_FASTBYTE          FLASH_TYPEPROGRAM_BYTE<br>
 #define TYPEPROGRAM_FASTHALFWORD      FLASH_TYPEPROGRAM_HALFWORD<br>
 #define TYPEPROGRAM_FASTWORD          FLASH_TYPEPROGRAM_WORD<br>
-- <br>
2.29.2.windows.2<br>
<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>