From patchwork Sun Sep 1 02:52:41 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bin Meng X-Patchwork-Id: 11125027 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id F06E914DE for ; Sun, 1 Sep 2019 02:56:05 +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 B6FF42190F for ; Sun, 1 Sep 2019 02:56:05 +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="RQ1zJjIe" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B6FF42190F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Received: from localhost ([::1]:47194 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i4G28-0001Cz-9r for patchwork-qemu-devel@patchwork.kernel.org; Sat, 31 Aug 2019 22:56:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:47501) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1i4FzY-0007Xt-Jh for qemu-devel@nongnu.org; Sat, 31 Aug 2019 22:53:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1i4FzU-00066J-8y for qemu-devel@nongnu.org; Sat, 31 Aug 2019 22:53:24 -0400 Received: from mail-pg1-x542.google.com ([2607:f8b0:4864:20::542]:34266) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1i4FzT-00063v-Ul; Sat, 31 Aug 2019 22:53:20 -0400 Received: by mail-pg1-x542.google.com with SMTP id n9so5504757pgc.1; Sat, 31 Aug 2019 19:53:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=mhMz8G0NsEA1wlStnRH5Ti9aIE6MiOk9YGOtS9Es+7U=; b=RQ1zJjIeK17qE3K7T/tKNokyD+Kb32w0CReSa8e6M0DDdTG87SkVTtgq6EeltbtNUH nYBZa0E+lvpiIbx7RZ+x4lHMpfh+yWZUPc4ogWdgoSWGgIBzW8+AULVkwNLLjZAOmMrJ DpqXnnYfFEJgjDYC6iCR/8CXyRKRjnpD8WYGeYrouXw8ydCsCnBrVLZne8DVjSVtmka3 QpWeFp4SMJzfetp9wIbHZJVpU0Z5l/R8KmQxOGnKZPIFsivV5f1H87JZweSlvItjgFEq BM12fZNcyKNb7soHgDkt2nVta1ynwzKfGb8k2RXX9dB6DHM3P237GBHkoY1JCty0tiIT iH4w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id:mime-version :content-transfer-encoding; bh=mhMz8G0NsEA1wlStnRH5Ti9aIE6MiOk9YGOtS9Es+7U=; b=dqLg43OPFskSlULM3/XP27zB+NOD6V1rFVUPhQFyDVxUIGC2qfQrkx7D/PWHFJltv3 iNhyD/TJaz7w3XUUTWxNhAXFVXD1IEqlmBTIS1JlIQfyHqgpxDlf06+LCldq35eGdEmq drZ4kEKV8ZqJ1drPI4gArJ4MTMQoNYsV4HHvNdKHIzjSVvAM1rDCLt22xPjzxi7WZ6sC u2V2zsll8W2UfUg3K6NAmG1ewTw7l2yusgobaVqlOkee+3FTY67FCqG4wN2iViiSIq7E N2ThAJDqZWPRlpwfxBGuM1kQnomVqGfCy5eeBj08R/7w5+4JfKd6//oQTiw/BnASlya3 KdSA== X-Gm-Message-State: APjAAAU2neUHmoFHOeSawi8WP9DRRhtcdMx8CQ/BpUGFZ0GTBZ+zw0fm 7A216AN8DmTtxdkP01zzJco= X-Google-Smtp-Source: APXvYqzKKcfHGsaqGHX4t8z4MIboqyM9v1buX8rR3Fuw8hbXqSkhnAuLfrllZTtrWuKngk2Wx4Y2Kg== X-Received: by 2002:a63:505a:: with SMTP id q26mr18953437pgl.18.1567306398368; Sat, 31 Aug 2019 19:53:18 -0700 (PDT) Received: from localhost.localdomain (unknown-224-80.windriver.com. [147.11.224.80]) by smtp.gmail.com with ESMTPSA id p10sm11453307pff.132.2019.08.31.19.53.17 (version=TLS1 cipher=AES128-SHA bits=128/128); Sat, 31 Aug 2019 19:53:17 -0700 (PDT) From: Bin Meng To: Alistair Francis , Palmer Dabbelt , qemu-devel@nongnu.org, qemu-riscv@nongnu.org Date: Sat, 31 Aug 2019 19:52:41 -0700 Message-Id: <1567306391-2682-1-git-send-email-bmeng.cn@gmail.com> X-Mailer: git-send-email 1.7.1 MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:4864:20::542 Subject: [Qemu-devel] [PATCH v7 00/30] riscv: sifive_u: Improve the emulation fidelity of sifive_u machine 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: , Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: "Qemu-devel" As of today, the QEMU 'sifive_u' machine is a special target that does not boot the upstream OpenSBI/U-Boot firmware images built for the real SiFive HiFive Unleashed board. Hence OpenSBI supports a special platform "qemu/sifive_u". For U-Boot, the sifive_fu540_defconfig is referenced in the OpenSBI doc as its payload, but that does not boot at all due to various issues in current QEMU 'sifive_u' machine codes. This series aims to improve the emulation fidelity of sifive_u machine, so that the upstream OpenSBI, U-Boot and kernel images built for the SiFive HiFive Unleashed board can be used out of the box without any special hack. The major changes include: - Heterogeneous harts creation supported, so that we can create a CPU that exactly mirrors the real hardware: 1 E51 + 4 U54. - Implemented a PRCI model for FU540 - Implemented an OTP model for FU540, primarily used for storing serial number of the board - Fixed GEM support that was seriously broken on sifive_u - Synced device tree with upstream Linux kernel on sifive_u OpenSBI v0.4 image built for sifive/fu540 is included as the default bios image for 'sifive_u' machine. The series is tested against OpenSBI v0.4 image for sifive/fu540 paltform, U-Boot v2019.10-rc1 image for sifive_fu540_defconfig, and Linux kernel v5.3-rc3 image with the following patch: macb: Update compatibility string for SiFive FU540-C000 [1] OpenSBI + U-Boot, ping/tftpboot with U-Boot MACB driver works well. Boot Linux 64-bit defconfig image, verified that system console on the serial 0 and ping host work pretty well. An OpenSBI patch [2] was sent to drop the special "qemu/sifive_u" platform support in OpenSBI. The original plan was to get the drop patch applied after this QEMU series is merged. However after discussion in the OpenSBI mailing list, it seems the best option for us is to let OpenSBI continue shipping the special "qemu/sifive_u" platform support to work with QEMU version <= 4.1 and deprecate the support sometime in the future. A patch will need to be sent to OpenSBI mailing list to update its document. v4 is now rebased on Palmer's QEMU RISC-V repo "for-master" branch. Dropped the following v3 patch that was already done by someone else. - riscv: sifive_u: Generate an aliases node in the device tree - riscv: sifive_u: Support loading initramfs The following v3 patch was dropped too due to a different cluster approach suggested by Richard Henderson is used in v4: - riscv: hart: Support heterogeneous harts population [1]: https://patchwork.kernel.org/patch/11050003/ [2]: http://lists.infradead.org/pipermail/opensbi/2019-August/000335.html Changes in v7: - use `s->hartid_base + idx` directly Changes in v6: - use s->hartid_base directly, instead of an extra variable - generate u cpus unconditionally (not test ms->smp.cpus > 1) since the minimal required number of cpu is now 2, due to this patch is reordered - fix incorrectly indented comment lines - remove unneeded brackets around assignment Changes in v5: - new patch to change to use qemu_log_mask(LOG_GUEST_ERROR,...) instead in various sifive models - new patch to remove the unnecessary include of target/riscv/cpu.h - change to use defines instead of enums - change to use qemu_log_mask(LOG_GUEST_ERROR,...) in sifive_u_prci - creating a 32-bit val variable and using that instead of casting everywhere in sifive_u_prci_write() - move all register initialization to sifive_u_prci_reset() function - drop sifive_u_prci_create() - s/codes that worked/code that works/g - create sifive_u_prci block directly in the machine codes, instead of calling sifive_u_prci_create() - change to use defines instead of enums - change to use qemu_log_mask(LOG_GUEST_ERROR,...) in sifive_u_otp - creating a 32-bit val variable and using that instead of casting everywhere in sifive_u_otp_write() - move all register initialization to sifive_u_otp_reset() function - drop sifive_u_otp_create() - create sifive_u_otp block directly in the machine codes, instead of calling sifive_u_otp_create() - add the missing "local-mac-address" property in the ethernet node Changes in v4: - remove 2 more "linux,phandle" instances in sifive_u.c and spike.c after rebasing on Palmer's QEMU RISC-V tree - change create_fdt() to return void in sifive_u.c too, after rebasing on Palmer's QEMU RISC-V tree - new patch to remove executable attribute of opensbi images - prefix all macros/variables/functions with SIFIVE_E/sifive_e in the sifive_e_prci driver - new patch to add a "hartid-base" property to RISC-V hart array - changed to create clusters for each cpu type - prefix all macros/variables/functions with SIFIVE_U/sifive_u in the sifive_u_prci driver - prefix all macros/variables/functions with SIFIVE_U/sifive_u in the sifive_u_otp driver - new patch to remove handcrafted clock nodes for UART and ethernet Changes in v3: - use management cpu count + 1 for the min_cpus - changed to use macros for management and compute cpu count - update IRQ numbers of both UARTs to match hardware as well Changes in v2: - keep the PLIC compatible string unchanged as OpenSBI uses that for DT fix up - drop patch "riscv: sifive: Move sifive_mmio_emulate() to a common place" - new patch "riscv: sifive_e: Drop sifive_mmio_emulate()" - update the file header to indicate at least 2 harts are created - fixed the "interrupts-extended" property size - use create_unimplemented_device() to create the GEM management block instead of sifive_mmio_emulate() - add "phy-handle" property to the ethernet node Bin Meng (30): riscv: hw: Remove superfluous "linux,phandle" property riscv: hw: Use qemu_fdt_setprop_cell() for property with only 1 cell riscv: hw: Remove not needed PLIC properties in device tree riscv: hw: Change create_fdt() to return void riscv: hw: Change to use qemu_log_mask(LOG_GUEST_ERROR,...) instead riscv: hw: Remove the unnecessary include of target/riscv/cpu.h riscv: roms: Remove executable attribute of opensbi images riscv: sifive_u: Remove the unnecessary include of prci header riscv: sifive: Rename sifive_prci.{c,h} to sifive_e_prci.{c,h} riscv: sifive_e: prci: Fix a typo of hfxosccfg register programming riscv: sifive_e: prci: Update the PRCI register block size riscv: sifive_e: Drop sifive_mmio_emulate() riscv: Add a sifive_cpu.h to include both E and U cpu type defines riscv: hart: Extract hart realize to a separate routine riscv: hart: Add a "hartid-base" property to RISC-V hart array riscv: sifive_u: Set the minimum number of cpus to 2 riscv: sifive_u: Update hart configuration to reflect the real FU540 SoC riscv: sifive_u: Update PLIC hart topology configuration string riscv: sifive: Implement PRCI model for FU540 riscv: sifive_u: Generate hfclk and rtcclk nodes riscv: sifive_u: Add PRCI block to the SoC riscv: sifive_u: Reference PRCI clocks in UART and ethernet nodes riscv: sifive_u: Update UART base addresses and IRQs riscv: sifive_u: Change UART node name in device tree riscv: roms: Update default bios for sifive_u machine riscv: sifive: Implement a model for SiFive FU540 OTP riscv: sifive_u: Instantiate OTP memory with a serial number riscv: sifive_u: Fix broken GEM support riscv: sifive_u: Remove handcrafted clock nodes for UART and ethernet riscv: sifive_u: Update model and compatible strings in device tree hw/riscv/Makefile.objs | 4 +- hw/riscv/riscv_hart.c | 34 ++-- hw/riscv/sifive_e.c | 27 ++- hw/riscv/sifive_e_prci.c | 123 +++++++++++++ hw/riscv/sifive_prci.c | 123 ------------- hw/riscv/sifive_test.c | 6 +- hw/riscv/sifive_u.c | 255 ++++++++++++++++++--------- hw/riscv/sifive_u_otp.c | 190 ++++++++++++++++++++ hw/riscv/sifive_u_prci.c | 169 ++++++++++++++++++ hw/riscv/sifive_uart.c | 10 +- hw/riscv/spike.c | 1 - hw/riscv/virt.c | 40 ++--- include/hw/riscv/riscv_hart.h | 1 + include/hw/riscv/sifive_cpu.h | 31 ++++ include/hw/riscv/sifive_e.h | 7 +- include/hw/riscv/sifive_e_prci.h | 71 ++++++++ include/hw/riscv/sifive_prci.h | 69 -------- include/hw/riscv/sifive_u.h | 31 ++-- include/hw/riscv/sifive_u_otp.h | 80 +++++++++ include/hw/riscv/sifive_u_prci.h | 91 ++++++++++ pc-bios/opensbi-riscv32-virt-fw_jump.bin | Bin pc-bios/opensbi-riscv64-sifive_u-fw_jump.bin | Bin 40968 -> 45064 bytes pc-bios/opensbi-riscv64-virt-fw_jump.bin | Bin roms/Makefile | 4 +- 24 files changed, 1007 insertions(+), 360 deletions(-) create mode 100644 hw/riscv/sifive_e_prci.c delete mode 100644 hw/riscv/sifive_prci.c create mode 100644 hw/riscv/sifive_u_otp.c create mode 100644 hw/riscv/sifive_u_prci.c create mode 100644 include/hw/riscv/sifive_cpu.h create mode 100644 include/hw/riscv/sifive_e_prci.h delete mode 100644 include/hw/riscv/sifive_prci.h create mode 100644 include/hw/riscv/sifive_u_otp.h create mode 100644 include/hw/riscv/sifive_u_prci.h mode change 100755 => 100644 pc-bios/opensbi-riscv32-virt-fw_jump.bin mode change 100755 => 100644 pc-bios/opensbi-riscv64-sifive_u-fw_jump.bin mode change 100755 => 100644 pc-bios/opensbi-riscv64-virt-fw_jump.bin