From: Agathe Porte <agathe.porte@nokia.com>
To: Guenter Roeck <linux@roeck-us.net>, linux-hwmon@vger.kernel.org
Cc: Jean Delvare <jdelvare@suse.com>,
Rob Herring <robh+dt@kernel.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Krzysztof Adamski <krzysztof.adamski@nokia.com>
Subject: Re: [PATCH v7 2/2] hwmon: Add driver for Texas Instruments TMP464 and TMP468
Date: Thu, 3 Mar 2022 16:31:39 +0100 [thread overview]
Message-ID: <c3a9b340-34b1-4bed-4810-4c5f39a8554b@nokia.com> (raw)
In-Reply-To: <abde8dce-810b-fd39-b0aa-4f1b2ec8f8d7@roeck-us.net>
Le 3/3/2022 à 4:00 PM, Guenter Roeck a écrit :
> On 3/3/22 00:57, Agathe Porte wrote:
>> Hi Guenter,
>>
>> Le 02/03/2022 à 18:59, Guenter Roeck a écrit :
>>> Any review / test feedback on this patch ? I would like to apply it
>>> before the commit window opens, but the time is getting short.
>>
>> I thought that you did receive the TMP464 samples and had the
>> opportunity to test on it. I will test v7 on our hardware equipped
>> with TMP464, verify that DT support works fine, and will reply to
>> this email with my findings.
>>
>
> Yes, I did, and thanks a lot for it! I even wrote a qemu emulation
> for the chip to be able to test the devicetree code.
Great!
> Still, I need to have someone else confirm that I didn't mess up.
I tested v7 on our hardware and the behavior seems to be the same as our
previous, in-house driver, if that gives you a point of comparison. We
only use temp*_input sysfs though.
No compilation warnings.
I can disable and enable sensors fine at runtime:
> # cat temp*_input
> 43063
> 35813
> 34938
> 39313
> 29125
> # echo 0 | tee temp*_enable
> 0
> # cat temp*_input
> cat: temp1_input: No data available
> cat: temp2_input: No data available
> cat: temp3_input: No data available
> cat: temp4_input: No data available
> cat: temp5_input: No data available
> # echo 1 | tee temp*_enable
> 1
> # cat temp*_input
> 43063
> 35750
> 34875
> 39313
> 29188
For what it's worth:
Tested-by: Agathe Porte <agathe.porte@nokia.com>
Bests.
next prev parent reply other threads:[~2022-03-03 15:31 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-22 22:36 [PATCH v7 1/2] dt-bindings: hwmon: add tmp464.yaml Guenter Roeck
2022-02-22 22:36 ` [PATCH v7 2/2] hwmon: Add driver for Texas Instruments TMP464 and TMP468 Guenter Roeck
2022-03-02 17:59 ` Guenter Roeck
2022-03-03 8:57 ` Agathe Porte
2022-03-03 15:00 ` Guenter Roeck
2022-03-03 15:31 ` Agathe Porte [this message]
2022-03-03 15:41 ` Guenter Roeck
[not found] ` <51ea03f0-627b-2e9d-5972-2053fa12b9b5@nokia.com>
2022-03-15 1:22 ` Guenter Roeck
2022-03-15 13:03 ` Agathe Porte
2022-03-15 15:57 ` Guenter Roeck
2022-02-24 16:11 ` [PATCH v7 1/2] dt-bindings: hwmon: add tmp464.yaml Rob Herring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c3a9b340-34b1-4bed-4810-4c5f39a8554b@nokia.com \
--to=agathe.porte@nokia.com \
--cc=devicetree@vger.kernel.org \
--cc=jdelvare@suse.com \
--cc=krzysztof.adamski@nokia.com \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).