Skip to content

Commit 2b4dfaf

Browse files
committed
portable: remove stray installer URL from IAR TrustZone non-secure ports
A stray IAR installer download URL was left outside the comment delimiters in the restore_special_regs section of the IAR TrustZone non-secure ports. The assembler parses the bare URL as an operand of the preceding ldmia, so any build of these ports aborts with: Error: garbage following instruction -- 'ldmia r2!,{r0,r3,lr} http:' The line is outside the configENABLE_PAC conditional block, so it breaks every configuration of the port, not only PAC-enabled builds. Commit 78e0cc7 ("ARMv8.1-M: Add task dedicated PAC key support", #1195) introduced the URL in the CM33/CM35P/CM55/CM85 and ARMv8M non-secure ports. The CM52 (#1334) and STAR-MC3 (#1363) ports inherited it when their portasm.s was created from an already-affected file. The stray URL is removed rather than moved inside the comment, matching the unaffected sibling port ARM_CM23/non_secure/portasm.s, which carries the same comment text without any URL. Verified with arm-none-eabi-gcc 13.3.1: all six files fail to assemble before the change and assemble cleanly after it. Relates to #1480. The ARM_CM33 non-secure port is handled separately by #1485, so it is intentionally not touched here.
1 parent 8be86d4 commit 2b4dfaf

6 files changed

Lines changed: 6 additions & 6 deletions

File tree

portable/ARMv8M/non_secure/portable/IAR/ARM_CM33/portasm.s

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ PendSV_Handler:
473473
msr PAC_KEY_P_0, r6
474474
clrm {r3-r6} /* Clear r3-r6. */
475475
#endif /* configENABLE_PAC */
476-
ldmia r2!, {r0, r3, lr} http://files.iar.com/ftp/pub/box/bxarm-9.60.3.deb/* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
476+
ldmia r2!, {r0, r3, lr} /* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
477477
msr psplim, r3 /* Restore the PSPLIM register value for the task. */
478478
ldr r3, =xSecureContext /* Read the location of xSecureContext i.e. &( xSecureContext ). */
479479
str r0, [r3] /* Restore the task's xSecureContext. */

portable/IAR/ARM_CM35P/non_secure/portasm.s

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ PendSV_Handler:
473473
msr PAC_KEY_P_0, r6
474474
clrm {r3-r6} /* Clear r3-r6. */
475475
#endif /* configENABLE_PAC */
476-
ldmia r2!, {r0, r3, lr} http://files.iar.com/ftp/pub/box/bxarm-9.60.3.deb/* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
476+
ldmia r2!, {r0, r3, lr} /* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
477477
msr psplim, r3 /* Restore the PSPLIM register value for the task. */
478478
ldr r3, =xSecureContext /* Read the location of xSecureContext i.e. &( xSecureContext ). */
479479
str r0, [r3] /* Restore the task's xSecureContext. */

portable/IAR/ARM_CM52/non_secure/portasm.s

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ PendSV_Handler:
473473
msr PAC_KEY_P_0, r6
474474
clrm {r3-r6} /* Clear r3-r6. */
475475
#endif /* configENABLE_PAC */
476-
ldmia r2!, {r0, r3, lr} http://files.iar.com/ftp/pub/box/bxarm-9.60.3.deb/* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
476+
ldmia r2!, {r0, r3, lr} /* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
477477
msr psplim, r3 /* Restore the PSPLIM register value for the task. */
478478
ldr r3, =xSecureContext /* Read the location of xSecureContext i.e. &( xSecureContext ). */
479479
str r0, [r3] /* Restore the task's xSecureContext. */

portable/IAR/ARM_CM55/non_secure/portasm.s

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ PendSV_Handler:
473473
msr PAC_KEY_P_0, r6
474474
clrm {r3-r6} /* Clear r3-r6. */
475475
#endif /* configENABLE_PAC */
476-
ldmia r2!, {r0, r3, lr} http://files.iar.com/ftp/pub/box/bxarm-9.60.3.deb/* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
476+
ldmia r2!, {r0, r3, lr} /* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
477477
msr psplim, r3 /* Restore the PSPLIM register value for the task. */
478478
ldr r3, =xSecureContext /* Read the location of xSecureContext i.e. &( xSecureContext ). */
479479
str r0, [r3] /* Restore the task's xSecureContext. */

portable/IAR/ARM_CM85/non_secure/portasm.s

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ PendSV_Handler:
473473
msr PAC_KEY_P_0, r6
474474
clrm {r3-r6} /* Clear r3-r6. */
475475
#endif /* configENABLE_PAC */
476-
ldmia r2!, {r0, r3, lr} http://files.iar.com/ftp/pub/box/bxarm-9.60.3.deb/* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
476+
ldmia r2!, {r0, r3, lr} /* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
477477
msr psplim, r3 /* Restore the PSPLIM register value for the task. */
478478
ldr r3, =xSecureContext /* Read the location of xSecureContext i.e. &( xSecureContext ). */
479479
str r0, [r3] /* Restore the task's xSecureContext. */

portable/IAR/ARM_STAR_MC3/non_secure/portasm.s

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ PendSV_Handler:
473473
msr PAC_KEY_P_0, r6
474474
clrm {r3-r6} /* Clear r3-r6. */
475475
#endif /* configENABLE_PAC */
476-
ldmia r2!, {r0, r3, lr} http://files.iar.com/ftp/pub/box/bxarm-9.60.3.deb/* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
476+
ldmia r2!, {r0, r3, lr} /* Read from stack - r0 = xSecureContext, r3 = PSPLIM and LR restored. */
477477
msr psplim, r3 /* Restore the PSPLIM register value for the task. */
478478
ldr r3, =xSecureContext /* Read the location of xSecureContext i.e. &( xSecureContext ). */
479479
str r0, [r3] /* Restore the task's xSecureContext. */

0 commit comments

Comments
 (0)