* Markus Armbruster (arm...@redhat.com) wrote:
> 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.
I wouldn't be too nasty about -drive; for me I still find it the
easiest way to start a VM.
Dave
--
Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK
