OpenCores

* Amber ARM-compatible core

Issue List
LDM/STM Writeback base register #27
Closed sleary opened this issue about 9 years ago
sleary commented about 9 years ago

I've mentioned this one already but i'm going to raise it so that we dont lose track of it. The following code writes back to R14_usr instead of R14_svc. It was introduced by a different fix. I'll put a patch on for it later.

<p> AREA |main|, CODE, READONLY EXPORT _start
ORG	&1000

_start TEQP PC,#3 ADR R14,datatable LDMIA R14!,{R0-R13}^ EXIT NOP NOP

SWI	&FFFFFF

NOP
NOP
	

datatable & &0000003c & &00000000 & &00000000 & &00000001 & &000010b8 & &0004b000 & &00000000 & &00000000 & &00001064 & &00050000 & &0381a290 & &0026dc06 & &00001000 & &01c01eb4

END
</p>
sleary commented about 9 years ago

AREA |main|, CODE, READONLY EXPORT _start

ORG	&1000

_start TEQP PC,#3 ADR R14,datatable LDMIA R14!,{R0-R13}^ EXIT NOP NOP

SWI	&FFFFFF

NOP
NOP
	

datatable & &0000003c & &00000000 & &00000000 & &00000001 & &000010b8 & &0004b000 & &00000000 & &00000000 & &00001064 & &00050000 & &0381a290 & &0026dc06 & &00001000 & &01c01eb4

END
sleary commented about 9 years ago

https://www.dropbox.com/s/u5iovqsl2avhnda/amber23_fixes.diff?dl=0

This patch fixes all open amber23 bugs.

  1. Fixes all the carry issues (verified with a fuzz - will post the fuzz result)
  2. Fixes all the rrx issues.
  3. Fixes the LDM/STM issue

Also patches the use of the wire/reg name type to itype. This is a reserved keyword on some verilog variations.

Note this patch doesnt include my Data abort fixes or the translate pin additions (Implemented as o_wb_tga).

csantifort commented about 9 years ago

Code changes from Stephen added to release 87 for a23, and release 88 for a25. This fixes the issue.

csantifort closed this about 9 years ago

Assignee
No one
Labels
Bug