OpenCores

* Amber ARM-compatible core

Issue List
LDM/STM Writeback base register #27
Closed sleary opened this issue over 10 years ago
sleary commented over 10 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 over 10 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 over 10 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 over 10 years ago

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

csantifort closed this over 10 years ago

Assignee
No one
Labels
Bug