Skip to content

Commit 54cd6a3

Browse files
eHammarstromnashif
authored andcommitted
drivers: sensor: lsm6dso: Change log level for failures during init
Log level in failures were changed from DBG to ERR so that the reason of initialization failure may be known. Signed-off-by: Emil Hammarström <[email protected]>
1 parent 5a2c282 commit 54cd6a3

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

drivers/sensor/st/lsm6dso/lsm6dso.c

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -700,33 +700,33 @@ static int lsm6dso_init_chip(const struct device *dev)
700700
* set the bank now.
701701
*/
702702
if (lsm6dso_mem_bank_set(ctx, LSM6DSO_USER_BANK) < 0) {
703-
LOG_DBG("Failed to set user bank");
703+
LOG_ERR("Failed to set user bank");
704704
return -EIO;
705705
}
706706

707707
if (lsm6dso_device_id_get(ctx, &chip_id) < 0) {
708-
LOG_DBG("Failed reading chip id");
708+
LOG_ERR("Failed reading chip id");
709709
return -EIO;
710710
}
711711

712712
LOG_INF("chip id 0x%x", chip_id);
713713

714714
if (chip_id != LSM6DSO_ID) {
715-
LOG_DBG("Invalid chip id 0x%x", chip_id);
715+
LOG_ERR("Invalid chip id 0x%x", chip_id);
716716
return -EIO;
717717
}
718718

719719
/* I3C disable stay preserved after s/w reset */
720720
if (lsm6dso_i3c_disable_set(ctx, LSM6DSO_I3C_DISABLE) < 0) {
721-
LOG_DBG("Failed to disable I3C");
721+
LOG_ERR("Failed to disable I3C");
722722
return -EIO;
723723
}
724724

725725
/* Per AN5192 §7.2.1, "… when applying the software reset procedure, the I2C master
726726
* must be disabled, followed by a 300 μs wait."
727727
*/
728728
if (lsm6dso_sh_master_get(ctx, &master_on) < 0) {
729-
LOG_DBG("Failed to get I2C_MASTER status");
729+
LOG_ERR("Failed to get I2C_MASTER status");
730730
return -EIO;
731731
}
732732
if (master_on) {
@@ -737,6 +737,7 @@ static int lsm6dso_init_chip(const struct device *dev)
737737

738738
/* reset device */
739739
if (lsm6dso_reset_set(ctx, 1) < 0) {
740+
LOG_ERR("Failed to reset device");
740741
return -EIO;
741742
}
742743

@@ -803,12 +804,12 @@ static int lsm6dso_init_chip(const struct device *dev)
803804

804805
/* Set FIFO bypass mode */
805806
if (lsm6dso_fifo_mode_set(ctx, LSM6DSO_BYPASS_MODE) < 0) {
806-
LOG_DBG("failed to set FIFO mode");
807+
LOG_ERR("failed to set FIFO mode");
807808
return -EIO;
808809
}
809810

810811
if (lsm6dso_block_data_update_set(ctx, 1) < 0) {
811-
LOG_DBG("failed to set BDU mode");
812+
LOG_ERR("failed to set BDU mode");
812813
return -EIO;
813814
}
814815

@@ -826,7 +827,7 @@ static int lsm6dso_init(const struct device *dev)
826827
data->dev = dev;
827828

828829
if (lsm6dso_init_chip(dev) < 0) {
829-
LOG_DBG("failed to initialize chip");
830+
LOG_ERR("failed to initialize chip");
830831
return -EIO;
831832
}
832833

0 commit comments

Comments
 (0)