diff options
| author | Tom Rini <[email protected]> | 2025-12-11 19:19:06 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-12-12 08:00:54 -0600 |
| commit | aff0f4d30ec6d7968578fc05fc6df6b5700d9913 (patch) | |
| tree | bf94298050837aecee2249adcc6e9a5d48d152ab /test | |
| parent | 802fbe0a287130c4800510ac6df3850c27606ff0 (diff) | |
Revert "clk: Return value calculated by ERR_PTR"
While this change was intended to fix a mistake in the code, of calling
the ERR_PTR macro but not making use of the result, it seems that
functionally platforms depend on the loop not existing here. The TI K3
families of platforms for example were broken by this commit.
This reverts commit fe780310cfa8bf5a093894b5cd7fe85c6b02fd91.
Reported-by: Nishanth Menon <[email protected]>
Reviewed-by: Andrew Goodbody <[email protected]>
Reviewed-by: Nishanth Menon <[email protected]>
Signed-off-by: Tom Rini <[email protected]>
Diffstat (limited to 'test')
| -rw-r--r-- | test/dm/clk.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/test/dm/clk.c b/test/dm/clk.c index c3363796498..790968e6477 100644 --- a/test/dm/clk.c +++ b/test/dm/clk.c @@ -89,9 +89,8 @@ static int dm_test_clk(struct unit_test_state *uts) SANDBOX_CLK_TEST_ID_SPI)); ut_asserteq(0, sandbox_clk_test_get_rate(dev_test, SANDBOX_CLK_TEST_ID_I2C)); - if (!CONFIG_IS_ENABLED(CLK_AUTO_ID)) - ut_asserteq(321, sandbox_clk_test_get_rate(dev_test, - SANDBOX_CLK_TEST_ID_DEVM1)); + ut_asserteq(321, sandbox_clk_test_get_rate(dev_test, + SANDBOX_CLK_TEST_ID_DEVM1)); ut_asserteq(0, sandbox_clk_test_get_rate(dev_test, SANDBOX_CLK_TEST_ID_DEVM2)); |