All of lore.kernel.org
 help / color / mirror / Atom feed
* does drive_get_next(IF_NONE) make sense?
@ 2021-11-02 15:14 Peter Maydell
  2021-11-02 15:28 ` Philippe Mathieu-Daudé
  2021-11-03  8:41 ` Markus Armbruster
  0 siblings, 2 replies; 15+ messages in thread
From: Peter Maydell @ 2021-11-02 15:14 UTC (permalink / raw)
  To: QEMU Developers
  Cc: Hao Wu, Bin Meng, Alistair Francis, Markus Armbruster, Qemu-block

Does it make sense for a device/board to do drive_get_next(IF_NONE) ?

At the moment we have exactly one user of this, which is
hw/misc/sifive_u_otp.c. This is a model of a one-time-programmable
fuse, and the drive is providing the backing store for the fuse
contents. Borrowing an IF_NONE for this seems a bit odd, but
it's not clear any of the other IF_ types is better.

We also just (this release cycle) added models of the Xilinx
efuse OTP fuses. Those have been implemented to use IF_PFLASH.
(This is a somewhat unfortunate inconsistency I guess.)

We also have a patchseries currently in the code review stage
which uses IF_NONE:
https://patchew.org/QEMU/20211101232346.1070813-1-wuhaotsh@google.com/20211101232346.1070813-6-wuhaotsh@google.com/
Here we are trying to provide a drive as backing store for some
EEPROMs that hang off the i2c buses on some npcm7xx boards.

Are these uses of IF_NONE OK, or should we be doing something
else (using IF_PFLASH, defining a new IF_*, ???)

thanks
-- PMM


^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2021-11-18 13:22 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-02 15:14 does drive_get_next(IF_NONE) make sense? Peter Maydell
2021-11-02 15:28 ` Philippe Mathieu-Daudé
2021-11-03  8:41 ` Markus Armbruster
2021-11-03  9:19   ` Thomas Huth
2021-11-12 13:33     ` Markus Armbruster
2021-11-14 17:16       ` Peter Maydell
2021-11-15  5:31         ` Markus Armbruster
2021-11-15  7:12           ` Alistair Francis
2021-11-15 16:09             ` Thomas Huth
2021-11-18 13:03               ` Alistair Francis
2021-11-18 13:20                 ` Peter Maydell
2021-11-15 13:24           ` Kevin Wolf
2021-11-15 13:31             ` Peter Maydell
2021-11-15 15:13               ` Kevin Wolf
2021-11-15 15:28               ` Markus Armbruster

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.