Philippe Mathieu-Daudé <phi...@redhat.com> writes: > On 2/22/21 9:20 AM, Sai Pavan Boddu wrote: >> From: Vincent Palatin <vpala...@chromium.org> >> >> Add new block device type. >> >> Signed-off-by: Vincent Palatin <vpala...@chromium.org> >> [SPB: Rebased over 5.1 version] >> Signed-off-by: Sai Pavan Boddu <sai.pavan.bo...@xilinx.com> >> Signed-off-by: Joel Stanley <j...@jms.id.au> >> Signed-off-by: Cédric Le Goater <c...@kaod.org> >> Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> >> --- >> include/sysemu/blockdev.h | 1 + >> blockdev.c | 1 + >> 2 files changed, 2 insertions(+) >> >> diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h >> index 3b5fcda..eefae9f 100644 >> --- a/include/sysemu/blockdev.h >> +++ b/include/sysemu/blockdev.h >> @@ -24,6 +24,7 @@ typedef enum { >> */ >> IF_NONE = 0, >> IF_IDE, IF_SCSI, IF_FLOPPY, IF_PFLASH, IF_MTD, IF_SD, IF_VIRTIO, IF_XEN, >> + IF_EMMC, >> IF_COUNT >> } BlockInterfaceType; >> >> diff --git a/blockdev.c b/blockdev.c >> index cd438e6..390d43c 100644 >> --- a/blockdev.c >> +++ b/blockdev.c >> @@ -83,6 +83,7 @@ static const char *const if_name[IF_COUNT] = { >> [IF_SD] = "sd", >> [IF_VIRTIO] = "virtio", >> [IF_XEN] = "xen", >> + [IF_EMMC] = "emmc", >> }; > > We don't need to introduce support for the legacy -drive magic. > > -device should be enough for this device, right?
External interface extensions need rationale: why do we want / need it? The commit message neglects to provide one. Even more so when the interface in question is in a state like -drive is.