[PATCH 0/2] reset: fix reset_get_by_index_nodev index handling

Neil Armstrong narmstrong at baylibre.com
Tue Apr 20 10:42:24 CEST 2021

A regression weas detected on Amlogic G12A/G12B SoCs, where HDMI output was disable
even when Linux was booting.

Bisect reports 139e4a1cbe ("drivers: reset: Add a managed API to get reset controllers from the DT")
as the offending commit.

But the error is in ea9dc35aab ("reset: Get the RESET by index without device") where a spurius "> 0"
was added to the index handling.

But the dm_test_reset_base() test did not catch it.

The first commit extends the test to catch the regression, and the second patch fixes the regression.

Neil Armstrong (2):
  test: reset: Extend base reset test to catch error
  reset: fix reset_get_by_index_nodev index handling

 arch/sandbox/dts/test.dts    |  4 ++--
 drivers/reset/reset-uclass.c |  2 +-
 test/dm/reset.c              | 39 +++++++++++++++++++++++++++++++-----
 3 files changed, 37 insertions(+), 8 deletions(-)


More information about the U-Boot mailing list