diff options
| author | Bin Meng <bmeng.cn@gmail.com> | 2018-08-03 01:14:38 -0700 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2018-08-08 12:49:31 +0100 |
| commit | dd4808f9fab54b94df76447b7a7f383cd1b2f5d0 (patch) | |
| tree | 21d6720cffd7bf7137dce19ed812aaa406fdb51c | |
| parent | 64e45f73bd0aa9d8a1069587540bc91fe45b62e0 (diff) | |
test: dm: pci: Remove unnecessary steps in dm_test_pci_swapcase()
The check on uclass_get_device() and device_active() is unnecessary
as the follow-up test operations will implicitly probe the driver.
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
| -rw-r--r-- | test/dm/pci.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/test/dm/pci.c b/test/dm/pci.c index 47b5d22af8..be1208cfbd 100644 --- a/test/dm/pci.c +++ b/test/dm/pci.c @@ -34,14 +34,12 @@ DM_TEST(dm_test_pci_busnum, DM_TESTF_SCAN_PDATA | DM_TESTF_SCAN_FDT); /* Test that we can use the swapcase device correctly */ static int dm_test_pci_swapcase(struct unit_test_state *uts) { - struct udevice *emul, *swap; + struct udevice *swap; ulong io_addr, mem_addr; char *ptr; /* Check that asking for the device automatically fires up PCI */ - ut_assertok(uclass_get_device(UCLASS_PCI_EMUL, 0, &emul)); ut_assertok(dm_pci_bus_find_bdf(PCI_BDF(0, 0x1f, 0), &swap)); - ut_assert(device_active(swap)); /* First test I/O */ io_addr = dm_pci_read_bar32(swap, 0); |