Hi Aleksandar. I guess no testing would be possible. My previous series of patches (about 2 years ago, before Sarah joined) did not contain any peripherals, there were only the CPU and the sample board, I used it to test instructions. Somebody complained that there are no peripherals and that was where I stopped to care as I did (and still don't) want to model devices. I believed that others would join and add devices. then two years passed by, Sarah implemented the timer and the UART devices. And here we are. Regards, Michael Rolnik On Sat, Nov 30, 2019 at 12:49 PM Aleksandar Markovic < aleksandar.m.mail@gmail.com> wrote: > > > On Wednesday, November 27, 2019, Michael Rolnik wrote: > >> A simple board setup that configures an AVR CPU to run a given firmware >> image. >> This is all that's useful to implement without peripheral emulation as >> AVR CPUs include a lot of on-board peripherals. >> >> NOTE: this is not a real board !!!! >> NOTE: it's used for CPU testing!!!! >> >> Signed-off-by: Michael Rolnik >> Reviewed-by: Aleksandar Markovic >> Nacked-by: Philippe Mathieu-Daudé >> --- >> hw/avr/sample.c | 282 +++++++++++++++++++++++++++++++++++++++++++ >> hw/Kconfig | 1 + >> hw/avr/Kconfig | 6 + >> hw/avr/Makefile.objs | 1 + >> 4 files changed, 290 insertions(+) >> create mode 100644 hw/avr/sample.c >> create mode 100644 hw/avr/Kconfig >> create mode 100644 hw/avr/Makefile.objs >> > > Michael, hi. > > I just need a clarification here: > > - What will happen if this patch is removed? Would boot and Avocado tests > work? What else in general wouldn't work or be available? What was, in > fact, the ultimate motivation for you to insert this patch? > > Thanks, > Aleksandar > > > >> diff --git a/hw/avr/sample.c b/hw/avr/sample.c >> new file mode 100644 >> index 0000000000..2295ec1b79 >> --- /dev/null >> +++ b/hw/avr/sample.c >> @@ -0,0 +1,282 @@ >> +/* >> + * QEMU AVR CPU >> + * >> + * Copyright (c) 2019 Michael Rolnik >> + * >> + * This library is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU Lesser General Public >> + * License as published by the Free Software Foundation; either >> + * version 2.1 of the License, or (at your option) any later version. >> + * >> + * This library is distributed in the hope that it will be useful, >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU >> + * Lesser General Public License for more details. >> + * >> + * You should have received a copy of the GNU Lesser General Public >> + * License along with this library; if not, see >> + * >> + */ >> + >> +/* >> + * NOTE: >> + * This is not a real AVR board, this is an example! >> + * The CPU is an approximation of an ATmega2560, but is missing >> various >> + * built-in peripherals. >> + * >> + * This example board loads provided binary file into flash memory >> and >> + * executes it from 0x00000000 address in the code memory space. >> + * >> + * Currently used for AVR CPU validation >> + * >> + */ >> + >> +#include "qemu/osdep.h" >> +#include "qapi/error.h" >> +#include "qemu-common.h" >> +#include "cpu.h" >> +#include "hw/hw.h" >> +#include "sysemu/sysemu.h" >> +#include "sysemu/qtest.h" >> +#include "ui/console.h" >> +#include "hw/boards.h" >> +#include "hw/loader.h" >> +#include "qemu/error-report.h" >> +#include "exec/address-spaces.h" >> +#include "include/hw/sysbus.h" >> +#include "include/hw/char/avr_usart.h" >> +#include "include/hw/timer/avr_timer16.h" >> +#include "include/hw/misc/avr_mask.h" >> +#include "elf.h" >> +#include "hw/misc/unimp.h" >> + >> +#define SIZE_FLASH 0x00040000 >> +#define SIZE_SRAM 0x00002000 >> +/* >> + * Size of additional "external" memory, as if the AVR were configured >> to use >> + * an external RAM chip. >> + * Note that the configuration registers that normally enable this >> feature are >> + * unimplemented. >> + */ >> +#define SIZE_EXMEM 0x00000000 >> + >> +/* Offsets of peripherals in emulated memory space (i.e. not host >> addresses) */ >> +#define PRR0_BASE 0x64 >> +#define PRR1_BASE 0x65 >> +#define USART_BASE 0xc0 >> +#define TIMER1_BASE 0x80 >> +#define TIMER1_IMSK_BASE 0x6f >> +#define TIMER1_IFR_BASE 0x36 >> + >> +/* Interrupt numbers used by peripherals */ >> +#define USART_RXC_IRQ 24 >> +#define USART_DRE_IRQ 25 >> +#define USART_TXC_IRQ 26 >> + >> +#define TIMER1_CAPT_IRQ 15 >> +#define TIMER1_COMPA_IRQ 16 >> +#define TIMER1_COMPB_IRQ 17 >> +#define TIMER1_COMPC_IRQ 18 >> +#define TIMER1_OVF_IRQ 19 >> + >> +/* Power reduction */ >> +#define PRR1_BIT_PRTIM5 0x05 /* Timer/Counter5 */ >> +#define PRR1_BIT_PRTIM4 0x04 /* Timer/Counter4 */ >> +#define PRR1_BIT_PRTIM3 0x03 /* Timer/Counter3 */ >> +#define PRR1_BIT_PRUSART3 0x02 /* USART3 */ >> +#define PRR1_BIT_PRUSART2 0x01 /* USART2 */ >> +#define PRR1_BIT_PRUSART1 0x00 /* USART1 */ >> + >> +#define PRR0_BIT_PRTWI 0x06 /* TWI */ >> +#define PRR0_BIT_PRTIM2 0x05 /* Timer/Counter2 */ >> +#define PRR0_BIT_PRTIM0 0x04 /* Timer/Counter0 */ >> +#define PRR0_BIT_PRTIM1 0x03 /* Timer/Counter1 */ >> +#define PRR0_BIT_PRSPI 0x02 /* Serial Peripheral Interface */ >> +#define PRR0_BIT_PRUSART0 0x01 /* USART0 */ >> +#define PRR0_BIT_PRADC 0x00 /* ADC */ >> + >> +typedef struct { >> + MachineClass parent; >> +} SampleMachineClass; >> + >> +typedef struct { >> + MachineState parent; >> + MemoryRegion *ram; >> + MemoryRegion *flash; >> + AVRUsartState *usart0; >> + AVRTimer16State *timer1; >> + AVRMaskState *prr[2]; >> +} SampleMachineState; >> + >> +#define TYPE_SAMPLE_MACHINE MACHINE_TYPE_NAME("sample") >> + >> +#define SAMPLE_MACHINE(obj) \ >> + OBJECT_CHECK(SampleMachineState, obj, TYPE_SAMPLE_MACHINE) >> +#define SAMPLE_MACHINE_GET_CLASS(obj) \ >> + OBJECT_GET_CLASS(SampleMachineClass, obj, TYPE_SAMPLE_MACHINE) >> +#define SAMPLE_MACHINE_CLASS(klass) \ >> + OBJECT_CLASS_CHECK(SampleMachineClass, klass, TYPE_SAMPLE_MACHINE) >> + >> +static void sample_init(MachineState *machine) >> +{ >> + SampleMachineState *sms = SAMPLE_MACHINE(machine); >> + MemoryRegion *system_memory = get_system_memory(); >> + AVRCPU *cpu; >> + const char *firmware = NULL; >> + const char *filename; >> + int bytes_loaded; >> + SysBusDevice *busdev; >> + DeviceState *cpudev; >> + >> + system_memory = get_system_memory(); >> + sms->ram = g_new(MemoryRegion, 1); >> + sms->flash = g_new(MemoryRegion, 1); >> + >> + cpu = AVR_CPU(cpu_create(machine->cpu_type)); >> + cpudev = DEVICE(cpu); >> + >> + >> + memory_region_init_rom(sms->flash, NULL, "avr.flash", SIZE_FLASH, >> + &error_fatal); >> + memory_region_add_subregion(system_memory, OFFSET_CODE, sms->flash); >> + >> + /* following are atmel2560 device */ >> + create_unimplemented_device("usart 3", OFFSET_DATA + 0x0130, 0x0007); >> + create_unimplemented_device("timer-counter-16bit 5", >> + OFFSET_DATA + 0x0120, 0x000e); >> + create_unimplemented_device("gpio L", OFFSET_DATA + 0x0109, 0x0003); >> + create_unimplemented_device("gpio K", OFFSET_DATA + 0x0106, 0x0003); >> + create_unimplemented_device("gpio J", OFFSET_DATA + 0x0103, 0x0003); >> + create_unimplemented_device("gpio H", OFFSET_DATA + 0x0100, 0x0003); >> + create_unimplemented_device("usart 2", OFFSET_DATA + 0x00d0, 0x0007); >> + create_unimplemented_device("usart 1", OFFSET_DATA + 0x00c8, 0x0007); >> + create_unimplemented_device("usart 0", OFFSET_DATA + 0x00c0, 0x0007); >> + create_unimplemented_device("twi", OFFSET_DATA + 0x00b8, 0x0006); >> + create_unimplemented_device("timer-counter-async-8bit 2", >> + OFFSET_DATA + 0x00b0, 0x0007); >> + create_unimplemented_device("timer-counter-16bit 4", >> + OFFSET_DATA + 0x00a0, 0x000e); >> + create_unimplemented_device("timer-counter-16bit 3", >> + OFFSET_DATA + 0x0090, 0x000e); >> + create_unimplemented_device("timer-counter-16bit 1", >> + OFFSET_DATA + 0x0080, 0x000e); >> + create_unimplemented_device("ac / adc", >> + OFFSET_DATA + 0x0078, 0x0008); >> + create_unimplemented_device("ext-mem-iface", >> + OFFSET_DATA + 0x0074, 0x0002); >> + create_unimplemented_device("int-controller", >> + OFFSET_DATA + 0x0068, 0x000c); >> + create_unimplemented_device("sys", >> + OFFSET_DATA + 0x0060, 0x0007); >> + create_unimplemented_device("spi", >> + OFFSET_DATA + 0x004c, 0x0003); >> + create_unimplemented_device("ext-mem-iface", >> + OFFSET_DATA + 0x004a, 0x0002); >> + create_unimplemented_device("timer-counter-pwm-8bit 0", >> + OFFSET_DATA + 0x0043, 0x0006); >> + create_unimplemented_device("ext-mem-iface", >> + OFFSET_DATA + 0x003e, 0x0005); >> + create_unimplemented_device("int-controller", >> + OFFSET_DATA + 0x0035, 0x0009); >> + create_unimplemented_device("gpio G", OFFSET_DATA + 0x0032, 0x0003); >> + create_unimplemented_device("gpio F", OFFSET_DATA + 0x002f, 0x0003); >> + create_unimplemented_device("gpio E", OFFSET_DATA + 0x002c, 0x0003); >> + create_unimplemented_device("gpio D", OFFSET_DATA + 0x0029, 0x0003); >> + create_unimplemented_device("gpio C", OFFSET_DATA + 0x0026, 0x0003); >> + create_unimplemented_device("gpio B", OFFSET_DATA + 0x0023, 0x0003); >> + create_unimplemented_device("gpio A", OFFSET_DATA + 0x0020, 0x0003); >> + >> + memory_region_allocate_system_memory( >> + sms->ram, NULL, "avr.ram", SIZE_SRAM + SIZE_EXMEM); >> + memory_region_add_subregion(system_memory, OFFSET_DATA + 0x200, >> sms->ram); >> + >> + /* Power Reduction built-in peripheral */ >> + sms->prr[0] = AVR_MASK(sysbus_create_simple(TYPE_AVR_MASK, >> + OFFSET_DATA + PRR0_BASE, NULL)); >> + sms->prr[1] = AVR_MASK(sysbus_create_simple(TYPE_AVR_MASK, >> + OFFSET_DATA + PRR1_BASE, NULL)); >> + >> + /* USART 0 built-in peripheral */ >> + sms->usart0 = AVR_USART(object_new(TYPE_AVR_USART)); >> + busdev = SYS_BUS_DEVICE(sms->usart0); >> + qdev_prop_set_chr(DEVICE(sms->usart0), "chardev", serial_hd(0)); >> + object_property_set_bool(OBJECT(sms->usart0), true, "realized", >> + &error_fatal); >> + sysbus_mmio_map(busdev, 0, OFFSET_DATA + USART_BASE); >> + /* >> + * These IRQ numbers don't match the datasheet because we're >> counting from >> + * zero and not including reset. >> + */ >> + sysbus_connect_irq(busdev, 0, qdev_get_gpio_in(cpudev, >> USART_RXC_IRQ)); >> + sysbus_connect_irq(busdev, 1, qdev_get_gpio_in(cpudev, >> USART_DRE_IRQ)); >> + sysbus_connect_irq(busdev, 2, qdev_get_gpio_in(cpudev, >> USART_TXC_IRQ)); >> + sysbus_connect_irq(SYS_BUS_DEVICE(sms->prr[1]), PRR1_BIT_PRUSART1, >> + qdev_get_gpio_in(DEVICE(sms->usart0), 0)); >> + >> + /* Timer 1 built-in periphal */ >> + sms->timer1 = AVR_TIMER16(object_new(TYPE_AVR_TIMER16)); >> + object_property_set_bool(OBJECT(sms->timer1), true, "realized", >> + &error_fatal); >> + busdev = SYS_BUS_DEVICE(sms->timer1); >> + sysbus_mmio_map(busdev, 0, OFFSET_DATA + TIMER1_BASE); >> + sysbus_mmio_map(busdev, 1, OFFSET_DATA + TIMER1_IMSK_BASE); >> + sysbus_mmio_map(busdev, 2, OFFSET_DATA + TIMER1_IFR_BASE); >> + sysbus_connect_irq(busdev, 0, qdev_get_gpio_in(cpudev, >> TIMER1_CAPT_IRQ)); >> + sysbus_connect_irq(busdev, 1, qdev_get_gpio_in(cpudev, >> TIMER1_COMPA_IRQ)); >> + sysbus_connect_irq(busdev, 2, qdev_get_gpio_in(cpudev, >> TIMER1_COMPB_IRQ)); >> + sysbus_connect_irq(busdev, 3, qdev_get_gpio_in(cpudev, >> TIMER1_COMPC_IRQ)); >> + sysbus_connect_irq(busdev, 4, qdev_get_gpio_in(cpudev, >> TIMER1_OVF_IRQ)); >> + sysbus_connect_irq(SYS_BUS_DEVICE(sms->prr[0]), PRR0_BIT_PRTIM1, >> + qdev_get_gpio_in(DEVICE(sms->timer1), 0)); >> + >> + /* Load firmware (contents of flash) trying to auto-detect format */ >> + firmware = machine->firmware; >> + if (firmware != NULL) { >> + filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, firmware); >> + if (filename == NULL) { >> + error_report("Unable to find %s", firmware); >> + exit(1); >> + } >> + >> + bytes_loaded = load_elf( >> + filename, NULL, NULL, NULL, NULL, NULL, NULL, 0, EM_NONE, 0, >> 0); >> + if (bytes_loaded < 0) { >> + bytes_loaded = load_image_targphys( >> + filename, OFFSET_CODE, SIZE_FLASH); >> + } >> + if (bytes_loaded < 0) { >> + error_report( >> + "Unable to load firmware image %s as ELF or raw binary", >> + firmware); >> + exit(1); >> + } >> + } >> +} >> + >> +static void sample_class_init(ObjectClass *oc, void *data) >> +{ >> + MachineClass *mc = MACHINE_CLASS(oc); >> + >> + mc->desc = "AVR sample/example board (ATmega2560)"; >> + mc->init = sample_init; >> + mc->default_cpus = 1; >> + mc->min_cpus = mc->default_cpus; >> + mc->max_cpus = mc->default_cpus; >> + mc->default_cpu_type = "avr6-avr-cpu"; /* ATmega2560. */ >> + mc->is_default = 1; >> +} >> + >> +static const TypeInfo sample_info = { >> + .name = TYPE_SAMPLE_MACHINE, >> + .parent = TYPE_MACHINE, >> + .instance_size = sizeof(SampleMachineState), >> + .class_size = sizeof(SampleMachineClass), >> + .class_init = sample_class_init, >> +}; >> + >> +static void sample_machine_init(void) >> +{ >> + type_register_static(&sample_info); >> +} >> + >> +type_init(sample_machine_init); >> diff --git a/hw/Kconfig b/hw/Kconfig >> index b9685b3944..07b8abb342 100644 >> --- a/hw/Kconfig >> +++ b/hw/Kconfig >> @@ -44,6 +44,7 @@ source watchdog/Kconfig >> # arch Kconfig >> source arm/Kconfig >> source alpha/Kconfig >> +source avr/Kconfig >> source cris/Kconfig >> source hppa/Kconfig >> source i386/Kconfig >> diff --git a/hw/avr/Kconfig b/hw/avr/Kconfig >> new file mode 100644 >> index 0000000000..92aa1e6afb >> --- /dev/null >> +++ b/hw/avr/Kconfig >> @@ -0,0 +1,6 @@ >> +config AVR_SAMPLE >> + bool >> + select AVR_TIMER16 >> + select AVR_USART >> + select AVR_MASK >> + select UNIMP >> diff --git a/hw/avr/Makefile.objs b/hw/avr/Makefile.objs >> new file mode 100644 >> index 0000000000..626b7064b3 >> --- /dev/null >> +++ b/hw/avr/Makefile.objs >> @@ -0,0 +1 @@ >> +obj-y += sample.o >> -- >> 2.17.2 (Apple Git-113) >> >> -- Best Regards, Michael Rolnik