linux-toradex.git/drivers/iio/pressure, branch v5.11-rc3

iio:pressure:mpl3115: Force alignment of buffer

2020-12-03T19:40:28+00:00

Whilst this is another case of the issue Lars reported with
an array of elements of smaller than 8 bytes being passed
to iio_push_to_buffers_with_timestamp(), the solution here is
a bit different from the other cases and relies on __aligned
working on the stack (true since 4.6?)

This one is unusual.  We have to do an explicit memset() each time
as we are reading 3 bytes into a potential 4 byte channel which
may sometimes be a 2 byte channel depending on what is enabled.
As such, moving the buffer to the heap in the iio_priv structure
doesn't save us much.  We can't use a nice explicit structure
on the stack either as the data channels have different storage
sizes and are all separately controlled.

Fixes: cc26ad455f57 ("iio: Add Freescale MPL3115A2 pressure / temperature sensor driver")
Reported-by: Lars-Peter Clausen 
Signed-off-by: Jonathan Cameron 
Reviewed-by: Andy Shevchenko 
Reviewed-by: Alexandru Ardelean 
Cc: Peter Meerwald 
Cc: 
Link: https://lore.kernel.org/r/20200920112742.170751-7-jic23@kernel.org

iio: pressure: bmp280: remove unneeded semicolon

2020-11-01T14:58:54+00:00

A semicolon is not needed after a switch statement.

Signed-off-by: Tom Rix 
Link: https://lore.kernel.org/r/20201031134506.2134698-1-trix@redhat.com
Signed-off-by: Jonathan Cameron

iio: cros_ec: unify hw fifo attributes into the core file

2020-09-29T16:27:07+00:00

The intent here is to minimize the use of iio_buffer_set_attrs(). Since we
are planning to add support for multiple IIO buffers per IIO device, the
issue has to do with:
1. Accessing 'indio_dev->buffer' directly (as is done with
   'iio_buffer_set_attrs(indio_dev->buffer, )').
2. The way that the buffer attributes would get handled or expanded when
   there are more buffers per IIO device. Current a sysfs kobj_type expands
   into a 'device' object that expands into an 'iio_dev' object.
   We will need to change this, so that the sysfs attributes for IIO
   buffers expand into IIO buffers at some point.

Right now, the current IIO framework works fine for the
'1 IIO device == 1 IIO buffer' case (that is now).

Signed-off-by: Alexandru Ardelean 
Reviewed-by: Guenter Roeck 
Link: https://lore.kernel.org/r/20200923130339.997902-1-alexandru.ardelean@analog.com
Signed-off-by: Jonathan Cameron

iio:pressure:zpa2326: Drop of_match_ptr protection

2020-09-21T17:41:30+00:00

This prevents use of this driver with ACPI via PRP0001 and is
an example of an anti pattern I'm trying to remove from IIO.
Hence drop from this driver.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Andy Shevchenko 
Cc: Gregor Boirie 
Link: https://lore.kernel.org/r/20200910173242.621168-21-jic23@kernel.org

iio:pressure:ms5637: Drop of_match_ptr protection

2020-09-21T17:41:30+00:00

This prevents use of this driver with ACPI via PRP0001 and is
an example of an anti pattern I'm trying to remove from IIO.
Hence drop from this driver.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Andy Shevchenko 
Cc: Ludovic Tancerel 
Link: https://lore.kernel.org/r/20200910173242.621168-20-jic23@kernel.org

iio:pressure:ms5611: Drop of_match_ptr and CONFIG_OF protections

2020-09-21T17:41:29+00:00

These prevents use of this driver with ACPI via PRP0001 and are
an example of an anti pattern I'm trying to remove from IIO.
Hence drop them from this driver.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Andy Shevchenko 
Cc: Tomasz Duszynski 
Link: https://lore.kernel.org/r/20200910173242.621168-19-jic23@kernel.org

iio:pressure:icp10100: Drop of_match_ptr and CONFIG_OF protections

2020-09-21T17:41:29+00:00

These prevents use of this driver with ACPI via PRP0001 and are
an example of an anti pattern I'm trying to remove from IIO.
Hence drop them from this driver.

Signed-off-by: Jonathan Cameron 
Reviewed-by: Andy Shevchenko 
Cc: Jean-Baptiste Maneyrol 
Link: https://lore.kernel.org/r/20200910173242.621168-18-jic23@kernel.org

Merge 5.8-rc6 into staging-next

2020-07-20T07:37:31+00:00

We need the staging fixes in here, and it resolves a merge issue with an
iio driver.

Signed-off-by: Greg Kroah-Hartman

Replace HTTP links with HTTPS ones: drivers/iio

2020-07-13T14:05:59+00:00

Rationale:
Reduces attack surface on kernel devs opening the links for MITM
as HTTPS traffic is much harder to manipulate.

Deterministic algorithm:
For each file:
  If not .svg:
    For each line:
      If doesn't contain `\bxmlns\b`:
        For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:
          If both the HTTP and HTTPS versions
          return 200 OK and serve the same content:
            Replace HTTP with HTTPS.

Signed-off-by: Alexander A. Klimov 
Signed-off-by: Jonathan Cameron

iio:pressure:ms5611 Fix buffer element alignment

2020-07-05T13:22:48+00:00

One of a class of bugs pointed out by Lars in a recent review.
iio_push_to_buffers_with_timestamp assumes the buffer used is aligned
to the size of the timestamp (8 bytes).  This is not guaranteed in
this driver which uses an array of smaller elements on the stack.
Here there is no data leak possibility so use an explicit structure
on the stack to ensure alignment and nice readable fashion.

The forced alignment of ts isn't strictly necessary in this driver
as the padding will be correct anyway (there isn't any).  However
it is probably less fragile to have it there and it acts as
documentation of the requirement.

Fixes: 713bbb4efb9dc ("iio: pressure: ms5611: Add triggered buffer support")
Reported-by: Lars-Peter Clausen 
Signed-off-by: Jonathan Cameron 
Acked-by: Tomasz Duszynski 
Cc: