On 06/12/2016 12:01 PM, Michael Rolnik wrote:
This series of patches adds 8bit AVR cores to QEMU. All instruction, except BREAK/DES/SPM/SPMX, are implemented. Not fully tested yet. However I was able to execute simple code with functions. e.g fibonacci calculation. This series of patches include a non real, sample board. No fuses support yet. PC is set to 0 at reset.the patches include the following 1. just a basic 8bit AVR CPU, without instruction decoding or translation 2. CPU features which allow define the following 8bit AVR cores avr1 avr2 avr25 avr3 avr31 avr35 avr4 avr5 avr51 avr6 xmega2 xmega4 xmega5 xmega6 xmega7 3. a definition of sample machine with SRAM, FLASH and CPU which allows to execute simple code 4. encoding for all AVR instructions 5. interrupt handling 6. helpers for IN, OUT, SLEEP, WBR & unsupported instructions 7. a decoder which given an opcode decides what istruction it is 8. translation of AVR instruction into TCG 9. all features together changes since v3
You should be continuing to add to this list of changes with every revision. At present I'm having to go back and diff this against your previous patch sets, something that most reviewers aren't going to do.
You need to keep up with checkpatch.pl warnings: total: 0 errors, 0 warnings, 1214 lines checked total: 0 errors, 0 warnings, 388 lines checked total: 0 errors, 0 warnings, 361 lines checked total: 0 errors, 0 warnings, 762 lines checked total: 0 errors, 0 warnings, 69 lines checked total: 0 errors, 0 warnings, 189 lines checked total: 0 errors, 0 warnings, 693 lines checked total: 2 errors, 0 warnings, 2743 lines checked total: 0 errors, 0 warnings, 203 lines checked total: 4 errors, 4 warnings, 115 lines checked total: 276 errors, 30 warnings, 1285 lines checked r~
