| Message ID | 1491736758-19540-5-git-send-email-sundeep.lkml@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Sun, Apr 9, 2017 at 4:19 AM, Subbaraya Sundeep <sundeep.lkml@gmail.com> wrote: > Emulated Emcraft's Smartfusion2 System On Module starter > kit. > > Signed-off-by: Subbaraya Sundeep <sundeep.lkml@gmail.com> Hey Sundeep, Cool patch, I have some comments below. > --- > default-configs/arm-softmmu.mak | 1 + > hw/arm/Makefile.objs | 2 +- > hw/arm/msf2_soc.c | 141 ++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 143 insertions(+), 1 deletion(-) > create mode 100644 hw/arm/msf2_soc.c > > diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak > index 1e3bd2b..379f7e1 100644 > --- a/default-configs/arm-softmmu.mak > +++ b/default-configs/arm-softmmu.mak > @@ -121,3 +121,4 @@ CONFIG_ACPI=y > CONFIG_SMBIOS=y > CONFIG_ASPEED_SOC=y > CONFIG_GPIO_KEY=y > +CONFIG_MSF2=y > diff --git a/hw/arm/Makefile.objs b/hw/arm/Makefile.objs > index 4c5c4ee..cce2759 100644 > --- a/hw/arm/Makefile.objs > +++ b/hw/arm/Makefile.objs > @@ -1,7 +1,7 @@ > obj-y += boot.o collie.o exynos4_boards.o gumstix.o highbank.o > obj-$(CONFIG_DIGIC) += digic_boards.o > obj-y += integratorcp.o mainstone.o musicpal.o nseries.o > -obj-y += omap_sx1.o palm.o realview.o spitz.o stellaris.o > +obj-y += omap_sx1.o palm.o realview.o spitz.o stellaris.o msf2_soc.o > obj-y += tosa.o versatilepb.o vexpress.o virt.o xilinx_zynq.o z2.o > obj-$(CONFIG_ACPI) += virt-acpi-build.o > obj-y += netduino2.o > diff --git a/hw/arm/msf2_soc.c b/hw/arm/msf2_soc.c > new file mode 100644 > index 0000000..7277b51 > --- /dev/null > +++ b/hw/arm/msf2_soc.c > @@ -0,0 +1,141 @@ > +/* > + * Smartfusion2 SOM starter kit(from Emcraft) emulation. > + * > + * Copyright (c) 2017 Subbaraya Sundeep <sundeep.lkml@gmail.com> > + * > + * Permission is hereby granted, free of charge, to any person obtaining a copy > + * of this software and associated documentation files (the "Software"), to deal > + * in the Software without restriction, including without limitation the rights > + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell > + * copies of the Software, and to permit persons to whom the Software is > + * furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice shall be included in > + * all copies or substantial portions of the Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN > + * THE SOFTWARE. > + */ > + > +#include "qemu/osdep.h" > +#include "qapi/error.h" > +#include "qemu-common.h" > +#include "hw/arm/arm.h" > +#include "exec/address-spaces.h" > +#include "hw/sysbus.h" > +#include "hw/char/serial.h" > +#include "hw/boards.h" > +#include "sysemu/block-backend.h" > +#include "hw/ssi/ssi.h" > + > +#define MSF2_NUM_USARTS 1 > +#define MSF2_NUM_TIMERS 2 > + > +#define ENVM_BASE_ADDRESS 0x60000000 > +#define ENVM_SIZE (128 * 1024) > + > +#define DDR_BASE_ADDRESS 0xA0000000 > +#define DDR_SIZE (64 * 1024 * 1024) > + > +#define SRAM_BASE_ADDRESS 0x20000000 > +#define SRAM_SIZE (64 * 1024) > + > +#define MSF2_TIMER_BASE 0x40004000 > +#define MSF2_UART0_BASE 0x40000000 > +#define MSF2_SYSREG_BASE 0x40038000 > +#define MSF2_SPI0_BASE 0x40001000 > + > +#define MSF2_SPI0_IRQ 2 > +#define MSF2_UART0_IRQ 10 > +#define MSF2_TIMER_IRQ0 14 > +#define MSF2_TIMER_IRQ1 15 > + > +static void msf2_init(MachineState *machine) > +{ > + const char *kernel_filename = NULL; > + DeviceState *dev, *nvic; > + MemoryRegion *system_memory = get_system_memory(); > + MemoryRegion *nvm = g_new(MemoryRegion, 1); > + MemoryRegion *nvm_alias = g_new(MemoryRegion, 1); > + MemoryRegion *sram = g_new(MemoryRegion, 1); > + MemoryRegion *ddr = g_new(MemoryRegion, 1); > + QemuOpts *machine_opts = qemu_get_machine_opts(); > + SysBusDevice *busdev; > + DriveInfo *dinfo = drive_get_next(IF_MTD); > + qemu_irq cs_line; > + SSIBus *spi; > + > + kernel_filename = qemu_opt_get(machine_opts, "kernel"); You can just get the kernel filename directly from: machine->kernel_filename > + > + memory_region_init_ram(nvm, NULL, "MSF2.envm", ENVM_SIZE, > + &error_fatal); > + memory_region_init_alias(nvm_alias, NULL, "MSF2.flash.alias", > + nvm, 0, ENVM_SIZE); > + vmstate_register_ram_global(nvm); > + > + memory_region_set_readonly(nvm, true); > + memory_region_set_readonly(nvm_alias, true); > + > + memory_region_add_subregion(system_memory, ENVM_BASE_ADDRESS, nvm); > + memory_region_add_subregion(system_memory, 0, nvm_alias); > + > + memory_region_init_ram(ddr, NULL, "MSF2.ddr", DDR_SIZE, > + &error_fatal); > + vmstate_register_ram_global(ddr); > + memory_region_add_subregion(system_memory, DDR_BASE_ADDRESS, ddr); > + > + memory_region_init_ram(sram, NULL, "MSF2.sram", SRAM_SIZE, > + &error_fatal); > + vmstate_register_ram_global(sram); > + memory_region_add_subregion(system_memory, SRAM_BASE_ADDRESS, sram); > + > + nvic = armv7m_init(system_memory, ENVM_SIZE, 96, > + kernel_filename, "cortex-m3"); > + > + if (serial_hds[0]) { > + serial_mm_init(get_system_memory(), MSF2_UART0_BASE, 2, > + qdev_get_gpio_in(nvic, MSF2_UART0_IRQ), > + 115200, serial_hds[0], DEVICE_NATIVE_ENDIAN); > + } > + dev = qdev_create(NULL, "msf2-timer"); > + qdev_prop_set_uint32(dev, "clock-frequency", 83 * 1000000); > + qdev_init_nofail(dev); > + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, MSF2_TIMER_BASE); > + sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0, > + qdev_get_gpio_in(nvic, MSF2_TIMER_IRQ0)); > + sysbus_connect_irq(SYS_BUS_DEVICE(dev), 1, > + qdev_get_gpio_in(nvic, MSF2_TIMER_IRQ1)); > + > + dev = qdev_create(NULL, "msf2-sysreg"); > + qdev_init_nofail(dev); > + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, MSF2_SYSREG_BASE); > + > + dev = qdev_create(NULL, "msf2-spi"); > + qdev_init_nofail(dev); > + busdev = SYS_BUS_DEVICE(dev); > + sysbus_mmio_map(busdev, 0, MSF2_SPI0_BASE); > + sysbus_connect_irq(busdev, 0, qdev_get_gpio_in(nvic, MSF2_SPI0_IRQ)); > + > + spi = (SSIBus *)qdev_get_child_bus(dev, "spi"); > + dev = ssi_create_slave_no_init(spi, "s25sl12801"); > + qdev_prop_set_uint8(dev, "spansion-cr2nv", 1); > + if (dinfo) > + qdev_prop_set_drive(dev, "drive", blk_by_legacy_dinfo(dinfo), > + &error_fatal); > + qdev_init_nofail(dev); > + cs_line = qdev_get_gpio_in_named(dev, SSI_GPIO_CS, 0); > + sysbus_connect_irq(busdev, 1, cs_line); Instead of calling all of these in the init function you should split it up over the machines init and realize function. Look at the stm32f205_soc or xlnx-zynqmp files for examples of how to do this. It also moves away from calling qdev_create() and qdev_init_nofail() and instead manually creates the objects. Otherwise this patch looks pretty good. Thanks, Alistair > +} > + > +static void msf2_machine_init(MachineClass *mc) > +{ > + mc->desc = "SmartFusion2 SOM kit from Emcraft"; > + mc->init = msf2_init; > +} > + > +DEFINE_MACHINE("smartfusion2-som", msf2_machine_init) > -- > 2.5.0 > >
Hii Alistair, On Sat, Apr 15, 2017 at 2:42 AM, Alistair Francis <alistair23@gmail.com> wrote: > On Sun, Apr 9, 2017 at 4:19 AM, Subbaraya Sundeep > <sundeep.lkml@gmail.com> wrote: >> Emulated Emcraft's Smartfusion2 System On Module starter >> kit. >> >> Signed-off-by: Subbaraya Sundeep <sundeep.lkml@gmail.com> > > Hey Sundeep, > > Cool patch, I have some comments below. > >> --- >> default-configs/arm-softmmu.mak | 1 + >> hw/arm/Makefile.objs | 2 +- >> hw/arm/msf2_soc.c | 141 ++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 143 insertions(+), 1 deletion(-) >> create mode 100644 hw/arm/msf2_soc.c >> >> diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak >> index 1e3bd2b..379f7e1 100644 >> --- a/default-configs/arm-softmmu.mak >> +++ b/default-configs/arm-softmmu.mak >> @@ -121,3 +121,4 @@ CONFIG_ACPI=y >> CONFIG_SMBIOS=y >> CONFIG_ASPEED_SOC=y >> CONFIG_GPIO_KEY=y >> +CONFIG_MSF2=y >> diff --git a/hw/arm/Makefile.objs b/hw/arm/Makefile.objs >> index 4c5c4ee..cce2759 100644 >> --- a/hw/arm/Makefile.objs >> +++ b/hw/arm/Makefile.objs >> @@ -1,7 +1,7 @@ >> obj-y += boot.o collie.o exynos4_boards.o gumstix.o highbank.o >> obj-$(CONFIG_DIGIC) += digic_boards.o >> obj-y += integratorcp.o mainstone.o musicpal.o nseries.o >> -obj-y += omap_sx1.o palm.o realview.o spitz.o stellaris.o >> +obj-y += omap_sx1.o palm.o realview.o spitz.o stellaris.o msf2_soc.o >> obj-y += tosa.o versatilepb.o vexpress.o virt.o xilinx_zynq.o z2.o >> obj-$(CONFIG_ACPI) += virt-acpi-build.o >> obj-y += netduino2.o >> diff --git a/hw/arm/msf2_soc.c b/hw/arm/msf2_soc.c >> new file mode 100644 >> index 0000000..7277b51 >> --- /dev/null >> +++ b/hw/arm/msf2_soc.c >> @@ -0,0 +1,141 @@ >> +/* >> + * Smartfusion2 SOM starter kit(from Emcraft) emulation. >> + * >> + * Copyright (c) 2017 Subbaraya Sundeep <sundeep.lkml@gmail.com> >> + * >> + * Permission is hereby granted, free of charge, to any person obtaining a copy >> + * of this software and associated documentation files (the "Software"), to deal >> + * in the Software without restriction, including without limitation the rights >> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell >> + * copies of the Software, and to permit persons to whom the Software is >> + * furnished to do so, subject to the following conditions: >> + * >> + * The above copyright notice and this permission notice shall be included in >> + * all copies or substantial portions of the Software. >> + * >> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR >> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, >> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL >> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER >> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, >> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN >> + * THE SOFTWARE. >> + */ >> + >> +#include "qemu/osdep.h" >> +#include "qapi/error.h" >> +#include "qemu-common.h" >> +#include "hw/arm/arm.h" >> +#include "exec/address-spaces.h" >> +#include "hw/sysbus.h" >> +#include "hw/char/serial.h" >> +#include "hw/boards.h" >> +#include "sysemu/block-backend.h" >> +#include "hw/ssi/ssi.h" >> + >> +#define MSF2_NUM_USARTS 1 >> +#define MSF2_NUM_TIMERS 2 >> + >> +#define ENVM_BASE_ADDRESS 0x60000000 >> +#define ENVM_SIZE (128 * 1024) >> + >> +#define DDR_BASE_ADDRESS 0xA0000000 >> +#define DDR_SIZE (64 * 1024 * 1024) >> + >> +#define SRAM_BASE_ADDRESS 0x20000000 >> +#define SRAM_SIZE (64 * 1024) >> + >> +#define MSF2_TIMER_BASE 0x40004000 >> +#define MSF2_UART0_BASE 0x40000000 >> +#define MSF2_SYSREG_BASE 0x40038000 >> +#define MSF2_SPI0_BASE 0x40001000 >> + >> +#define MSF2_SPI0_IRQ 2 >> +#define MSF2_UART0_IRQ 10 >> +#define MSF2_TIMER_IRQ0 14 >> +#define MSF2_TIMER_IRQ1 15 >> + >> +static void msf2_init(MachineState *machine) >> +{ >> + const char *kernel_filename = NULL; >> + DeviceState *dev, *nvic; >> + MemoryRegion *system_memory = get_system_memory(); >> + MemoryRegion *nvm = g_new(MemoryRegion, 1); >> + MemoryRegion *nvm_alias = g_new(MemoryRegion, 1); >> + MemoryRegion *sram = g_new(MemoryRegion, 1); >> + MemoryRegion *ddr = g_new(MemoryRegion, 1); >> + QemuOpts *machine_opts = qemu_get_machine_opts(); >> + SysBusDevice *busdev; >> + DriveInfo *dinfo = drive_get_next(IF_MTD); >> + qemu_irq cs_line; >> + SSIBus *spi; >> + >> + kernel_filename = qemu_opt_get(machine_opts, "kernel"); > > You can just get the kernel filename directly from: machine->kernel_filename > I have to check again. I remember getting NULL from machine->kernel_filename. I will comment on this again. >> + >> + memory_region_init_ram(nvm, NULL, "MSF2.envm", ENVM_SIZE, >> + &error_fatal); >> + memory_region_init_alias(nvm_alias, NULL, "MSF2.flash.alias", >> + nvm, 0, ENVM_SIZE); >> + vmstate_register_ram_global(nvm); >> + >> + memory_region_set_readonly(nvm, true); >> + memory_region_set_readonly(nvm_alias, true); >> + >> + memory_region_add_subregion(system_memory, ENVM_BASE_ADDRESS, nvm); >> + memory_region_add_subregion(system_memory, 0, nvm_alias); >> + >> + memory_region_init_ram(ddr, NULL, "MSF2.ddr", DDR_SIZE, >> + &error_fatal); >> + vmstate_register_ram_global(ddr); >> + memory_region_add_subregion(system_memory, DDR_BASE_ADDRESS, ddr); >> + >> + memory_region_init_ram(sram, NULL, "MSF2.sram", SRAM_SIZE, >> + &error_fatal); >> + vmstate_register_ram_global(sram); >> + memory_region_add_subregion(system_memory, SRAM_BASE_ADDRESS, sram); >> + >> + nvic = armv7m_init(system_memory, ENVM_SIZE, 96, >> + kernel_filename, "cortex-m3"); >> + >> + if (serial_hds[0]) { >> + serial_mm_init(get_system_memory(), MSF2_UART0_BASE, 2, >> + qdev_get_gpio_in(nvic, MSF2_UART0_IRQ), >> + 115200, serial_hds[0], DEVICE_NATIVE_ENDIAN); >> + } >> + dev = qdev_create(NULL, "msf2-timer"); >> + qdev_prop_set_uint32(dev, "clock-frequency", 83 * 1000000); >> + qdev_init_nofail(dev); >> + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, MSF2_TIMER_BASE); >> + sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0, >> + qdev_get_gpio_in(nvic, MSF2_TIMER_IRQ0)); >> + sysbus_connect_irq(SYS_BUS_DEVICE(dev), 1, >> + qdev_get_gpio_in(nvic, MSF2_TIMER_IRQ1)); >> + >> + dev = qdev_create(NULL, "msf2-sysreg"); >> + qdev_init_nofail(dev); >> + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, MSF2_SYSREG_BASE); >> + >> + dev = qdev_create(NULL, "msf2-spi"); >> + qdev_init_nofail(dev); >> + busdev = SYS_BUS_DEVICE(dev); >> + sysbus_mmio_map(busdev, 0, MSF2_SPI0_BASE); >> + sysbus_connect_irq(busdev, 0, qdev_get_gpio_in(nvic, MSF2_SPI0_IRQ)); >> + >> + spi = (SSIBus *)qdev_get_child_bus(dev, "spi"); >> + dev = ssi_create_slave_no_init(spi, "s25sl12801"); >> + qdev_prop_set_uint8(dev, "spansion-cr2nv", 1); >> + if (dinfo) >> + qdev_prop_set_drive(dev, "drive", blk_by_legacy_dinfo(dinfo), >> + &error_fatal); >> + qdev_init_nofail(dev); >> + cs_line = qdev_get_gpio_in_named(dev, SSI_GPIO_CS, 0); >> + sysbus_connect_irq(busdev, 1, cs_line); > > Instead of calling all of these in the init function you should split > it up over the machines init and realize function. > > Look at the stm32f205_soc or xlnx-zynqmp files for examples of how to do this. > > It also moves away from calling qdev_create() and qdev_init_nofail() > and instead manually creates the objects. > I am still learning all these. Please correct me if am wrong. I need to create a SoC file and a board file like stm32f205 and xlnx-zynqmp now right? > Otherwise this patch looks pretty good. Thank you :) Sundeep > > Thanks, > > Alistair > >> +} >> + >> +static void msf2_machine_init(MachineClass *mc) >> +{ >> + mc->desc = "SmartFusion2 SOM kit from Emcraft"; >> + mc->init = msf2_init; >> +} >> + >> +DEFINE_MACHINE("smartfusion2-som", msf2_machine_init) >> -- >> 2.5.0 >> >>
>> >> Instead of calling all of these in the init function you should split >> it up over the machines init and realize function. >> >> Look at the stm32f205_soc or xlnx-zynqmp files for examples of how to do this. >> >> It also moves away from calling qdev_create() and qdev_init_nofail() >> and instead manually creates the objects. >> > I am still learning all these. Please correct me if am wrong. > I need to create a SoC file and a board file like stm32f205 and > xlnx-zynqmp now right? Hey Sundeep, I don't think you have to do it like that. I think for some SoCs/boards it makes sense. For example the ZynqMP SoCs are included on multiple different boards (EP108 and ZCU102) so it makes sense to have a SoC and a board separately defined. On the other hand if you had a SoC that is always on the same board it doesn't make as much sense. It is probably is a good idea to split it between a board and an SoC unless you have a good reason not to though. Thanks, Alistair > >> Otherwise this patch looks pretty good. > > Thank you :) > Sundeep > >> >> Thanks, >> >> Alistair >> >>> +} >>> + >>> +static void msf2_machine_init(MachineClass *mc) >>> +{ >>> + mc->desc = "SmartFusion2 SOM kit from Emcraft"; >>> + mc->init = msf2_init; >>> +} >>> + >>> +DEFINE_MACHINE("smartfusion2-som", msf2_machine_init) >>> -- >>> 2.5.0 >>> >>>
Hi Alistair, On Mon, Apr 24, 2017 at 11:23 PM, Alistair Francis <alistair23@gmail.com> wrote: >>> >>> Instead of calling all of these in the init function you should split >>> it up over the machines init and realize function. >>> >>> Look at the stm32f205_soc or xlnx-zynqmp files for examples of how to do this. >>> >>> It also moves away from calling qdev_create() and qdev_init_nofail() >>> and instead manually creates the objects. >>> >> I am still learning all these. Please correct me if am wrong. >> I need to create a SoC file and a board file like stm32f205 and >> xlnx-zynqmp now right? > > Hey Sundeep, > > I don't think you have to do it like that. I think for some > SoCs/boards it makes sense. For example the ZynqMP SoCs are included > on multiple different boards (EP108 and ZCU102) so it makes sense to > have a SoC and a board separately defined. On the other hand if you > had a SoC that is always on the same board it doesn't make as much > sense. > > It is probably is a good idea to split it between a board and an SoC > unless you have a good reason not to though. There are multiples boards with the SoC. I will split into seperate files. Thanks, Sundeep > > Thanks, > > Alistair > >> >>> Otherwise this patch looks pretty good. >> >> Thank you :) >> Sundeep >> >>> >>> Thanks, >>> >>> Alistair >>> >>>> +} >>>> + >>>> +static void msf2_machine_init(MachineClass *mc) >>>> +{ >>>> + mc->desc = "SmartFusion2 SOM kit from Emcraft"; >>>> + mc->init = msf2_init; >>>> +} >>>> + >>>> +DEFINE_MACHINE("smartfusion2-som", msf2_machine_init) >>>> -- >>>> 2.5.0 >>>> >>>>
diff --git a/default-configs/arm-softmmu.mak b/default-configs/arm-softmmu.mak index 1e3bd2b..379f7e1 100644 --- a/default-configs/arm-softmmu.mak +++ b/default-configs/arm-softmmu.mak @@ -121,3 +121,4 @@ CONFIG_ACPI=y CONFIG_SMBIOS=y CONFIG_ASPEED_SOC=y CONFIG_GPIO_KEY=y +CONFIG_MSF2=y diff --git a/hw/arm/Makefile.objs b/hw/arm/Makefile.objs index 4c5c4ee..cce2759 100644 --- a/hw/arm/Makefile.objs +++ b/hw/arm/Makefile.objs @@ -1,7 +1,7 @@ obj-y += boot.o collie.o exynos4_boards.o gumstix.o highbank.o obj-$(CONFIG_DIGIC) += digic_boards.o obj-y += integratorcp.o mainstone.o musicpal.o nseries.o -obj-y += omap_sx1.o palm.o realview.o spitz.o stellaris.o +obj-y += omap_sx1.o palm.o realview.o spitz.o stellaris.o msf2_soc.o obj-y += tosa.o versatilepb.o vexpress.o virt.o xilinx_zynq.o z2.o obj-$(CONFIG_ACPI) += virt-acpi-build.o obj-y += netduino2.o diff --git a/hw/arm/msf2_soc.c b/hw/arm/msf2_soc.c new file mode 100644 index 0000000..7277b51 --- /dev/null +++ b/hw/arm/msf2_soc.c @@ -0,0 +1,141 @@ +/* + * Smartfusion2 SOM starter kit(from Emcraft) emulation. + * + * Copyright (c) 2017 Subbaraya Sundeep <sundeep.lkml@gmail.com> + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + */ + +#include "qemu/osdep.h" +#include "qapi/error.h" +#include "qemu-common.h" +#include "hw/arm/arm.h" +#include "exec/address-spaces.h" +#include "hw/sysbus.h" +#include "hw/char/serial.h" +#include "hw/boards.h" +#include "sysemu/block-backend.h" +#include "hw/ssi/ssi.h" + +#define MSF2_NUM_USARTS 1 +#define MSF2_NUM_TIMERS 2 + +#define ENVM_BASE_ADDRESS 0x60000000 +#define ENVM_SIZE (128 * 1024) + +#define DDR_BASE_ADDRESS 0xA0000000 +#define DDR_SIZE (64 * 1024 * 1024) + +#define SRAM_BASE_ADDRESS 0x20000000 +#define SRAM_SIZE (64 * 1024) + +#define MSF2_TIMER_BASE 0x40004000 +#define MSF2_UART0_BASE 0x40000000 +#define MSF2_SYSREG_BASE 0x40038000 +#define MSF2_SPI0_BASE 0x40001000 + +#define MSF2_SPI0_IRQ 2 +#define MSF2_UART0_IRQ 10 +#define MSF2_TIMER_IRQ0 14 +#define MSF2_TIMER_IRQ1 15 + +static void msf2_init(MachineState *machine) +{ + const char *kernel_filename = NULL; + DeviceState *dev, *nvic; + MemoryRegion *system_memory = get_system_memory(); + MemoryRegion *nvm = g_new(MemoryRegion, 1); + MemoryRegion *nvm_alias = g_new(MemoryRegion, 1); + MemoryRegion *sram = g_new(MemoryRegion, 1); + MemoryRegion *ddr = g_new(MemoryRegion, 1); + QemuOpts *machine_opts = qemu_get_machine_opts(); + SysBusDevice *busdev; + DriveInfo *dinfo = drive_get_next(IF_MTD); + qemu_irq cs_line; + SSIBus *spi; + + kernel_filename = qemu_opt_get(machine_opts, "kernel"); + + memory_region_init_ram(nvm, NULL, "MSF2.envm", ENVM_SIZE, + &error_fatal); + memory_region_init_alias(nvm_alias, NULL, "MSF2.flash.alias", + nvm, 0, ENVM_SIZE); + vmstate_register_ram_global(nvm); + + memory_region_set_readonly(nvm, true); + memory_region_set_readonly(nvm_alias, true); + + memory_region_add_subregion(system_memory, ENVM_BASE_ADDRESS, nvm); + memory_region_add_subregion(system_memory, 0, nvm_alias); + + memory_region_init_ram(ddr, NULL, "MSF2.ddr", DDR_SIZE, + &error_fatal); + vmstate_register_ram_global(ddr); + memory_region_add_subregion(system_memory, DDR_BASE_ADDRESS, ddr); + + memory_region_init_ram(sram, NULL, "MSF2.sram", SRAM_SIZE, + &error_fatal); + vmstate_register_ram_global(sram); + memory_region_add_subregion(system_memory, SRAM_BASE_ADDRESS, sram); + + nvic = armv7m_init(system_memory, ENVM_SIZE, 96, + kernel_filename, "cortex-m3"); + + if (serial_hds[0]) { + serial_mm_init(get_system_memory(), MSF2_UART0_BASE, 2, + qdev_get_gpio_in(nvic, MSF2_UART0_IRQ), + 115200, serial_hds[0], DEVICE_NATIVE_ENDIAN); + } + dev = qdev_create(NULL, "msf2-timer"); + qdev_prop_set_uint32(dev, "clock-frequency", 83 * 1000000); + qdev_init_nofail(dev); + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, MSF2_TIMER_BASE); + sysbus_connect_irq(SYS_BUS_DEVICE(dev), 0, + qdev_get_gpio_in(nvic, MSF2_TIMER_IRQ0)); + sysbus_connect_irq(SYS_BUS_DEVICE(dev), 1, + qdev_get_gpio_in(nvic, MSF2_TIMER_IRQ1)); + + dev = qdev_create(NULL, "msf2-sysreg"); + qdev_init_nofail(dev); + sysbus_mmio_map(SYS_BUS_DEVICE(dev), 0, MSF2_SYSREG_BASE); + + dev = qdev_create(NULL, "msf2-spi"); + qdev_init_nofail(dev); + busdev = SYS_BUS_DEVICE(dev); + sysbus_mmio_map(busdev, 0, MSF2_SPI0_BASE); + sysbus_connect_irq(busdev, 0, qdev_get_gpio_in(nvic, MSF2_SPI0_IRQ)); + + spi = (SSIBus *)qdev_get_child_bus(dev, "spi"); + dev = ssi_create_slave_no_init(spi, "s25sl12801"); + qdev_prop_set_uint8(dev, "spansion-cr2nv", 1); + if (dinfo) + qdev_prop_set_drive(dev, "drive", blk_by_legacy_dinfo(dinfo), + &error_fatal); + qdev_init_nofail(dev); + cs_line = qdev_get_gpio_in_named(dev, SSI_GPIO_CS, 0); + sysbus_connect_irq(busdev, 1, cs_line); +} + +static void msf2_machine_init(MachineClass *mc) +{ + mc->desc = "SmartFusion2 SOM kit from Emcraft"; + mc->init = msf2_init; +} + +DEFINE_MACHINE("smartfusion2-som", msf2_machine_init)
Emulated Emcraft's Smartfusion2 System On Module starter kit. Signed-off-by: Subbaraya Sundeep <sundeep.lkml@gmail.com> --- default-configs/arm-softmmu.mak | 1 + hw/arm/Makefile.objs | 2 +- hw/arm/msf2_soc.c | 141 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 143 insertions(+), 1 deletion(-) create mode 100644 hw/arm/msf2_soc.c