Skip to content

Conversation

@ccli8
Copy link

@ccli8 ccli8 commented Nov 18, 2024

Summary of changes

This migrates to new memory bank information for Nuvoton targets. It updates memory bank symbols in linker scripts and related files as follows:

  1. MBED_APP_START/SIZE -> MBED_CONFIGURED_ROM_BANK_IROM1_START/SIZE
  2. MBED_RAM_APP_START/SIZE -> MBED_CONFIGURED_RAM_BANK_IRAM1_START/SIZE
  3. APPLICATION_ADDR/SIZE -> MBED_CONFIGURED_ROM_BANK_IROM1_START/SIZE
  4. APPLICATION_RAM_ADDR/SIZE -> MBED_CONFIGURED_RAM_BANK_IRAM1_START/SIZE
  5. -> MBED_CONFIGURED_RAM_BANK_XRAM1_START/SIZE (NUC472)
  6. NU_HYPERRAM_START/SIZE -> MBED_CONFIGURED_RAM_BANK_HYPERRAM1_START/SIZE (M467)

Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Armclang and IAR toolchains are not supported in Mbed CE. Remove them
for clean.
Copy link
Collaborator

@multiplemonomials multiplemonomials left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome, thank you for updating these!

Change memory bank symbols:
1. MBED_APP_START/SIZE -> MBED_CONFIGURED_ROM_BANK_IROM1_START/SIZE
   NOTE: IROM2 for M2354
2. MBED_RAM_APP_START/SIZE -> MBED_CONFIGURED_RAM_BANK_IRAM1_START/SIZE
   NOTE: IRAM2 for M2354
3. APPLICATION_ADDR/SIZE -> MBED_CONFIGURED_ROM_BANK_IROM1_START/SIZE
4. APPLICATION_RAM_ADDR/SIZE -> MBED_CONFIGURED_RAM_BANK_IRAM1_START/SIZE
5. -> MBED_CONFIGURED_RAM_BANK_XRAM1_START/SIZE
   NOTE: Specific to NUC472
6. NU_HYPERRAM_START/SIZE -> MBED_CONFIGURED_RAM_BANK_HYPERRAM1_START/SIZE
   NOTE: Specific to M467
@ccli8 ccli8 force-pushed the nvt_migrate_to_new_mem_bank_info branch from c827166 to c434ea2 Compare November 22, 2024 08:38
@multiplemonomials multiplemonomials merged commit 99fcb5b into mbed-ce:master Nov 22, 2024
52 checks passed
@ccli8 ccli8 deleted the nvt_migrate_to_new_mem_bank_info branch November 22, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants