drivers: sensor: lis2de12: add length check in spi write incr routine

narangmayank · nashif · commit 8bbbf1d5b652 · 2025-01-29T10:11:57.000-05:00
Added a length check in the spi write incr routine to handle both single and multi byte write operations properly. Signed-off-by: Mayank Narang <narang.may77@gmail.com> (cherry picked from commit cb60881)
diff --git a/drivers/sensor/st/stmemsc/stmemsc_spi.c b/drivers/sensor/st/stmemsc/stmemsc_spi.c
@@ -73,6 +73,9 @@ int stmemsc_spi_read_incr(const struct spi_dt_spec *stmemsc,
 int stmemsc_spi_write_incr(const struct spi_dt_spec *stmemsc,
 			   uint8_t reg_addr, uint8_t *value, uint8_t len)
 {
-	reg_addr |= STMEMSC_SPI_ADDR_AUTO_INCR;
+	if (len > 1) {
+		reg_addr |= STMEMSC_SPI_ADDR_AUTO_INCR;
+	}
+
 	return stmemsc_spi_write(stmemsc, reg_addr, value, len);
 }

Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,9 @@ int stmemsc_spi_read_incr(const struct spi_dt_spec *stmemsc,`
`73`	`73`	`int stmemsc_spi_write_incr(const struct spi_dt_spec *stmemsc,`
`74`	`74`	`uint8_t reg_addr, uint8_t *value, uint8_t len)`
`75`	`75`	`{`
`76`		`- reg_addr \|= STMEMSC_SPI_ADDR_AUTO_INCR;`
	`76`	`+ if (len > 1) {`
	`77`	`+ reg_addr \|= STMEMSC_SPI_ADDR_AUTO_INCR;`
	`78`	`+ }`
	`79`	`+`
`77`	`80`	`return stmemsc_spi_write(stmemsc, reg_addr, value, len);`
`78`	`81`	`}`