diff options
| author | Simon Glass <sjg@chromium.org> | 2015-06-23 15:38:58 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2015-07-21 17:39:26 -0600 |
| commit | 3b55d30f6ff2f16c5fd02685bff7639211e6d326 (patch) | |
| tree | ad95bf1feedf12bfb96ab33b0b60db70a0159ed7 /test | |
| parent | 7837ceab1ef8d4a88fb218bfa6b042b0b865b15f (diff) | |
| download | u-boot-3b55d30f6ff2f16c5fd02685bff7639211e6d326.tar.gz | |
dm: pmic: Split output from function
The regulator_autoset() function mixes printf() output and PMIC adjustment
code. It provides a boolean to control the output. It is better to avoid
missing logic and output, and this permits a smaller SPL code size. So
split the output into a separate function.
Also rename the function to have a by_name() suffix, since we would like
to be able to pass a device when we know it, and thus avoid the name
search.
Signed-off-by: Simon Glass <sjg@chromium.org>
Tested-by: Przemyslaw Marczak <p.marczak@samsung.com>
Acked-by: Przemyslaw Marczak <p.marczak@samsung.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/dm/regulator.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/dm/regulator.c b/test/dm/regulator.c index d279c04c84..3d0056f2dc 100644 --- a/test/dm/regulator.c +++ b/test/dm/regulator.c @@ -210,7 +210,7 @@ static int dm_test_power_regulator_autoset(struct unit_test_state *uts) * Expected output state: uV=1200000; uA=200000; output enabled */ platname = regulator_names[BUCK1][PLATNAME]; - ut_assertok(regulator_autoset(platname, &dev_autoset, false)); + ut_assertok(regulator_autoset_by_name(platname, &dev_autoset)); /* Check, that the returned device is proper */ ut_assertok(regulator_get_by_platname(platname, &dev)); |