Re: [PATCH 6/7] platform/chrome: cros_ec_proto: add Kunit tests for get_sensor_count

From: Guenter Roeck
Date: Thu Jun 23 2022 - 12:39:50 EST

Next message: Bjorn Helgaas: "Re: [PATCH v4 0/2] dt-bindings: PCI: uniphier: Fix endpoint descriptions"
Previous message: Andy Shevchenko: "Re: [PATCH v2 2/2] of: base: Avoid console probe delay when fw_devlink.strict=1"
In reply to: Tzung-Bi Shih: "[PATCH 6/7] platform/chrome: cros_ec_proto: add Kunit tests for get_sensor_count"
Next in thread: Tzung-Bi Shih: "[PATCH 7/7] platform/chrome: cros_ec_proto: add Kunit test for cros_ec_cmd()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Tue, Jun 21, 2022 at 9:11 PM Tzung-Bi Shih <tzungbi@xxxxxxxxxx> wrote:
>
> cros_ec_get_sensor_count() gets number of MEMS sensors.
>
> Add Kunit tests for cros_ec_get_sensor_count().
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@xxxxxxxxxx>

Reviewed-by: Guenter Roeck <groeck@xxxxxxxxxxxx>

> ---
> drivers/platform/chrome/cros_ec_proto_test.c | 153 +++++++++++++++++++
> drivers/platform/chrome/cros_kunit_util.c | 22 +++
> drivers/platform/chrome/cros_kunit_util.h | 7 +
> 3 files changed, 182 insertions(+)
>
> diff --git a/drivers/platform/chrome/cros_ec_proto_test.c b/drivers/platform/chrome/cros_ec_proto_test.c
> index 93c1700deaef..6b26ce3104f4 100644
> --- a/drivers/platform/chrome/cros_ec_proto_test.c
> +++ b/drivers/platform/chrome/cros_ec_proto_test.c
> @@ -2442,6 +2442,156 @@ static void cros_ec_proto_test_check_features_not_cached(struct kunit *test)
> }
> }
>
> +static void cros_ec_proto_test_get_sensor_count_normal(struct kunit *test)
> +{
> + struct cros_ec_proto_test_priv *priv = test->priv;
> + struct cros_ec_device *ec_dev = &priv->ec_dev;
> + struct ec_xfer_mock *mock;
> + int ret;
> + struct cros_ec_dev ec;
> +
> + ec_dev->max_request = 0xff;
> + ec_dev->max_response = 0xee;
> + ec.ec_dev = ec_dev;
> + ec.dev = ec_dev->dev;
> + ec.cmd_offset = 0;
> +
> + /* For EC_CMD_MOTION_SENSE_CMD. */
> + {
> + struct ec_response_motion_sense *data;
> +
> + mock = cros_kunit_ec_xfer_mock_add(test, sizeof(*data));
> + KUNIT_ASSERT_PTR_NE(test, mock, NULL);
> +
> + data = (struct ec_response_motion_sense *)mock->o_data;
> + data->dump.sensor_count = 0xbf;
> + }
> +
> + ret = cros_ec_get_sensor_count(&ec);
> + KUNIT_EXPECT_EQ(test, ret, 0xbf);
> +
> + /* For EC_CMD_MOTION_SENSE_CMD. */
> + {
> + struct ec_params_motion_sense *data;
> +
> + mock = cros_kunit_ec_xfer_mock_next();
> + KUNIT_EXPECT_PTR_NE(test, mock, NULL);
> +
> + KUNIT_EXPECT_EQ(test, mock->msg.version, 1);
> + KUNIT_EXPECT_EQ(test, mock->msg.command, EC_CMD_MOTION_SENSE_CMD);
> + KUNIT_EXPECT_EQ(test, mock->msg.insize, sizeof(struct ec_response_motion_sense));
> + KUNIT_EXPECT_EQ(test, mock->msg.outsize, sizeof(*data));
> +
> + data = (struct ec_params_motion_sense *)mock->i_data;
> + KUNIT_EXPECT_EQ(test, data->cmd, MOTIONSENSE_CMD_DUMP);
> + }
> +}
> +
> +static void cros_ec_proto_test_get_sensor_count_xfer_error(struct kunit *test)
> +{
> + struct cros_ec_proto_test_priv *priv = test->priv;
> + struct cros_ec_device *ec_dev = &priv->ec_dev;
> + struct ec_xfer_mock *mock;
> + int ret;
> + struct cros_ec_dev ec;
> +
> + ec_dev->max_request = 0xff;
> + ec_dev->max_response = 0xee;
> + ec.ec_dev = ec_dev;
> + ec.dev = ec_dev->dev;
> + ec.cmd_offset = 0;
> +
> + /* For EC_CMD_MOTION_SENSE_CMD. */
> + {
> + mock = cros_kunit_ec_xfer_mock_addx(test, -EPROTO, EC_RES_SUCCESS, 0);
> + KUNIT_ASSERT_PTR_NE(test, mock, NULL);
> + }
> +
> + ret = cros_ec_get_sensor_count(&ec);
> + KUNIT_EXPECT_EQ(test, ret, -EPROTO);
> +
> + /* For EC_CMD_MOTION_SENSE_CMD. */
> + {
> + struct ec_params_motion_sense *data;
> +
> + mock = cros_kunit_ec_xfer_mock_next();
> + KUNIT_EXPECT_PTR_NE(test, mock, NULL);
> +
> + KUNIT_EXPECT_EQ(test, mock->msg.version, 1);
> + KUNIT_EXPECT_EQ(test, mock->msg.command, EC_CMD_MOTION_SENSE_CMD);
> + KUNIT_EXPECT_EQ(test, mock->msg.insize, sizeof(struct ec_response_motion_sense));
> + KUNIT_EXPECT_EQ(test, mock->msg.outsize, sizeof(*data));
> +
> + data = (struct ec_params_motion_sense *)mock->i_data;
> + KUNIT_EXPECT_EQ(test, data->cmd, MOTIONSENSE_CMD_DUMP);
> + }
> +}
> +
> +static void cros_ec_proto_test_get_sensor_count_legacy(struct kunit *test)
> +{
> + struct cros_ec_proto_test_priv *priv = test->priv;
> + struct cros_ec_device *ec_dev = &priv->ec_dev;
> + struct ec_xfer_mock *mock;
> + int ret, i;
> + struct cros_ec_dev ec;
> + struct {
> + u8 readmem_data;
> + int expected_result;
> + } test_data[] = {
> + { 0, 0 },
> + { EC_MEMMAP_ACC_STATUS_PRESENCE_BIT, 2 },
> + };
> +
> + ec_dev->max_request = 0xff;
> + ec_dev->max_response = 0xee;
> + ec_dev->cmd_readmem = cros_kunit_readmem_mock;
> + ec.ec_dev = ec_dev;
> + ec.dev = ec_dev->dev;
> + ec.cmd_offset = 0;
> +
> + for (i = 0; i < ARRAY_SIZE(test_data); ++i) {
> + /* For EC_CMD_MOTION_SENSE_CMD. */
> + {
> + mock = cros_kunit_ec_xfer_mock_addx(test, -EPROTO, EC_RES_SUCCESS, 0);
> + KUNIT_ASSERT_PTR_NE(test, mock, NULL);
> + }
> +
> + /* For readmem. */
> + {
> + cros_kunit_readmem_mock_data = kunit_kzalloc(test, 1, GFP_KERNEL);
> + KUNIT_ASSERT_PTR_NE(test, cros_kunit_readmem_mock_data, NULL);
> + cros_kunit_readmem_mock_data[0] = test_data[i].readmem_data;
> +
> + cros_kunit_ec_xfer_mock_default_ret = 1;
> + }
> +
> + ret = cros_ec_get_sensor_count(&ec);
> + KUNIT_EXPECT_EQ(test, ret, test_data[i].expected_result);
> +
> + /* For EC_CMD_MOTION_SENSE_CMD. */
> + {
> + struct ec_params_motion_sense *data;
> +
> + mock = cros_kunit_ec_xfer_mock_next();
> + KUNIT_EXPECT_PTR_NE(test, mock, NULL);
> +
> + KUNIT_EXPECT_EQ(test, mock->msg.version, 1);
> + KUNIT_EXPECT_EQ(test, mock->msg.command, EC_CMD_MOTION_SENSE_CMD);
> + KUNIT_EXPECT_EQ(test, mock->msg.insize,
> + sizeof(struct ec_response_motion_sense));
> + KUNIT_EXPECT_EQ(test, mock->msg.outsize, sizeof(*data));
> +
> + data = (struct ec_params_motion_sense *)mock->i_data;
> + KUNIT_EXPECT_EQ(test, data->cmd, MOTIONSENSE_CMD_DUMP);
> + }
> +
> + /* For readmem. */
> + {
> + KUNIT_EXPECT_EQ(test, cros_kunit_readmem_mock_offset, EC_MEMMAP_ACC_STATUS);
> + }
> + }
> +}
> +
> static void cros_ec_proto_test_release(struct device *dev)
> {
> }
> @@ -2537,6 +2687,9 @@ static struct kunit_case cros_ec_proto_test_cases[] = {
> KUNIT_CASE(cros_ec_proto_test_get_host_event_normal),
> KUNIT_CASE(cros_ec_proto_test_check_features_cached),
> KUNIT_CASE(cros_ec_proto_test_check_features_not_cached),
> + KUNIT_CASE(cros_ec_proto_test_get_sensor_count_normal),
> + KUNIT_CASE(cros_ec_proto_test_get_sensor_count_xfer_error),
> + KUNIT_CASE(cros_ec_proto_test_get_sensor_count_legacy),
> {}
> };
>
> diff --git a/drivers/platform/chrome/cros_kunit_util.c b/drivers/platform/chrome/cros_kunit_util.c
> index 3ede971e82ee..d37c334b416d 100644
> --- a/drivers/platform/chrome/cros_kunit_util.c
> +++ b/drivers/platform/chrome/cros_kunit_util.c
> @@ -105,6 +105,24 @@ struct ec_xfer_mock *cros_kunit_ec_xfer_mock_next(void)
> }
> EXPORT_SYMBOL_GPL(cros_kunit_ec_xfer_mock_next);
>
> +int cros_kunit_readmem_mock_offset;
> +EXPORT_SYMBOL_GPL(cros_kunit_readmem_mock_offset);
> +u8 *cros_kunit_readmem_mock_data;
> +EXPORT_SYMBOL_GPL(cros_kunit_readmem_mock_data);
> +int cros_kunit_readmem_mock_ret;
> +EXPORT_SYMBOL_GPL(cros_kunit_readmem_mock_ret);
> +
> +int cros_kunit_readmem_mock(struct cros_ec_device *ec_dev, unsigned int offset,
> + unsigned int bytes, void *dest)
> +{
> + cros_kunit_readmem_mock_offset = offset;
> +
> + memcpy(dest, cros_kunit_readmem_mock_data, bytes);
> +
> + return cros_kunit_readmem_mock_ret;
> +}
> +EXPORT_SYMBOL_GPL(cros_kunit_readmem_mock);
> +
> void cros_kunit_mock_reset(void)
> {
> cros_kunit_ec_xfer_mock_default_ret = 0;
> @@ -112,6 +130,10 @@ void cros_kunit_mock_reset(void)
> cros_kunit_ec_pkt_xfer_mock_called = 0;
> INIT_LIST_HEAD(&cros_kunit_ec_xfer_mock_in);
> INIT_LIST_HEAD(&cros_kunit_ec_xfer_mock_out);
> +
> + cros_kunit_readmem_mock_offset = 0;
> + cros_kunit_readmem_mock_data = NULL;
> + cros_kunit_readmem_mock_ret = 0;
> }
> EXPORT_SYMBOL_GPL(cros_kunit_mock_reset);
>
> diff --git a/drivers/platform/chrome/cros_kunit_util.h b/drivers/platform/chrome/cros_kunit_util.h
> index ae4080cb13f1..88134c9f1acf 100644
> --- a/drivers/platform/chrome/cros_kunit_util.h
> +++ b/drivers/platform/chrome/cros_kunit_util.h
> @@ -35,6 +35,13 @@ struct ec_xfer_mock *cros_kunit_ec_xfer_mock_addx(struct kunit *test,
> int ret, int result, size_t size);
> struct ec_xfer_mock *cros_kunit_ec_xfer_mock_next(void);
>
> +extern int cros_kunit_readmem_mock_offset;
> +extern u8 *cros_kunit_readmem_mock_data;
> +extern int cros_kunit_readmem_mock_ret;
> +
> +int cros_kunit_readmem_mock(struct cros_ec_device *ec_dev, unsigned int offset,
> + unsigned int bytes, void *dest);
> +
> void cros_kunit_mock_reset(void);
>
> #endif
> --
> 2.37.0.rc0.104.g0611611a94-goog
>

Next message: Bjorn Helgaas: "Re: [PATCH v4 0/2] dt-bindings: PCI: uniphier: Fix endpoint descriptions"
Previous message: Andy Shevchenko: "Re: [PATCH v2 2/2] of: base: Avoid console probe delay when fw_devlink.strict=1"
In reply to: Tzung-Bi Shih: "[PATCH 6/7] platform/chrome: cros_ec_proto: add Kunit tests for get_sensor_count"
Next in thread: Tzung-Bi Shih: "[PATCH 7/7] platform/chrome: cros_ec_proto: add Kunit test for cros_ec_cmd()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]