From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.0 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 10BA8C433DF for ; Thu, 9 Jul 2020 06:12:04 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C27D7206C3 for ; Thu, 9 Jul 2020 06:12:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="jpJBlRGG" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C27D7206C3 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=amsat.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:59644 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jtPmt-0005ox-35 for qemu-devel@archiver.kernel.org; Thu, 09 Jul 2020 02:12:03 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51654) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jtPmI-0005M5-U0; Thu, 09 Jul 2020 02:11:26 -0400 Received: from mail-wm1-x342.google.com ([2a00:1450:4864:20::342]:37384) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jtPmG-00070I-GM; Thu, 09 Jul 2020 02:11:26 -0400 Received: by mail-wm1-x342.google.com with SMTP id o2so589979wmh.2; Wed, 08 Jul 2020 23:11:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=AcrvAFzR3VEpJ0QfHT+z34lhL8ogRt2h+zhfuJ4IApI=; b=jpJBlRGGvcZqbQTbigW7msB/RP1A0zgYmiIprlF7/balOAI+J6VJvaiHvTHSimxTw9 FZejDeu1JbbKHWkXT7k5BYSaTphopQnfQMsEgXylFewO2QZK4/Jr44DACTudLL95pt/B 9HS3V+EVIeGHtvIMQf1y4MiILGHOYs7xUP7O3+igHY6osAfLQMT3Izg7ePtyf3w2oF+e 0Bd9BJovgIXGHBTz14NdFB8MI8gfTxRHSWIDUWCwSBNok4l9lASb+aYeo5B75v9XXt9g 1FfirOlM8Pq1UVio7lfKbBYMllcRx3MoQeyXpSNz6hQOpHtS1nNHrtQN/n8AFnENgCSs uqhg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:subject:to:cc:references:from:message-id :date:user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=AcrvAFzR3VEpJ0QfHT+z34lhL8ogRt2h+zhfuJ4IApI=; b=spWkuXSf8wYQfBa9HC01vsYLAko7aJIX0Jg5+vXIS3nbrwLlGvJi/VgFPBFG3GNFgA jxy97MePDrNSpxli9rO20lsUAoQliCMWBsZfQ0lUxlaMKdw+kXCZ5MsMAC7A8TtgYcjr i+eCTOR2MGSOcchBh6FYZWcrZDlg8lYG8bznvWRRrvJfZW6EoOe8U7BgWrI3qqljjHHX 1IFJKyY0sSSkbTOvFuecpu4o8+i4kh7XwXfmOqb4LJkP/5CEGvhRaMfkDUGHh9qpf5sJ w4ZH3/fqEOHK3jDbdHCbEybrXm4QqhuJh8xV8XU6qrwzvEhw0sHinz2m/8NWFpsiCzQh Dv7w== X-Gm-Message-State: AOAM532ctrB7LBX7CfLM386/q9nkddPo9HRNUQEHWncPznGRfA1Iti0m 5NGD9iuIGPnuG6/rEZFo45A= X-Google-Smtp-Source: ABdhPJxoBBpZZ6rdR+/5+1ND1OdcBPSj57qgIjWG/tRBtvDbISvgeK8j3esA7yP9Xr372OUFXzGEmw== X-Received: by 2002:a1c:de07:: with SMTP id v7mr12638925wmg.56.1594275082356; Wed, 08 Jul 2020 23:11:22 -0700 (PDT) Received: from [192.168.1.36] (138.red-83-57-170.dynamicip.rima-tde.net. [83.57.170.138]) by smtp.gmail.com with ESMTPSA id r1sm3718410wrw.24.2020.07.08.23.11.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 08 Jul 2020 23:11:21 -0700 (PDT) Subject: Re: [PATCH v5 04/11] hw/arm: Add NPCM730 and NPCM750 SoC models To: Havard Skinnemoen , peter.maydell@linaro.org References: <20200709003608.3834629-1-hskinnemoen@google.com> <20200709003608.3834629-5-hskinnemoen@google.com> From: =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= Message-ID: <21c5b350-69a0-598f-7ffd-221bac0ed6e0@amsat.org> Date: Thu, 9 Jul 2020 08:11:20 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 MIME-Version: 1.0 In-Reply-To: <20200709003608.3834629-5-hskinnemoen@google.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=2a00:1450:4864:20::342; envelope-from=philippe.mathieu.daude@gmail.com; helo=mail-wm1-x342.google.com X-detected-operating-system: by eggs.gnu.org: No matching host in p0f cache. That's all we know. X-Spam_score_int: 0 X-Spam_score: 0.0 X-Spam_bar: / X-Spam_report: (0.0 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FORGED_FROMDOMAIN=1, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kfting@nuvoton.com, qemu-arm@nongnu.org, Joel Stanley , qemu-devel@nongnu.org, Avi.Fishman@nuvoton.com Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On 7/9/20 2:36 AM, Havard Skinnemoen wrote: > The Nuvoton NPCM7xx SoC family are used to implement Baseboard > Management Controllers in servers. While the family includes four SoCs, > this patch implements limited support for two of them: NPCM730 (targeted > for Data Center applications) and NPCM750 (targeted for Enterprise > applications). > > This patch includes little more than the bare minimum needed to boot a > Linux kernel built with NPCM7xx support in direct-kernel mode: > > - Two Cortex-A9 CPU cores with built-in periperhals. > - Global Configuration Registers. > - Clock Management. > - 3 Timer Modules with 5 timers each. > - 4 serial ports. > > The chips themselves have a lot more features, some of which will be > added to the model at a later stage. > > Reviewed-by: Tyrone Ting > Reviewed-by: Joel Stanley > Signed-off-by: Havard Skinnemoen > --- > include/hw/arm/npcm7xx.h | 86 +++++++++ > hw/arm/npcm7xx.c | 376 +++++++++++++++++++++++++++++++++++++++ > hw/arm/Kconfig | 5 + > hw/arm/Makefile.objs | 1 + > 4 files changed, 468 insertions(+) > create mode 100644 include/hw/arm/npcm7xx.h > create mode 100644 hw/arm/npcm7xx.c > > diff --git a/include/hw/arm/npcm7xx.h b/include/hw/arm/npcm7xx.h > new file mode 100644 > index 0000000000..95d9224f59 > --- /dev/null > +++ b/include/hw/arm/npcm7xx.h > @@ -0,0 +1,86 @@ > +/* > + * Nuvoton NPCM7xx SoC family. > + * > + * Copyright 2020 Google LLC > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program 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 General Public License > + * for more details. > + */ > +#ifndef NPCM7XX_H > +#define NPCM7XX_H > + > +#include "hw/boards.h" > +#include "hw/cpu/a9mpcore.h" > +#include "hw/misc/npcm7xx_clk.h" > +#include "hw/misc/npcm7xx_gcr.h" > +#include "hw/timer/npcm7xx_timer.h" > +#include "target/arm/cpu.h" > + > +#define NPCM7XX_MAX_NUM_CPUS (2) > + > +/* The first half of the address space is reserved for DDR4 DRAM. */ > +#define NPCM7XX_DRAM_BA (0x00000000) > +#define NPCM7XX_DRAM_SZ (2 * GiB) > + > +/* Magic addresses for setting up direct kernel booting and SMP boot stubs. */ > +#define NPCM7XX_LOADER_START (0x00000000) /* Start of SDRAM */ > +#define NPCM7XX_SMP_LOADER_START (0xffff0000) /* Boot ROM */ > +#define NPCM7XX_SMP_BOOTREG_ADDR (0xf080013c) /* GCR.SCRPAD */ > +#define NPCM7XX_GIC_CPU_IF_ADDR (0xf03fe100) /* GIC within A9 */ > + > +typedef struct NPCM7xxState { > + DeviceState parent; > + > + ARMCPU cpu[NPCM7XX_MAX_NUM_CPUS]; > + A9MPPrivState a9mpcore; > + > + MemoryRegion sram; > + MemoryRegion irom; > + MemoryRegion ram3; > + MemoryRegion *dram; > + > + NPCM7xxGCRState gcr; > + NPCM7xxCLKState clk; > + NPCM7xxTimerCtrlState tim[3]; > +} NPCM7xxState; > + > +#define TYPE_NPCM7XX "npcm7xx" > +#define NPCM7XX(obj) OBJECT_CHECK(NPCM7xxState, (obj), TYPE_NPCM7XX) > + > +#define TYPE_NPCM730 "npcm730" > +#define TYPE_NPCM750 "npcm750" > + > +typedef struct NPCM7xxClass { > + DeviceClass parent; > + > + /* Bitmask of modules that are permanently disabled on this chip. */ > + uint32_t disabled_modules; > + /* Number of CPU cores enabled in this SoC class (may be 1 or 2). */ > + uint32_t num_cpus; > +} NPCM7xxClass; > + > +#define NPCM7XX_CLASS(klass) \ > + OBJECT_CLASS_CHECK(NPCM7xxClass, (klass), TYPE_NPCM7XX) > +#define NPCM7XX_GET_CLASS(obj) \ > + OBJECT_GET_CLASS(NPCM7xxClass, (obj), TYPE_NPCM7XX) > + > +/** > + * npcm7xx_write_secondary_boot - Write stub for booting secondary CPU. > + * @cpu: The CPU to be booted. > + * @info: Boot info structure for the board. > + * > + * This will write a short code stub to the internal ROM that will keep the > + * secondary CPU spinning until the primary CPU writes an address to the SCRPAD > + * register in the GCR, after which the secondary CPU will jump there. > + */ > +extern void npcm7xx_write_secondary_boot(ARMCPU *cpu, > + const struct arm_boot_info *info); extern?! :) > + > +#endif /* NPCM7XX_H */ > diff --git a/hw/arm/npcm7xx.c b/hw/arm/npcm7xx.c > new file mode 100644 > index 0000000000..af45f3c716 > --- /dev/null > +++ b/hw/arm/npcm7xx.c > @@ -0,0 +1,376 @@ > +/* > + * Nuvoton NPCM7xx SoC family. > + * > + * Copyright 2020 Google LLC > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms of the GNU General Public License as published by the > + * Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * This program 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 General Public License > + * for more details. > + */ > + > +#include "qemu/osdep.h" > + > +#include "exec/address-spaces.h" > +#include "hw/arm/npcm7xx.h" > +#include "hw/char/serial.h" > +#include "hw/loader.h" > +#include "hw/misc/unimp.h" > +#include "hw/qdev-properties.h" > +#include "qapi/error.h" > +#include "qemu/units.h" > +#include "sysemu/sysemu.h" > + > +/* > + * This covers the whole MMIO space. We'll use this to catch any MMIO accesses > + * that aren't handled by any device. > + */ > +#define NPCM7XX_MMIO_BA (0x80000000) > +#define NPCM7XX_MMIO_SZ (0x7ffd0000) > + > +/* Core system modules. */ > +#define NPCM7XX_L2C_BA (0xf03fc000) > +#define NPCM7XX_CPUP_BA (0xf03fe000) > +#define NPCM7XX_GCR_BA (0xf0800000) > +#define NPCM7XX_CLK_BA (0xf0801000) > + > +/* Internal AHB SRAM */ > +#define NPCM7XX_RAM3_BA (0xc0008000) > +#define NPCM7XX_RAM3_SZ (4 * KiB) > + > +/* Memory blocks at the end of the address space */ > +#define NPCM7XX_RAM2_BA (0xfffd0000) > +#define NPCM7XX_RAM2_SZ (128 * KiB) > +#define NPCM7XX_ROM_BA (0xffff0000) > +#define NPCM7XX_ROM_SZ (64 * KiB) > + > +/* > + * Interrupt lines going into the GIC. This does not include internal Cortex-A9 > + * interrupts. > + */ > +enum NPCM7xxInterrupt { > + NPCM7XX_UART0_IRQ = 2, > + NPCM7XX_UART1_IRQ, > + NPCM7XX_UART2_IRQ, > + NPCM7XX_UART3_IRQ, > + NPCM7XX_TIMER0_IRQ = 32, /* Timer Module 0 */ > + NPCM7XX_TIMER1_IRQ, > + NPCM7XX_TIMER2_IRQ, > + NPCM7XX_TIMER3_IRQ, > + NPCM7XX_TIMER4_IRQ, > + NPCM7XX_TIMER5_IRQ, /* Timer Module 1 */ > + NPCM7XX_TIMER6_IRQ, > + NPCM7XX_TIMER7_IRQ, > + NPCM7XX_TIMER8_IRQ, > + NPCM7XX_TIMER9_IRQ, > + NPCM7XX_TIMER10_IRQ, /* Timer Module 2 */ > + NPCM7XX_TIMER11_IRQ, > + NPCM7XX_TIMER12_IRQ, > + NPCM7XX_TIMER13_IRQ, > + NPCM7XX_TIMER14_IRQ, > +}; > + > +/* Total number of GIC interrupts, including internal Cortex-A9 interrupts. */ > +#define NPCM7XX_NUM_IRQ (160) > + > +/* Register base address for each Timer Module */ > +static const hwaddr npcm7xx_tim_addr[] = { > + 0xf0008000, > + 0xf0009000, > + 0xf000a000, > +}; > + > +/* Register base address for each 16550 UART */ > +static const hwaddr npcm7xx_uart_addr[] = { > + 0xf0001000, > + 0xf0002000, > + 0xf0003000, > + 0xf0004000, > +}; > + > +void npcm7xx_write_secondary_boot(ARMCPU *cpu, const struct arm_boot_info *info) So this confirms my comment from v4, instead I'd keep it static, and add npcm7xx_load_kernel(). > +{ > + /* > + * The default smpboot stub halts the secondary CPU with a 'wfi' > + * instruction, but the arch/arm/mach-npcm/platsmp.c in the Linux kernel > + * does not send an IPI to wake it up, so the second CPU fails to boot. So > + * we need to provide our own smpboot stub that can not use 'wfi', it has > + * to spin the secondary CPU until the first CPU writes to the SCRPAD reg. > + */ > + uint32_t smpboot[] = { > + 0xe59f2018, /* ldr r2, bootreg_addr */ > + 0xe3a00000, /* mov r0, #0 */ > + 0xe5820000, /* str r0, [r2] */ > + 0xe320f002, /* wfe */ > + 0xe5921000, /* ldr r1, [r2] */ > + 0xe1110001, /* tst r1, r1 */ > + 0x0afffffb, /* beq */ > + 0xe12fff11, /* bx r1 */ > + NPCM7XX_SMP_BOOTREG_ADDR, > + }; > + int i; > + > + for (i = 0; i < ARRAY_SIZE(smpboot); i++) { > + smpboot[i] = tswap32(smpboot[i]); > + } > + > + rom_add_blob_fixed("smpboot", smpboot, sizeof(smpboot), > + NPCM7XX_SMP_LOADER_START); > +} > + > +static qemu_irq npcm7xx_irq(NPCM7xxState *s, int n) > +{ > + return qdev_get_gpio_in(DEVICE(&s->a9mpcore), n); > +} > + > +static void npcm7xx_init(Object *obj) > +{ > + NPCM7xxState *s = NPCM7XX(obj); > + int i; > + > + for (i = 0; i < NPCM7XX_MAX_NUM_CPUS; i++) { > + object_initialize_child(obj, "cpu[*]", &s->cpu[i], > + ARM_CPU_TYPE_NAME("cortex-a9")); > + } > + > + object_initialize_child(obj, "a9mpcore", &s->a9mpcore, TYPE_A9MPCORE_PRIV); > + object_initialize_child(obj, "gcr", &s->gcr, TYPE_NPCM7XX_GCR); > + object_property_add_alias(obj, "power-on-straps", OBJECT(&s->gcr), > + "power-on-straps"); > + object_initialize_child(obj, "clk", &s->clk, TYPE_NPCM7XX_CLK); > + > + for (i = 0; i < ARRAY_SIZE(s->tim); i++) { > + object_initialize_child(obj, "tim[*]", &s->tim[i], TYPE_NPCM7XX_TIMER); > + } > +} > + > +static void npcm7xx_realize(DeviceState *dev, Error **errp) > +{ > + NPCM7xxState *s = NPCM7XX(dev); > + NPCM7xxClass *nc = NPCM7XX_GET_CLASS(s); > + int i; > + > + /* CPUs */ > + for (i = 0; i < nc->num_cpus; i++) { > + object_property_set_int(OBJECT(&s->cpu[i]), > + arm_cpu_mp_affinity(i, NPCM7XX_MAX_NUM_CPUS), > + "mp-affinity", &error_abort); > + object_property_set_int(OBJECT(&s->cpu[i]), NPCM7XX_GIC_CPU_IF_ADDR, > + "reset-cbar", &error_abort); > + object_property_set_bool(OBJECT(&s->cpu[i]), true, > + "reset-hivecs", &error_abort); > + > + /* Disable security extensions. */ > + object_property_set_bool(OBJECT(&s->cpu[i]), false, "has_el3", > + &error_abort); > + > + qdev_realize(DEVICE(&s->cpu[i]), NULL, &error_abort); > + } > + > + /* A9MPCORE peripherals */ > + object_property_set_int(OBJECT(&s->a9mpcore), nc->num_cpus, "num-cpu", > + &error_abort); > + object_property_set_int(OBJECT(&s->a9mpcore), NPCM7XX_NUM_IRQ, "num-irq", > + &error_abort); > + sysbus_realize(SYS_BUS_DEVICE(&s->a9mpcore), &error_abort); > + sysbus_mmio_map(SYS_BUS_DEVICE(&s->a9mpcore), 0, NPCM7XX_CPUP_BA); > + > + for (i = 0; i < nc->num_cpus; i++) { > + sysbus_connect_irq(SYS_BUS_DEVICE(&s->a9mpcore), i, > + qdev_get_gpio_in(DEVICE(&s->cpu[i]), ARM_CPU_IRQ)); > + sysbus_connect_irq(SYS_BUS_DEVICE(&s->a9mpcore), i + nc->num_cpus, > + qdev_get_gpio_in(DEVICE(&s->cpu[i]), ARM_CPU_FIQ)); > + } > + > + /* L2 cache controller */ > + sysbus_create_simple("l2x0", NPCM7XX_L2C_BA, NULL); > + > + /* System Global Control Registers (GCR) */ > + object_property_set_int(OBJECT(&s->gcr), nc->disabled_modules, > + "disabled-modules", &error_abort); > + object_property_add_const_link(OBJECT(&s->gcr), "dram-mr", OBJECT(s->dram)); > + sysbus_realize(SYS_BUS_DEVICE(&s->gcr), &error_abort); > + sysbus_mmio_map(SYS_BUS_DEVICE(&s->gcr), 0, NPCM7XX_GCR_BA); > + > + /* Clock Control Registers (CLK) */ > + sysbus_realize(SYS_BUS_DEVICE(&s->clk), &error_abort); > + sysbus_mmio_map(SYS_BUS_DEVICE(&s->clk), 0, NPCM7XX_CLK_BA); > + > + /* Timer Modules (TIM) */ > + QEMU_BUILD_BUG_ON(ARRAY_SIZE(npcm7xx_tim_addr) != ARRAY_SIZE(s->tim)); > + for (i = 0; i < ARRAY_SIZE(s->tim); i++) { > + SysBusDevice *sbd = SYS_BUS_DEVICE(&s->tim[i]); > + int first_irq; > + int j; > + > + sysbus_realize(sbd, &error_abort); > + sysbus_mmio_map(sbd, 0, npcm7xx_tim_addr[i]); > + > + first_irq = NPCM7XX_TIMER0_IRQ + i * NPCM7XX_TIMERS_PER_CTRL; > + for (j = 0; j < NPCM7XX_TIMERS_PER_CTRL; j++) { > + qemu_irq irq = npcm7xx_irq(s, first_irq + j); > + sysbus_connect_irq(sbd, j, irq); > + } > + } > + > + /* UART0..3 (16550 compatible) */ > + for (i = 0; i < ARRAY_SIZE(npcm7xx_uart_addr); i++) { > + serial_mm_init(get_system_memory(), npcm7xx_uart_addr[i], 2, > + npcm7xx_irq(s, NPCM7XX_UART0_IRQ + i), 115200, > + serial_hd(i), DEVICE_LITTLE_ENDIAN); > + } > + > + /* RAM2 (SRAM) */ > + memory_region_init_ram(&s->sram, OBJECT(dev), "ram2", > + NPCM7XX_RAM2_SZ, &error_abort); > + memory_region_add_subregion(get_system_memory(), NPCM7XX_RAM2_BA, &s->sram); > + > + /* RAM3 (SRAM) */ > + memory_region_init_ram(&s->ram3, OBJECT(dev), "ram3", > + NPCM7XX_RAM3_SZ, &error_abort); > + memory_region_add_subregion(get_system_memory(), NPCM7XX_RAM3_BA, &s->ram3); > + > + /* Internal ROM */ > + memory_region_init_rom(&s->irom, OBJECT(dev), "irom", NPCM7XX_ROM_SZ, > + &error_abort); > + memory_region_add_subregion(get_system_memory(), NPCM7XX_ROM_BA, &s->irom); > + > + create_unimplemented_device("npcm7xx.shm", 0xc0001000, 4 * KiB); > + create_unimplemented_device("npcm7xx.vdmx", 0xe0800000, 4 * KiB); > + create_unimplemented_device("npcm7xx.pcierc", 0xe1000000, 64 * KiB); > + create_unimplemented_device("npcm7xx.kcs", 0xf0007000, 4 * KiB); > + create_unimplemented_device("npcm7xx.rng", 0xf000b000, 4 * KiB); > + create_unimplemented_device("npcm7xx.adc", 0xf000c000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gfxi", 0xf000e000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[0]", 0xf0010000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[1]", 0xf0011000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[2]", 0xf0012000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[3]", 0xf0013000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[4]", 0xf0014000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[5]", 0xf0015000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[6]", 0xf0016000, 4 * KiB); > + create_unimplemented_device("npcm7xx.gpio[7]", 0xf0017000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[0]", 0xf0080000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[1]", 0xf0081000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[2]", 0xf0082000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[3]", 0xf0083000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[4]", 0xf0084000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[5]", 0xf0085000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[6]", 0xf0086000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[7]", 0xf0087000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[8]", 0xf0088000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[9]", 0xf0089000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[10]", 0xf008a000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[11]", 0xf008b000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[12]", 0xf008c000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[13]", 0xf008d000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[14]", 0xf008e000, 4 * KiB); > + create_unimplemented_device("npcm7xx.smbus[15]", 0xf008f000, 4 * KiB); > + create_unimplemented_device("npcm7xx.espi", 0xf009f000, 4 * KiB); > + create_unimplemented_device("npcm7xx.peci", 0xf0100000, 4 * KiB); > + create_unimplemented_device("npcm7xx.siox[1]", 0xf0101000, 4 * KiB); > + create_unimplemented_device("npcm7xx.siox[2]", 0xf0102000, 4 * KiB); > + create_unimplemented_device("npcm7xx.pwm[0]", 0xf0103000, 4 * KiB); > + create_unimplemented_device("npcm7xx.pwm[1]", 0xf0104000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[0]", 0xf0180000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[1]", 0xf0181000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[2]", 0xf0182000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[3]", 0xf0183000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[4]", 0xf0184000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[5]", 0xf0185000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[6]", 0xf0186000, 4 * KiB); > + create_unimplemented_device("npcm7xx.mft[7]", 0xf0187000, 4 * KiB); > + create_unimplemented_device("npcm7xx.pspi1", 0xf0200000, 4 * KiB); > + create_unimplemented_device("npcm7xx.pspi2", 0xf0201000, 4 * KiB); > + create_unimplemented_device("npcm7xx.ahbpci", 0xf0400000, 1 * MiB); > + create_unimplemented_device("npcm7xx.mcphy", 0xf05f0000, 64 * KiB); > + create_unimplemented_device("npcm7xx.gmac1", 0xf0802000, 8 * KiB); > + create_unimplemented_device("npcm7xx.gmac2", 0xf0804000, 8 * KiB); > + create_unimplemented_device("npcm7xx.ehci", 0xf0806000, 4 * KiB); > + create_unimplemented_device("npcm7xx.ohci", 0xf0807000, 4 * KiB); > + create_unimplemented_device("npcm7xx.vcd", 0xf0810000, 64 * KiB); > + create_unimplemented_device("npcm7xx.ece", 0xf0820000, 8 * KiB); > + create_unimplemented_device("npcm7xx.vdma", 0xf0822000, 8 * KiB); > + create_unimplemented_device("npcm7xx.emc1", 0xf0825000, 4 * KiB); > + create_unimplemented_device("npcm7xx.emc2", 0xf0826000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[0]", 0xf0830000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[1]", 0xf0831000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[2]", 0xf0832000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[3]", 0xf0833000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[4]", 0xf0834000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[5]", 0xf0835000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[6]", 0xf0836000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[7]", 0xf0837000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[8]", 0xf0838000, 4 * KiB); > + create_unimplemented_device("npcm7xx.usbd[9]", 0xf0839000, 4 * KiB); > + create_unimplemented_device("npcm7xx.sd", 0xf0840000, 8 * KiB); > + create_unimplemented_device("npcm7xx.mmc", 0xf0842000, 8 * KiB); > + create_unimplemented_device("npcm7xx.pcimbx", 0xf0848000, 512 * KiB); > + create_unimplemented_device("npcm7xx.aes", 0xf0858000, 4 * KiB); > + create_unimplemented_device("npcm7xx.des", 0xf0859000, 4 * KiB); > + create_unimplemented_device("npcm7xx.sha", 0xf085a000, 4 * KiB); > + create_unimplemented_device("npcm7xx.secacc", 0xf085b000, 4 * KiB); > + create_unimplemented_device("npcm7xx.spixcs0", 0xf8000000, 16 * MiB); > + create_unimplemented_device("npcm7xx.spixcs1", 0xf9000000, 16 * MiB); > + create_unimplemented_device("npcm7xx.spix", 0xfb001000, 4 * KiB); Thanks a lot for this! Without 'extern': Reviewed-by: Philippe Mathieu-Daudé > +} > + > +static Property npcm7xx_properties[] = { > + DEFINE_PROP_LINK("dram-mr", NPCM7xxState, dram, TYPE_MEMORY_REGION, > + MemoryRegion *), > + DEFINE_PROP_END_OF_LIST(), > +}; > + > +static void npcm7xx_class_init(ObjectClass *oc, void *data) > +{ > + DeviceClass *dc = DEVICE_CLASS(oc); > + > + dc->realize = npcm7xx_realize; > + dc->user_creatable = false; > + device_class_set_props(dc, npcm7xx_properties); > +} > + > +static void npcm730_class_init(ObjectClass *oc, void *data) > +{ > + NPCM7xxClass *nc = NPCM7XX_CLASS(oc); > + > + /* NPCM730 is optimized for data center use, so no graphics, etc. */ > + nc->disabled_modules = 0x00300395; > + nc->num_cpus = 2; > +} > + > +static void npcm750_class_init(ObjectClass *oc, void *data) > +{ > + NPCM7xxClass *nc = NPCM7XX_CLASS(oc); > + > + /* NPCM750 has 2 cores and a full set of peripherals */ > + nc->disabled_modules = 0x00000000; > + nc->num_cpus = 2; > +} > + > +static const TypeInfo npcm7xx_soc_types[] = { > + { > + .name = TYPE_NPCM7XX, > + .parent = TYPE_DEVICE, > + .instance_size = sizeof(NPCM7xxState), > + .instance_init = npcm7xx_init, > + .class_size = sizeof(NPCM7xxClass), > + .class_init = npcm7xx_class_init, > + .abstract = true, > + }, { > + .name = TYPE_NPCM730, > + .parent = TYPE_NPCM7XX, > + .class_init = npcm730_class_init, > + }, { > + .name = TYPE_NPCM750, > + .parent = TYPE_NPCM7XX, > + .class_init = npcm750_class_init, > + }, > +}; > + > +DEFINE_TYPES(npcm7xx_soc_types); > diff --git a/hw/arm/Kconfig b/hw/arm/Kconfig > index 192a8dec3b..a31d0d282f 100644 > --- a/hw/arm/Kconfig > +++ b/hw/arm/Kconfig > @@ -356,6 +356,11 @@ config XLNX_VERSAL > > config NPCM7XX > bool > + select A9MPCORE > + select ARM_GIC > + select PL310 # cache controller > + select SERIAL > + select UNIMP > > config FSL_IMX25 > bool > diff --git a/hw/arm/Makefile.objs b/hw/arm/Makefile.objs > index 534a6a119e..13d163a599 100644 > --- a/hw/arm/Makefile.objs > +++ b/hw/arm/Makefile.objs > @@ -41,6 +41,7 @@ obj-$(CONFIG_STM32F205_SOC) += stm32f205_soc.o > obj-$(CONFIG_STM32F405_SOC) += stm32f405_soc.o > obj-$(CONFIG_XLNX_ZYNQMP_ARM) += xlnx-zynqmp.o xlnx-zcu102.o > obj-$(CONFIG_XLNX_VERSAL) += xlnx-versal.o xlnx-versal-virt.o > +obj-$(CONFIG_NPCM7XX) += npcm7xx.o > obj-$(CONFIG_FSL_IMX25) += fsl-imx25.o imx25_pdk.o > obj-$(CONFIG_FSL_IMX31) += fsl-imx31.o kzm.o > obj-$(CONFIG_FSL_IMX6) += fsl-imx6.o >