linux-kernel - Re: [PATCH v1] iio: gyro: mpu3050: Fix reported temperature value

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <18507b2d-0133-190d-724a-4cf038016c17@gmail.com>
Date:   Thu, 22 Apr 2021 20:42:48 +0300
From:   Dmitry Osipenko <digetx@...il.com>
To:     Andy Shevchenko <andy.shevchenko@...il.com>
Cc:     Linus Walleij <linus.walleij@...aro.org>,
        Jonathan Cameron <jic23@...nel.org>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Maxim Schwalm <maxim.schwalm@...il.com>,
        Svyatoslav Ryhel <clamor95@...il.com>,
        "linux-iio@...r.kernel.org" <linux-iio@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v1] iio: gyro: mpu3050: Fix reported temperature value

22.04.2021 09:46, Andy Shevchenko пишет:
>                     case IIO_TEMP:
>     -                       /* The temperature scaling is (x+23000)/280
>     Celsius */
>     +                       /*
>     +                        * The temperature scaling is (x+23000)/280
>     Celsius,
>     +                        * where 23000 includes room temperature
>     offset of
>     +                        * +35C, 280 is the precision scale and x is
>     the 16-bit
>     +                        * signed integer which corresponds to the
>     temperature
>     +                        * range of -40C..85C.
> 
> 
> 
> Datasheet says typical -30°... also think about the other comment you
> gave, I.e. about temperature of the die itself. Are you suggesting that
> at -40° the die T is also -40°?  Does it have perpetuum mobile ? ;) it
> might dissipate not to much energy, but we don’t know the linearity of
> the curve there.

My understanding that -40C is the minimum temperature which sensor may
report.

Alright, I think the comment could be improved a tad anyways. I'll
prepare v3, thanks.