Merge remote-tracking branch cros/main into firmware-fpmcu-dartmonkey-releasefirmware-fpmcu-dartmonkey-release

Generated by: ./util/update_release_branch.py --board dartmonkey --relevant_paths_file
./util/fingerprint-relevant-paths.txt firmware-fpmcu-dartmonkey-release

Relevant changes:

git log --oneline 6c1587ca70..28712dae9d -- board/nocturne_fp
board/dartmonkey common/fpsensor docs/fingerprint driver/fingerprint
util/getversion.sh

ded9307b79 util/getversion.sh: Fix version when not in a git repo
956055e692 board: change Google USB vendor info
71b2ef709d Update license boilerplate text in source code files
33e11afda0 Revert "fpsensor: Build fpsensor source file with C++"
c8d0360723 fpsensor: Build fpsensor source file with C++
bc113abd53 fpsensor: Fix g++ compiler error
150a58a0dc fpsensor: Fix fp_set_sensor_mode return type
b33b5ce85b fpsensor: Remove nested designators for C++ compatibility
2e864b2539 tree-wide: const-ify argv for console commands
56d8b360f9 test: Add test for get ikm failure when seed not set
3a3d6c3690 test: Add test for fpsensor trivial key failure
233e6bbd08 fpsensor_crypto: Abstract calls to hmac_SHA256
0a041b285b docs/fingerprint: Typo correction
c03fab67e2 docs/fingerprint: Fix the path of fputils.py
0b5d4baf5a util/getversion.sh: Fix empty file list handling
6e128fe760 FPMCU dev board environment with Satlab
3eb29b6aa5 builtin: Move ssize_t to sys/types.h
345d62ebd1 docs/fingerprint: Update power numbers for latest dartmonkey release
c25ffdb316 common: Conditionally support printf %l and %i modifiers
9a3c514b45 test: Add a test to check if the debugger is connected
54e603413f Move standard library tests to their own file
43fa6b4bf8 docs/fingerprint: Update power numbers for latest bloonchipper release
25536f9a84 driver/fingerprint/fpc/bep/fpc_sensor_spi.c: Format with clang-format
4face99efd driver/fingerprint/fpc/libfp/fpc_sensor_pal.h: Format with clang-format
738de2b575 trng: Rename rand to trng_rand
14b8270edd docs/fingerprint: Update dragonclaw power numbers
0b268f93d1 driver/fingerprint/fpc/libfp/fpc_private.c: Format with clang-format
f80da163f2 driver/fingerprint/fpc/libfp/fpc_private.h: Format with clang-format
a0751778f4 board/nocturne_fp/ro_workarounds.c: Format with clang-format
5e9c85c9b1 driver/fingerprint/fpc/libfp/fpc_sensor_pal.c: Format with clang-format
c1f9dd3cf8 driver/fingerprint/fpc/libfp/fpc_bio_algorithm.h: Format with clang-format
eb1e1bed8d driver/fingerprint/fpc/libfp/fpc1145_private.h: Format with clang-format
6e7b611821 driver/fingerprint/fpc/bep/fpc_bio_algorithm.h: Format with clang-format
e0589cd5e2 driver/fingerprint/fpc/bep/fpc1035_private.h: Format with clang-format
58f0246dbe board/nocturne_fp/board_ro.c: Format with clang-format
7905e556a0 common/fpsensor/fpsensor_crypto.c: Format with clang-format
21289d170c driver/fingerprint/fpc/bep/fpc1025_private.h: Format with clang-format
98a20f937e common/fpsensor/fpsensor_state.c: Format with clang-format
a2d255d8af common/fpsensor/fpsensor.c: Format with clang-format
84e53a65da board/nocturne_fp/board.h: Format with clang-format
73055eeb3f driver/fingerprint/fpc/bep/fpc_private.c: Format with clang-format
0f7b5cb509 common/fpsensor/fpsensor_private.h: Format with clang-format
1ceade6e65 driver/fingerprint/fpc/bep/fpc_private.h: Format with clang-format
dca9d74321 Revert "trng: Rename rand to trng_rand"
a6b0b3554f trng: Rename rand to trng_rand
28d0b75b70 third_party/boringssl: Remove unused header

