[PATCH 3/5] bsp/lpc32xx: Remove hard to maintain comments

Sebastian Huber sebastian.huber at embedded-brains.de
Mon Aug 7 07:02:18 UTC 2023


These comments duplicate the memory and region definitions of the linker
command file.
---
 bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx   | 20 ------------------
 .../start/linkcmds.lpc32xx_mzx_stage_1        | 20 ------------------
 .../start/linkcmds.lpc32xx_mzx_stage_2        | 20 ------------------
 .../lpc32xx/start/linkcmds.lpc32xx_phycore    | 21 -------------------
 4 files changed, 81 deletions(-)

diff --git a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx
index cf388dfbba..3c7a0bce08 100644
--- a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx
+++ b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx
@@ -12,26 +12,6 @@
  * @ingroup bsp_linker
  *
  * @brief MZX application memory map.
- *
- * <table>
- *   <tr><th>Region Name</th><th>Region Begin</th><th>Region Size</th></tr>
- *   <tr><td>RAM_INT</td><td>0x08000000</td><td>256k</td></tr>
- *   <tr><td>RAM_MMU</td><td>0x80000000</td><td>16k</td></tr>
- *   <tr><td>RAM_EXT</td><td>0x80004000</td><td>32M - 16k</td></tr>
- * </table>
- *
- * <table>
- *   <tr><th>Section Name</th><th>Section Runtime Region</th><th>Section Load Region</th></tr>
- *   <tr><td>.start</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.vector</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.text</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.rodata</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.data</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.fast</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.bss</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.work</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.stack</td><td>RAM_INT</td><td></td></tr>
- * </table>
  */
 
 MEMORY {
diff --git a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_1 b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_1
index eae076e235..d50f17d7b7 100644
--- a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_1
+++ b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_1
@@ -12,26 +12,6 @@
  * @ingroup bsp_linker
  *
  * @brief MZX stage-1 program memory map.
- *
- * <table>
- *   <tr><th>Region Name</th><th>Region Begin</th><th>Region Size</th></tr>
- *   <tr><td>RAM_INT</td><td>0x08000000</td><td>232k</td></tr>
- *   <tr><td>RAM_MMU</td><td>0x0803a000</td><td>16k</td></tr>
- *   <tr><td>RAM_VEC</td><td>0x0803d000</td><td>8k</td></tr>
- * </table>
- *
- * <table>
- *   <tr><th>Section Name</th><th>Section Runtime Region</th><th>Section Load Region</th></tr>
- *   <tr><td>.start</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.vector</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.text</td><td>RAM_INT</td><td>RAM_INT</td></tr>
- *   <tr><td>.rodata</td><td>RAM_INT</td><td>RAM_INT</td></tr>
- *   <tr><td>.data</td><td>RAM_INT</td><td>RAM_INT</td></tr>
- *   <tr><td>.fast</td><td>RAM_INT</td><td>RAM_INT</td></tr>
- *   <tr><td>.bss</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.work</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.stack</td><td>RAM_INT</td><td></td></tr>
- * </table>
  */
 
 MEMORY {
diff --git a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_2 b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_2
index 2aa8c34091..b32e0f0a45 100644
--- a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_2
+++ b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_mzx_stage_2
@@ -12,26 +12,6 @@
  * @ingroup bsp_linker
  *
  * @brief MZX stage-2 program memory map.
- *
- * <table>
- *   <tr><th>Region Name</th><th>Region Begin</th><th>Region Size</th></tr>
- *   <tr><td>RAM_INT</td><td>0x08000000</td><td>256k</td></tr>
- *   <tr><td>RAM_MMU</td><td>0x81c00000</td><td>16k</td></tr>
- *   <tr><td>RAM_EXT</td><td>0x81c04000</td><td>4M - 16k</td></tr>
- * </table>
- *
- * <table>
- *   <tr><th>Section Name</th><th>Section Runtime Region</th><th>Section Load Region</th></tr>
- *   <tr><td>.start</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.vector</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.text</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.rodata</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.data</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.fast</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.bss</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.work</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.stack</td><td>RAM_INT</td><td></td></tr>
- * </table>
  */
 
 MEMORY {
diff --git a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_phycore b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_phycore
index 2459da2557..34542a6dcb 100644
--- a/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_phycore
+++ b/bsps/arm/lpc32xx/start/linkcmds.lpc32xx_phycore
@@ -12,27 +12,6 @@
  * @ingroup bsp_linker
  *
  * @brief phyCORE-LPC3250 memory map.
- *
- * <table>
- *   <tr><th>Region Name</th><th>Region Begin</th><th>Region Size</th></tr>
- *   <tr><td>RAM_INT</td><td>0x08000000</td><td>256k</td></tr>
- *   <tr><td>RAM_MMU</td><td>0x80000000</td><td>16k</td></tr>
- *   <tr><td>RAM_EXT</td><td>0x80004000</td><td>64M - 16k</td></tr>
- *   <tr><td>ROM_EXT</td><td>0xe0000000</td><td>2M</td></tr>
- * </table>
- *
- * <table>
- *   <tr><th>Section Name</th><th>Section Runtime Region</th><th>Section Load Region</th></tr>
- *   <tr><td>.start</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.vector</td><td>RAM_INT</td><td></td></tr>
- *   <tr><td>.text</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.rodata</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.data</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.fast</td><td>RAM_EXT</td><td>RAM_EXT</td></tr>
- *   <tr><td>.bss</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.work</td><td>RAM_EXT</td><td></td></tr>
- *   <tr><td>.stack</td><td>RAM_INT</td><td></td></tr>
- * </table>
  */
 
 MEMORY {
-- 
2.35.3



More information about the devel mailing list