Hi again Mehdi,
Just one thing still crossed to my mind...
On 6/13/23 13:22, Mehdi Djait wrote:
Kionix KX132-1211 is a tri-axis 16-bit accelerometer that can support
ranges from ±2G to ±16G, digital output through I²C/SPI.
Add support for basic accelerometer features such as reading acceleration
via IIO using raw reads, triggered buffer (data-ready), or the WMI IRQ.
Datasheet: https://kionixfs.azureedge.net/en/document/KX132-1211-Technical-Reference-Manual-Rev-5.0.pdf
Acked-by: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Signed-off-by: Mehdi Djait <mehdi.djait.k@xxxxxxxxx>
---
+static int kx132_get_fifo_bytes(struct kx022a_data *data)
+{
+ __le16 buf_status;
+ int ret, fifo_bytes;
+
+ ret = regmap_bulk_read(data->regmap, data->chip_info->buf_status1,
+ &buf_status, sizeof(buf_status));
+ if (ret) {
+ dev_err(data->dev, "Error reading buffer status\n");
+ return ret;
+ }
+
+ fifo_bytes = le16_to_cpu(buf_status);
+ fifo_bytes &= data->chip_info->buf_smp_lvl_mask;
Does this mask guarantee the fifo_bytes can't be bigger than the
allocated buffer no matter what is read from I2C?
I am asking this because it's not uncommon to see some noise in I2C, or
read 0xff 0xff, ... if IC is for example resetting. Hence, I would sleep
a tad better if I knew the fifo_bytes read from IC is not 100% trusted -
but is verified against known maximum number of samples.
This does not matter with the KX022A because the size of max number of
samples exceeds the value indicated by 0xff (the 8bit SMP_LVL register).
Please correct me if I am wrong but it seems to me we use 10bits of data
from smp_lvl register on KX132. Hence, the maximum value if all bits are
set is 1023. This equals to 1023/6bytes samples - which is roughly 170.
I think the FIFO size was defined to be maximum of 86 samples on KX132 -
which means that if we have some bogus value from I2C, we may overwrite
allocated buffer. Hence we might want to ensure the fifo_bytes is never
bigger than the fifo_length * KX022A_FIFO_SAMPLES_SIZE_BYTES - no matter
what we read from I2C.
+
+ return fifo_bytes;
+}
+
Yours,
-- Matti
--
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland
~~ When things go utterly wrong vim users can always type :help! ~~