BRANCH=None
BUG=b:244387210 b:242720240 b:215613183 b:242720910 b:236386294
BUG=b:234181908 b:244781166 b:234781655 b:234143158 b:234181908
BUG=b:237344361 b:236025198 b:234181908 b:180945056 chromium:1098010
BUG=b:246424843 b:234181908 b:131913998
TEST=`make -j buildall`
TEST=./util/run_device_tests.py --board dartmonkey
     Test "aes": PASSED
     Test "cec": PASSED
     Test "cortexm_fpu": PASSED
     Test "crc": PASSED
     Test "flash_physical": PASSED
     Test "flash_write_protect": PASSED
     Test "fpsensor_hw": PASSED
     Test "fpsensor_spi_ro": PASSED
     Test "fpsensor_spi_rw": PASSED
     Test "fpsensor_uart_ro": PASSED
     Test "fpsensor_uart_rw": PASSED
     Test "mpu_ro": PASSED
     Test "mpu_rw": PASSED
     Test "mutex": PASSED
     Test "pingpong": PASSED
     Test "printf": PASSED
     Test "queue": PASSED
     Test "rollback_region0": PASSED
     Test "rollback_region1": PASSED
     Test "rollback_entropy": PASSED
     Test "rtc": PASSED
     Test "sha256": PASSED
     Test "sha256_unrolled": PASSED
     Test "static_if": PASSED
     Test "stdlib": PASSED
     Test "system_is_locked_wp_on": PASSED
     Test "system_is_locked_wp_off": PASSED
     Test "timer_dos": PASSED
     Test "utils": PASSED
     Test "utils_str": PASSED
     Test "panic_data_dartmonkey_v2.0.2887": PASSED
     Test "panic_data_nocturne_fp_v2.2.64": PASSED
     Test "panic_data_nami_fp_v2.2.144": PASSED
Force-Relevant-Builds: all
Signed-off-by: Tom Hughes <tomhughes@chromium.org>

Change-Id: I2c312583a709fedae8fe11d92c22328c3b634bc7

1 files changed, 685 insertions, 0 deletions

