patrick/zephyr
1
0
Fork 0
Code Issues Pull requests Actions 8 Packages Projects Releases Wiki Activity

emul: spi: emul_bmi160: Move logging to access functions

Browse source 
Move the logging for transactions to the functions that actually access
registers and sample data. This avoids needed to repeat this code when
I2C starts calling these functions.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2020-10-08 12:57:11 -06:00 committed by Carles Cufí
parent c953c32cbd
commit e43c181934
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
subsys/emul/spi/emul_bmi160.c
View file

@ -47,6 +47,7 @@ static void sample_read(struct bmi160_emul_data *data, union bmi160_sample *buf)
{
int i;
LOG_INF("Sample read");
buf->dummy_byte = 0;
for (i = 0; i < BMI160_AXES; i++) {
/*
@ -62,6 +63,7 @@ static void reg_write(const struct bmi160_emul_cfg *cfg, int regn, int val)
{
struct bmi160_emul_data *data = cfg->data;
LOG_INF("write %x = %x", regn, val);
cfg->reg[regn] = val;
switch (regn) {
case BMI160_REG_ACC_CONF:
@ -121,6 +123,7 @@ static int reg_read(const struct bmi160_emul_cfg *cfg, int regn)
struct bmi160_emul_data *data = cfg->data;
int val;
LOG_INF("read %x =", regn);
val = cfg->reg[regn];
switch (regn) {
case BMI160_REG_CHIPID:
@ -146,6 +149,7 @@ static int reg_read(const struct bmi160_emul_cfg *cfg, int regn)
default:
LOG_INF("Unknown read %x", regn);
}
LOG_INF(" = %x", val);
return val;
}
@ -181,19 +185,15 @@ static int bmi160_emul_io(struct spi_emul *emul,
case 1:
if (regn & BMI160_REG_READ) {
regn &= BMI160_REG_MASK;
LOG_INF("read %x =", regn);
val = reg_read(cfg, regn);
*(uint8_t *)rxd->buf = val;
LOG_INF(" = %x", val);
} else {
val = *(uint8_t *)txd->buf;
LOG_INF("write %x = %x", regn, val);
reg_write(cfg, regn, val);
}
break;
case BMI160_SAMPLE_SIZE:
if (regn & BMI160_REG_READ) {
LOG_INF("Sample read");
sample_read(data, rxd->buf);
} else {
LOG_INF("Unknown sample write");