test: dm: led: Fix LED enumeration
authorMarek Vasut <marex@denx.de>
Mon, 25 Apr 2022 16:33:50 +0000 (18:33 +0200)
committerTom Rini <trini@konsulko.com>
Thu, 28 Apr 2022 13:26:44 +0000 (09:26 -0400)
The GPIO LED driver no longer considers the top level node an LED,
because it is not an LED. With this bug fixed, the LED enumeration
has changed. Update the test accordingly.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Patrice Chotard <patrice.chotard@foss.st.com>
Cc: Patrick Delaunay <patrick.delaunay@foss.st.com>
Cc: Sean Anderson <seanga2@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Steven Lawrance <steven.lawrance@softathome.com>
test/dm/led.c

index 5bbe04648a1ca6bae455c5c27a9ad2a64ac99607..eed3f4654c568aba8c1d9abdf4ec03f066875605 100644 (file)
@@ -21,8 +21,7 @@ static int dm_test_led_base(struct unit_test_state *uts)
        ut_assertok(uclass_get_device(UCLASS_LED, 1, &dev));
        ut_assertok(uclass_get_device(UCLASS_LED, 2, &dev));
        ut_assertok(uclass_get_device(UCLASS_LED, 3, &dev));
-       ut_assertok(uclass_get_device(UCLASS_LED, 4, &dev));
-       ut_asserteq(-ENODEV, uclass_get_device(UCLASS_LED, 5, &dev));
+       ut_asserteq(-ENODEV, uclass_get_device(UCLASS_LED, 4, &dev));
 
        return 0;
 }
@@ -52,10 +51,10 @@ static int dm_test_led_gpio(struct unit_test_state *uts)
        struct udevice *dev, *gpio;
 
        /*
-        * Check that we can manipulate an LED. LED 1 is connected to GPIO
+        * Check that we can manipulate an LED. LED 0 is connected to GPIO
         * bank gpio_a, offset 1.
         */
-       ut_assertok(uclass_get_device(UCLASS_LED, 1, &dev));
+       ut_assertok(uclass_get_device(UCLASS_LED, 0, &dev));
        ut_assertok(uclass_get_device(UCLASS_GPIO, 1, &gpio));
        ut_asserteq(0, sandbox_gpio_get_value(gpio, offset));
        ut_assertok(led_set_state(dev, LEDST_ON));
@@ -77,10 +76,10 @@ static int dm_test_led_toggle(struct unit_test_state *uts)
        struct udevice *dev, *gpio;
 
        /*
-        * Check that we can manipulate an LED. LED 1 is connected to GPIO
+        * Check that we can manipulate an LED. LED 0 is connected to GPIO
         * bank gpio_a, offset 1.
         */
-       ut_assertok(uclass_get_device(UCLASS_LED, 1, &dev));
+       ut_assertok(uclass_get_device(UCLASS_LED, 0, &dev));
        ut_assertok(uclass_get_device(UCLASS_GPIO, 1, &gpio));
        ut_asserteq(0, sandbox_gpio_get_value(gpio, offset));
        ut_assertok(led_set_state(dev, LEDST_TOGGLE));
@@ -102,12 +101,12 @@ static int dm_test_led_label(struct unit_test_state *uts)
 
        ut_assertok(led_get_by_label("sandbox:red", &dev));
        ut_asserteq(1, device_active(dev));
-       ut_assertok(uclass_get_device(UCLASS_LED, 1, &cmp));
+       ut_assertok(uclass_get_device(UCLASS_LED, 0, &cmp));
        ut_asserteq_ptr(dev, cmp);
 
        ut_assertok(led_get_by_label("sandbox:green", &dev));
        ut_asserteq(1, device_active(dev));
-       ut_assertok(uclass_get_device(UCLASS_LED, 2, &cmp));
+       ut_assertok(uclass_get_device(UCLASS_LED, 1, &cmp));
        ut_asserteq_ptr(dev, cmp);
 
        ut_asserteq(-ENODEV, led_get_by_label("sandbox:blue", &dev));
@@ -127,7 +126,7 @@ static int dm_test_led_blink(struct unit_test_state *uts)
         * Check that we get an error when trying to blink an LED, since it is
         * not supported by the GPIO LED driver.
         */
-       ut_assertok(uclass_get_device(UCLASS_LED, 1, &dev));
+       ut_assertok(uclass_get_device(UCLASS_LED, 0, &dev));
        ut_assertok(uclass_get_device(UCLASS_GPIO, 1, &gpio));
        ut_asserteq(0, sandbox_gpio_get_value(gpio, offset));
        ut_asserteq(-ENOSYS, led_set_state(dev, LEDST_BLINK));