diff --git a/zephyr/test/drivers/default/src/ln9310.c b/zephyr/test/drivers/default/src/ln9310.c
new file mode 100644
index 0000000000..326e8480ef
--- /dev/null
+++ b/zephyr/test/drivers/default/src/ln9310.c
@@ -0,0 +1,685 @@
+/* Copyright 2021 The ChromiumOS Authors
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#include <zephyr/ztest.h>
+#include <zephyr/drivers/emul.h>
+#include <zephyr/kernel.h>
+#include <zephyr/ztest_assert.h>
+#include <zephyr/drivers/i2c_emul.h>
+
+#include "driver/ln9310.h"
+#include "emul/emul_ln9310.h"
+#include "emul/emul_common_i2c.h"
+#include "timer.h"
+#include "test/drivers/test_state.h"
+
+/*
+ * TODO(b/201420132): Implement approach for tests to immediately schedule work
+ * to avoid any sleeping
+ */
+#define TEST_DELAY_MS 50
+
+#define EMUL_LN9310_NODE DT_NODELABEL(ln9310)
+
+/*
+ * Chip revisions below LN9310_BC_STS_C_CHIP_REV_FIXED require an alternative
+ * software startup to properly initialize and power up.
+ */
+#define REQUIRES_CFLY_PRECHARGE_STARTUP_CHIP_REV \
+	(LN9310_BC_STS_C_CHIP_REV_FIXED - 1)
+
+ZTEST(ln9310, test_ln9310_read_chip_fails)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery and chip rev won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	i2c_common_emul_set_read_fail_reg(common_data, LN9310_REG_BC_STS_C);
+
+	zassert_true(ln9310_init() != 0, NULL);
+	zassert_false(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_read_fail_reg(common_data,
+					  I2C_COMMON_EMUL_NO_FAIL_REG);
+}
+
+ZTEST(ln9310, test_ln9310_2s_powers_up)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	ln9310_software_enable(true);
+
+	k_msleep(TEST_DELAY_MS);
+	zassert_true(ln9310_power_good(), NULL);
+}
+
+ZTEST(ln9310, test_ln9310_3s_powers_up)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_3S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	ln9310_software_enable(true);
+
+	k_msleep(TEST_DELAY_MS);
+	zassert_true(ln9310_power_good(), NULL);
+}
+
+struct startup_workaround_data {
+	bool startup_workaround_attempted;
+	bool startup_workaround_should_fail;
+};
+
+static int mock_write_fn_intercept_startup_workaround(const struct emul *emul,
+						      int reg, uint8_t val,
+						      int bytes, void *data)
+{
+	struct startup_workaround_data *test_data = data;
+
+	uint8_t startup_workaround_val =
+		(LN9310_TEST_MODE_CTRL_FORCE_SC_OUT_PRECHARGE_ON |
+		 LN9310_TEST_MODE_CTRL_FORCE_SC_OUT_PREDISCHARGE_ON);
+
+	test_data->startup_workaround_attempted =
+		test_data->startup_workaround_attempted ||
+		((reg == LN9310_REG_TEST_MODE_CTRL) &&
+		 (val == startup_workaround_val));
+
+	if (test_data->startup_workaround_should_fail)
+		return -1;
+
+	return 1;
+}
+
+ZTEST(ln9310, test_ln9310_2s_cfly_precharge_startup)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+
+	struct startup_workaround_data test_data = {
+		.startup_workaround_attempted = false,
+		.startup_workaround_should_fail = false,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator,
+				REQUIRES_CFLY_PRECHARGE_STARTUP_CHIP_REV);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(
+		common_data, mock_write_fn_intercept_startup_workaround,
+		&test_data);
+
+	ln9310_software_enable(true);
+	zassert_true(test_data.startup_workaround_attempted, NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_true(ln9310_power_good(), NULL);
+
+	ln9310_software_enable(false);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(common_data, NULL, NULL);
+}
+
+ZTEST(ln9310, test_ln9310_3s_cfly_precharge_startup)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+
+	struct startup_workaround_data test_data = {
+		.startup_workaround_attempted = false,
+		.startup_workaround_should_fail = false,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_3S);
+	ln9310_emul_set_version(emulator,
+				REQUIRES_CFLY_PRECHARGE_STARTUP_CHIP_REV);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(
+		common_data, mock_write_fn_intercept_startup_workaround,
+		&test_data);
+
+	ln9310_software_enable(true);
+	zassert_true(test_data.startup_workaround_attempted, NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_true(ln9310_power_good(), NULL);
+
+	ln9310_software_enable(false);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(common_data, NULL, NULL);
+}
+
+ZTEST(ln9310, test_ln9310_cfly_precharge_exceeds_retries)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+
+	struct startup_workaround_data test_data = {
+		.startup_workaround_attempted = false,
+		.startup_workaround_should_fail = true,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/*
+	 * Battery and chip rev won't matter for statement
+	 * coverage here so only testing one pair.
+	 */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator,
+				REQUIRES_CFLY_PRECHARGE_STARTUP_CHIP_REV);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(
+		common_data, mock_write_fn_intercept_startup_workaround,
+		&test_data);
+
+	ln9310_software_enable(true);
+	zassert_true(test_data.startup_workaround_attempted, NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(common_data, NULL, NULL);
+}
+
+ZTEST(ln9310, test_ln9310_battery_unknown)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/*
+	 * Chip rev won't matter for statement
+	 * cov so only testing one version.
+	 */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_UNKNOWN);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_true(ln9310_init() != 0, NULL);
+	zassert_false(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	ln9310_software_enable(true);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+}
+
+ZTEST(ln9310, test_ln9310_2s_battery_read_fails)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	i2c_common_emul_set_read_fail_reg(common_data, LN9310_REG_BC_STS_B);
+
+	zassert_true(ln9310_init() != 0, NULL);
+	zassert_false(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	/* For Battery 2S Versions: Test Read Battery Voltage Failure Too */
+	ln9310_emul_reset(emulator);
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	i2c_common_emul_set_read_fail_reg(common_data, LN9310_REG_TRACK_CTRL);
+
+	zassert_false(ln9310_init() == 0, NULL);
+	i2c_common_emul_set_read_fail_reg(common_data,
+					  I2C_COMMON_EMUL_NO_FAIL_REG);
+}
+
+ZTEST(ln9310, test_ln9310_lion_ctrl_reg_fails)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery won't matter here so only testing one version */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	i2c_common_emul_set_read_fail_reg(common_data, LN9310_REG_LION_CTRL);
+
+	zassert_true(ln9310_init() != 0, NULL);
+	zassert_false(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	ln9310_software_enable(true);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_read_fail_reg(common_data,
+					  I2C_COMMON_EMUL_NO_FAIL_REG);
+}
+
+struct precharge_timeout_data {
+	timestamp_t time_to_mock;
+	bool handled_clearing_standby_en_bit_timeout;
+};
+
+static int mock_intercept_startup_ctrl_reg(const struct emul *emulator, int reg,
+					   uint8_t val, int bytes, void *data)
+{
+	struct precharge_timeout_data *test_data = data;
+
+	if (reg == LN9310_REG_STARTUP_CTRL &&
+	    test_data->handled_clearing_standby_en_bit_timeout == false) {
+		if (val == 0) {
+			timestamp_t time = get_time();
+
+			time.val += 1 + LN9310_CFLY_PRECHARGE_TIMEOUT;
+			test_data->time_to_mock = time;
+			get_time_mock = &test_data->time_to_mock;
+		} else {
+			/* ln9310 aborts a startup attempt */
+			test_data->handled_clearing_standby_en_bit_timeout =
+				true;
+			get_time_mock = NULL;
+		}
+	}
+	return 1;
+}
+
+ZTEST(ln9310, test_ln9310_cfly_precharge_timesout)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+	struct precharge_timeout_data test_data = {
+		.time_to_mock = {
+			.val = -1,
+			.le = {
+				.lo = -1,
+				.hi = -1,
+			},
+		},
+		.handled_clearing_standby_en_bit_timeout = false,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery and chip rev won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator,
+				REQUIRES_CFLY_PRECHARGE_STARTUP_CHIP_REV);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(
+		common_data, mock_intercept_startup_ctrl_reg, &test_data);
+
+	ln9310_software_enable(true);
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_true(test_data.handled_clearing_standby_en_bit_timeout, NULL);
+	/* It only times out on one attempt, it should subsequently startup */
+	zassert_true(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_write_func(common_data, NULL, NULL);
+}
+
+struct reg_to_fail_data {
+	int reg_access_to_fail;
+	int reg_access_fail_countdown;
+};
+
+static int mock_read_intercept_reg_to_fail(const struct emul *emul, int reg,
+					   uint8_t *val, int bytes, void *data)
+{
+	struct reg_to_fail_data *test_data = data;
+
+	if (reg == test_data->reg_access_to_fail) {
+		test_data->reg_access_fail_countdown--;
+		if (test_data->reg_access_fail_countdown <= 0)
+			return -1;
+	}
+	return 1;
+}
+
+ZTEST(ln9310, test_ln9310_interrupt_reg_fail)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+	struct reg_to_fail_data test_data = {
+		.reg_access_to_fail = 0,
+		.reg_access_fail_countdown = 0,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery and chip rev won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	i2c_common_emul_set_read_func(
+		common_data, mock_read_intercept_reg_to_fail, &test_data);
+
+	/* Fail in beginning of software enable */
+	test_data.reg_access_to_fail = LN9310_REG_INT1;
+	test_data.reg_access_fail_countdown = 1;
+
+	ln9310_software_enable(true);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+	zassert_true(test_data.reg_access_fail_countdown <= 0, NULL);
+
+	/* Fail in irq interrupt handler */
+	test_data.reg_access_fail_countdown = 2;
+
+	ln9310_software_enable(true);
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+	zassert_false(ln9310_power_good(), NULL);
+	zassert_true(test_data.reg_access_fail_countdown <= 0, NULL);
+
+	i2c_common_emul_set_read_func(common_data, NULL, NULL);
+}
+
+ZTEST(ln9310, test_ln9310_sys_sts_reg_fail)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+	struct reg_to_fail_data test_data = {
+		.reg_access_to_fail = 0,
+		.reg_access_fail_countdown = 0,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery and chip rev won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	i2c_common_emul_set_read_func(
+		common_data, &mock_read_intercept_reg_to_fail, &test_data);
+
+	/* Register only read once and in the interrupt handler */
+	test_data.reg_access_to_fail = LN9310_REG_SYS_STS;
+	test_data.reg_access_fail_countdown = 1;
+
+	ln9310_software_enable(1);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+
+	zassert_false(ln9310_power_good(), NULL);
+	zassert_true(test_data.reg_access_fail_countdown <= 0, NULL);
+
+	i2c_common_emul_set_read_func(common_data, NULL, NULL);
+}
+
+struct reg_to_intercept {
+	int reg;
+	uint8_t replace_val;
+};
+
+static int mock_read_interceptor(const struct emul *emul, int reg, uint8_t *val,
+				 int bytes, void *data)
+{
+	struct reg_to_intercept *test_data = data;
+
+	if (test_data->reg == reg) {
+		*val = test_data->replace_val;
+		return 0;
+	}
+
+	return 1;
+}
+
+ZTEST(ln9310, test_ln9310_reset_explicit_detected_startup)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+	struct reg_to_intercept test_data = {
+		.reg = LN9310_REG_LION_CTRL,
+		.replace_val = 0,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery and chip rev won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	i2c_common_emul_set_read_func(common_data, &mock_read_interceptor,
+				      &test_data);
+
+	ln9310_software_enable(true);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+
+	zassert_true(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_read_func(common_data, NULL, NULL);
+}
+
+ZTEST(ln9310, test_ln9310_update_startup_seq_fails)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+	struct reg_to_fail_data test_data = {
+		.reg_access_to_fail = LN9310_REG_CFG_4,
+		.reg_access_fail_countdown = 1,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	/* Requires older version of chip */
+	ln9310_emul_set_version(emulator,
+				REQUIRES_CFLY_PRECHARGE_STARTUP_CHIP_REV);
+
+	i2c_common_emul_set_read_func(
+		common_data, &mock_read_intercept_reg_to_fail, &test_data);
+
+	zassert_false(ln9310_init() == 0, NULL);
+	zassert_false(ln9310_emul_is_init(emulator), NULL);
+
+	ln9310_software_enable(true);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+
+	zassert_false(ln9310_power_good(), NULL);
+	zassert_true(test_data.reg_access_fail_countdown <= 0, NULL);
+
+	i2c_common_emul_set_read_func(common_data, NULL, NULL);
+}
+
+ZTEST(ln9310, test_ln9310_state_change_only_on_mode_change_interrupt)
+{
+	const struct emul *emulator = EMUL_DT_GET(EMUL_LN9310_NODE);
+	struct i2c_common_emul_data *common_data =
+		emul_ln9310_get_i2c_common_data(emulator);
+	struct reg_to_intercept test_data = {
+		.reg = LN9310_REG_INT1,
+		.replace_val = 0,
+	};
+
+	zassert_not_null(emulator, NULL);
+
+	ln9310_emul_set_context(emulator);
+	ln9310_emul_reset(emulator);
+	/* Battery and chip rev won't matter here so only testing one pair */
+	ln9310_emul_set_battery_cell_type(emulator, BATTERY_CELL_TYPE_2S);
+	ln9310_emul_set_version(emulator, LN9310_BC_STS_C_CHIP_REV_FIXED);
+
+	zassert_ok(ln9310_init(), NULL);
+	zassert_true(ln9310_emul_is_init(emulator), NULL);
+
+	i2c_common_emul_set_read_func(common_data, &mock_read_interceptor,
+				      &test_data);
+
+	ln9310_software_enable(true);
+
+	/* TODO(b/201420132) */
+	k_msleep(TEST_DELAY_MS);
+
+	zassert_false(ln9310_power_good(), NULL);
+
+	i2c_common_emul_set_read_func(common_data, NULL, NULL);
+}
+
+static inline void reset_ln9310_state(void)
+{
+	ln9310_reset_to_initial_state();
+	get_time_mock = NULL;
+}
+
+static void ln9310_before(void *state)
+{
+	ARG_UNUSED(state);
+	reset_ln9310_state();
+}
+
+static void ln9310_after(void *state)
+{
+	ARG_UNUSED(state);
+	reset_ln9310_state();
+}
+
+ZTEST_SUITE(ln9310, drivers_predicate_post_main, NULL, ln9310_before,
+	    ln9310_after, NULL);