Messages in this thread | | | From | Linus Walleij <> | Date | Thu, 22 Apr 2021 02:38:52 +0200 | Subject | Re: [PATCH v1] iio: gyro: mpu3050: Fix reported temperature value |
| |
On Thu, Apr 22, 2021 at 1:49 AM Dmitry Osipenko <digetx@gmail.com> wrote:
> The raw temperature value is a signed 16bit integer. The sign casting > is missed in the code, which results in a wrong temperature reported by > userspace tools, fix it. > > Cc: stable@vger.kernel.org > Link: https://www.cdiweb.com/datasheets/invensense/mpu-3000a.pdf > Tested-by: Maxim Schwalm <maxim.schwalm@gmail.com> # Asus TF700T > Tested-by: Svyatoslav Ryhel <clamor95@gmail.com> # Asus TF201 > Reported-by: Svyatoslav Ryhel <clamor95@gmail.com> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
+/- Andy's comments: Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
I never thought this driver would have so many users (3 people signed testing it!) but I realize it is more widely deployed than I thought.
I have totally ignored the MPU3050's ability to act as a "sensor hub" and talk to accelerometers and magnetometers directly. I always thought it would be better to just route the I2C right through it and put Linux in direct control, but I realize this was not Invensese's intention. I don't know if it can be actually utilized in some generic way, all kernels using that have separate hacky drivers for all the sub-sensors duplicating the kernel drivers we already have ...
Yours, Linus Walleij
| |