qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 6b717a: hw/misc/sifive_u_otp: Use IF_PFLASH f


From: Richard Henderson
Subject: [Qemu-commits] [qemu/qemu] 6b717a: hw/misc/sifive_u_otp: Use IF_PFLASH for the OTP de...
Date: Sun, 21 Nov 2021 23:53:44 -0800

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 6b717a8d4482fa88fb2a40b5afd5a55e7a0ba412
      
https://github.com/qemu/qemu/commit/6b717a8d4482fa88fb2a40b5afd5a55e7a0ba412
  Author: Thomas Huth <thuth@redhat.com>
  Date:   2021-11-22 (Mon, 22 Nov 2021)

  Changed paths:
    M docs/about/deprecated.rst
    M hw/misc/sifive_u_otp.c

  Log Message:
  -----------
  hw/misc/sifive_u_otp: Use IF_PFLASH for the OTP device instead of IF_NONE

Configuring a drive with "if=none" is meant for creation of a backend
only, it should not get automatically assigned to a device frontend.
Use "if=pflash" for the One-Time-Programmable device instead (like
it is e.g. also done for the efuse device in hw/arm/xlnx-zcu102.c).

Since the old way of configuring the device has already been published
with the previous QEMU versions, we cannot remove this immediately, but
have to deprecate it and support it for at least two more releases.

Signed-off-by: Thomas Huth <thuth@redhat.com>
Acked-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-id: 20211119102549.217755-1-thuth@redhat.com
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>


  Commit: 526e7443027c71fe7b04c29df529e1f9f425f9e3
      
https://github.com/qemu/qemu/commit/526e7443027c71fe7b04c29df529e1f9f425f9e3
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-11-22 (Mon, 22 Nov 2021)

  Changed paths:
    M hw/misc/sifive_u_otp.c

  Log Message:
  -----------
  hw/misc/sifive_u_otp: Do not reset OTP content on hardware reset

Once a "One Time Programmable" is programmed, it shouldn't be reset.

Do not re-initialize the OTP content in the DeviceReset handler,
initialize it once in the DeviceRealize one.

Fixes: 9fb45c62ae8 ("riscv: sifive: Implement a model for SiFive FU540 OTP")
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20211119104757.331579-1-f4bug@amsat.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>


  Commit: edf1aa8d44d5070828ac10c5b82dbc6caf29e7d3
      
https://github.com/qemu/qemu/commit/edf1aa8d44d5070828ac10c5b82dbc6caf29e7d3
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-11-22 (Mon, 22 Nov 2021)

  Changed paths:
    M docs/about/deprecated.rst
    M hw/misc/sifive_u_otp.c

  Log Message:
  -----------
  Merge tag 'pull-riscv-to-apply-20211122' of github.com:alistair23/qemu into 
staging

Seventh RISC-V PR for QEMU 6.2

 - Deprecate IF_NONE for SiFive OTP
 - Don't reset SiFive OTP content

# gpg: Signature made Mon 22 Nov 2021 07:51:24 AM CET
# gpg:                using RSA key F6C4AC46D4934868D3B8CE8F21E10D29DF977054
# gpg: Good signature from "Alistair Francis <alistair@alistair23.me>" [full]

* tag 'pull-riscv-to-apply-20211122' of github.com:alistair23/qemu:
  hw/misc/sifive_u_otp: Do not reset OTP content on hardware reset
  hw/misc/sifive_u_otp: Use IF_PFLASH for the OTP device instead of IF_NONE

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Compare: https://github.com/qemu/qemu/compare/c5fbdd60cf1f...edf1aa8d44d5



reply via email to

[Prev in Thread] Current Thread [Next in Thread]