diff options
author | Abe Levkoy <alevkoy@chromium.org> | 2022-06-03 12:06:10 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-06-08 15:09:13 +0000 |
commit | f7fcc32c8ac47f64ce9b79ce2903254253533640 (patch) | |
tree | ca520d96cbf59cd11a7987b0a97e2dc06270be48 | |
parent | 5949c9a39700b2645a806513a02e2eacfe85509a (diff) | |
download | chrome-ec-f7fcc32c8ac47f64ce9b79ce2903254253533640.tar.gz |
zephyr test: Improve SYV682x test naming
The UUT for this test is the SYV682x device driver, which works for part
versions A, B, and C. The (emulated) part in the devicetree is an
SYV682C, but the majority of the driver interface is unaffected by the
specific part. Update file, suite, and variable names to reflect this.
BUG=b:234886404
TEST=zmake testall
BRANCH=none
Signed-off-by: Abe Levkoy <alevkoy@chromium.org>
Change-Id: I8024239d4f4b4aa0793ea9b1e5a4b9df126a4009
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3689799
Reviewed-by: Al Semjonovs <asemjonovs@google.com>
-rw-r--r-- | zephyr/test/drivers/CMakeLists.txt | 2 | ||||
-rw-r--r-- | zephyr/test/drivers/src/ppc_syv682x.c (renamed from zephyr/test/drivers/src/ppc_syv682c.c) | 48 |
2 files changed, 27 insertions, 23 deletions
diff --git a/zephyr/test/drivers/CMakeLists.txt b/zephyr/test/drivers/CMakeLists.txt index 6ba55f51ec..33163427b5 100644 --- a/zephyr/test/drivers/CMakeLists.txt +++ b/zephyr/test/drivers/CMakeLists.txt @@ -53,7 +53,7 @@ target_sources(app PRIVATE src/panic.c src/power_common.c src/ppc_sn5s330.c - src/ppc_syv682c.c + src/ppc_syv682x.c src/ps8xxx.c src/smart.c src/stm_mems_common.c diff --git a/zephyr/test/drivers/src/ppc_syv682c.c b/zephyr/test/drivers/src/ppc_syv682x.c index 6bfcca1c9e..9f5330478f 100644 --- a/zephyr/test/drivers/src/ppc_syv682c.c +++ b/zephyr/test/drivers/src/ppc_syv682x.c @@ -23,7 +23,7 @@ #define SYV682X_ORD DT_DEP_ORD(DT_NODELABEL(syv682x_emul)) #define GPIO_USB_C1_FRS_EN_PATH DT_PATH(named_gpios, usb_c1_frs_en) -struct ppc_syv682c_fixture { +struct ppc_syv682x_fixture { struct i2c_emul *ppc_emul; const struct device *frs_en_gpio_port; int frs_en_gpio_pin; @@ -37,9 +37,9 @@ struct reg_to_fail_data { static const int syv682x_port = 1; -static void *syv682c_test_setup(void) +static void *syv682x_test_setup(void) { - static struct ppc_syv682c_fixture fixture; + static struct ppc_syv682x_fixture fixture; fixture.ppc_emul = syv682x_emul_get(SYV682X_ORD); fixture.frs_en_gpio_port = @@ -51,7 +51,7 @@ static void *syv682c_test_setup(void) static void syv682x_test_after(void *data) { - struct ppc_syv682c_fixture *fixture = data; + struct ppc_syv682x_fixture *fixture = data; struct i2c_emul *emul = fixture->ppc_emul; syv682x_emul_set_condition(emul, SYV682X_STATUS_NONE, @@ -66,11 +66,15 @@ static void syv682x_test_after(void *data) i2c_common_emul_set_write_fail_reg(emul, I2C_COMMON_EMUL_NO_FAIL_REG); } -ZTEST_SUITE(ppc_syv682c, drivers_predicate_post_main, syv682c_test_setup, NULL, +ZTEST_SUITE(ppc_syv682x, drivers_predicate_post_main, syv682x_test_setup, NULL, syv682x_test_after, NULL); -ZTEST_F(ppc_syv682c, test_syv682x_board_is_syv682c) +ZTEST_F(ppc_syv682x, test_syv682x_board_is_syv682c) { + /* + * The SYV682x driver should assume a version-C part in the absence of a + * board override. + */ zassert_true(syv682x_board_is_syv682c(syv682x_port), NULL); } @@ -95,7 +99,7 @@ static void check_control_1_default_init(uint8_t control_1) "Default init, but 5V power path selected"); } -ZTEST_F(ppc_syv682c, test_syv682x_init) +ZTEST_F(ppc_syv682x, test_syv682x_init) { uint8_t reg; int ilim; @@ -187,7 +191,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_init) SYV682X_CONTROL_4_NONE); } -ZTEST_F(ppc_syv682c, test_syv682x_vbus_enable) +ZTEST_F(ppc_syv682x, test_syv682x_vbus_enable) { uint8_t reg; @@ -210,7 +214,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_vbus_enable) "PPC is not sourcing VBUS after VBUS enabled"); } -ZTEST_F(ppc_syv682c, test_syv682x_interrupt) +ZTEST_F(ppc_syv682x, test_syv682x_interrupt) { uint8_t reg; @@ -364,7 +368,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_interrupt) SYV682X_CONTROL_4_NONE); } -ZTEST_F(ppc_syv682c, test_syv682x_frs) +ZTEST_F(ppc_syv682x, test_syv682x_frs) { const struct device *gpio_dev = DEVICE_DT_GET(DT_GPIO_CTLR(GPIO_USB_C1_FRS_EN_PATH, gpios)); @@ -422,7 +426,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_frs) SYV682X_CONTROL_4_NONE); } -ZTEST_F(ppc_syv682c, test_syv682x_source_current_limit) +ZTEST_F(ppc_syv682x, test_syv682x_source_current_limit) { uint8_t reg; int ilim_val; @@ -458,7 +462,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_source_current_limit) "Set 3.0A Rp value, but 5V_ILIM is %d", ilim_val); } -ZTEST_F(ppc_syv682c, test_syv682x_write_busy) +ZTEST_F(ppc_syv682x, test_syv682x_write_busy) { /* * Writes should fail while the BUSY bit is set, except that writes to @@ -486,7 +490,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_write_busy) syv682x_emul_set_busy_reads(this->ppc_emul, 0); } -ZTEST_F(ppc_syv682c, test_syv682x_dev_is_connected) +ZTEST_F(ppc_syv682x, test_syv682x_dev_is_connected) { uint8_t reg; @@ -510,7 +514,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_dev_is_connected) "Could not connect device as source"); } -ZTEST_F(ppc_syv682c, test_syv682x_vbus_sink_enable) +ZTEST_F(ppc_syv682x, test_syv682x_vbus_sink_enable) { uint8_t reg; int ilim; @@ -554,7 +558,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_vbus_sink_enable) "Sink disabled, but power path enabled"); } -ZTEST_F(ppc_syv682c, test_syv682x_vbus_sink_oc_limit) +ZTEST_F(ppc_syv682x, test_syv682x_vbus_sink_oc_limit) { zassert_ok(ppc_vbus_sink_enable(syv682x_port, true), "Sink enable failed"); @@ -582,7 +586,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_vbus_sink_oc_limit) "Sink disable failed"); } -ZTEST_F(ppc_syv682c, test_syv682x_set_vconn) +ZTEST_F(ppc_syv682x, test_syv682x_set_vconn) { syv682x_emul_set_condition(this->ppc_emul, SYV682X_STATUS_NONE, SYV682X_CONTROL_4_VBAT_OVP); @@ -590,7 +594,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_set_vconn) "VBAT OVP, but ppc_set_vconn succeeded"); } -ZTEST(ppc_syv682c, test_syv682x_ppc_dump) +ZTEST(ppc_syv682x, test_syv682x_ppc_dump) { /* * The ppc_dump command should succeed for this port. Don't check the @@ -618,7 +622,7 @@ static int mock_read_intercept_reg_fail(struct i2c_emul *emul, int reg, return 1; } -ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_status) +ZTEST_F(ppc_syv682x, test_syv682x_i2c_error_status) { /* Failed STATUS read should cause init to fail. */ i2c_common_emul_set_read_fail_reg(this->ppc_emul, SYV682X_STATUS_REG); @@ -628,7 +632,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_status) I2C_COMMON_EMUL_NO_FAIL_REG); } -ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_1) +ZTEST_F(ppc_syv682x, test_syv682x_i2c_error_control_1) { const struct ppc_drv *drv = ppc_chips[syv682x_port].drv; struct reg_to_fail_data reg_fail = { @@ -695,7 +699,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_1) I2C_COMMON_EMUL_NO_FAIL_REG); } -ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_2) +ZTEST_F(ppc_syv682x, test_syv682x_i2c_error_control_2) { /* Failed CONTROL_2 read */ i2c_common_emul_set_read_fail_reg(this->ppc_emul, @@ -714,7 +718,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_2) I2C_COMMON_EMUL_NO_FAIL_REG); } -ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_3) +ZTEST_F(ppc_syv682x, test_syv682x_i2c_error_control_3) { /* Failed CONTROL_3 read */ i2c_common_emul_set_read_fail_reg(this->ppc_emul, @@ -733,7 +737,7 @@ ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_3) I2C_COMMON_EMUL_NO_FAIL_REG); } -ZTEST_F(ppc_syv682c, test_syv682x_i2c_error_control_4) +ZTEST_F(ppc_syv682x, test_syv682x_i2c_error_control_4) { /* Failed CONTROL_4 read */ i2c_common_emul_set_read_fail_reg(this->ppc_emul, |