virtio: sandbox: Bind RNG rather than block device
authorAndrew Scull <ascull@google.com>
Mon, 16 May 2022 10:41:37 +0000 (10:41 +0000)
committerTom Rini <trini@konsulko.com>
Wed, 8 Jun 2022 13:24:04 +0000 (09:24 -0400)
The virtio-rng driver is extremely simple, making it suitable for
testing more of the virtio uclass logic. Have the sandbox driver bind
the virtio-rng driver rather than the virtio-blk driver so it can be
used in tests.

Signed-off-by: Andrew Scull <ascull@google.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
drivers/virtio/virtio_sandbox.c
test/dm/Makefile
test/dm/virtio_device.c

index a73b123..5484ae3 100644 (file)
@@ -161,7 +161,7 @@ static int virtio_sandbox_probe(struct udevice *udev)
 
        /* fake some information for testing */
        priv->device_features = BIT_ULL(VIRTIO_F_VERSION_1);
-       uc_priv->device = VIRTIO_ID_BLOCK;
+       uc_priv->device = VIRTIO_ID_RNG;
        uc_priv->vendor = ('u' << 24) | ('b' << 16) | ('o' << 8) | 't';
 
        return 0;
index 29dd143..809f0f2 100644 (file)
@@ -109,7 +109,7 @@ obj-$(CONFIG_DM_USB) += usb.o
 obj-$(CONFIG_DM_VIDEO) += video.o
 ifeq ($(CONFIG_VIRTIO_SANDBOX),y)
 obj-y += virtio.o
-obj-$(CONFIG_VIRTIO_BLK) += virtio_device.o
+obj-$(CONFIG_VIRTIO_RNG) += virtio_device.o
 endif
 ifeq ($(CONFIG_WDT_GPIO)$(CONFIG_WDT_SANDBOX),yy)
 obj-y += wdt.o
index 46f4798..f5f2349 100644 (file)
@@ -25,10 +25,10 @@ static int dm_test_virtio_base(struct unit_test_state *uts)
        ut_assertok(uclass_first_device(UCLASS_VIRTIO, &bus));
        ut_assertnonnull(bus);
 
-       /* check the child virtio-blk device is bound */
+       /* check the child virtio-rng device is bound */
        ut_assertok(device_find_first_child(bus, &dev));
        ut_assertnonnull(dev);
-       ut_assertok(strcmp(dev->name, "virtio-blk#0"));
+       ut_asserteq_str("virtio-rng#0", dev->name);
 
        /* check driver status */
        ut_assertok(virtio_get_status(dev, &status));