From: "Alexey Klimov" <alexey.klimov@linaro.org>
To: "Mateusz Majewski" <m.majewski2@samsung.com>
Cc: <linux-samsung-soc@vger.kernel.org>, <linux-pm@vger.kernel.org>,
<devicetree@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>,
"Bartlomiej Zolnierkiewicz" <bzolnier@gmail.com>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
"Daniel Lezcano" <daniel.lezcano@linaro.org>,
"Zhang Rui" <rui.zhang@intel.com>,
"Lukasz Luba" <lukasz.luba@arm.com>,
"Rob Herring" <robh@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Alim Akhtar" <alim.akhtar@samsung.com>,
"Sam Protsenko" <semen.protsenko@linaro.org>,
"Anand Moon" <linux.amoon@gmail.com>,
"Marek Szyprowski" <m.szyprowski@samsung.com>
Subject: Re: [PATCH v4 0/7] Add initial Exynos850 support to the thermal driver
Date: Tue, 10 Mar 2026 02:56:23 +0000 [thread overview]
Message-ID: <DGYRJ0FUBJIB.3HUS30VW280A4@linaro.org> (raw)
In-Reply-To: <20260303090211.673143-1-m.majewski2@samsung.com>
Hello Mateusz,
Sorry for delayed response too.
On Tue Mar 3, 2026 at 9:02 AM GMT, Mateusz Majewski wrote:
> Hello and sorry for missing your mail, thankfully Marek pointed it out
> to me.
>
>> I applied the whole series locally, it applies fine on 6.19 but I didn't
>> check linux-next; and tested it on Exynos850 E850-96 board with the
>> following DT node:
>>
>> tmuctrl_0: tmu@10070000 {
>> compatible = "samsung,exynos850-tmu";
>> reg = <0x10070000 0x800>;
>> interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>;
>> clocks = <&cmu_peri CLK_GOUT_BUSIF_TMU_PCLK>;
>> clock-names = "tmu_apbif";
>> #thermal-sensor-cells = <0>;
>> };
>>
>> and thermal zones as you mentioned way back. It works just fine.
>> Temp goes up to 48 C with a loaded CPUs and settles at 36-37C when idle.
>>
>> So for the whole series:
>> Tested-by: Alexey Klimov <alexey.klimov@linaro.org>
>
> Thank you!
>
>> However, do you have plans to update it or re-submit?
>> Or any other plans working on it?
>
> Honestly not sure. If any other patches are merged that conflict with
> this series, I definitely can resolve the commits (for now it applies
> just fine to next-20260227, though I haven't compiled the result yet).
> Correct me if I am wrong, but I don't recall anything to be done in this
> series otherwise. If there is interest in this series, I can retest this
> on the other boards and re-submit this later this week.
So, I bumped into the same issue you described in your other email.
Other sensors seems to be not initialised and always return some weird
temp (minus 29 or smth like that). Only first sensors works.
I initially started to look at this after enabling idle states on E850-96
(they work with only some un-upstreamable change). Having at least one
sensor is better than nothing.
I am looking at enabling ACPM thermal thingy but so far I see similar
issue that temp of all sensors is reported as 10 C regardless via ACPM.
Having said that I am not going to oppose this series moving forward if
you wish.
Thank you.
Best regards,
Alexey
next prev parent reply other threads:[~2026-03-10 2:56 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20240911121149eucas1p29b9ccf99a545cfaa924b122cd8dd3183@eucas1p2.samsung.com>
2024-09-11 12:11 ` Mateusz Majewski
[not found] ` <CGME20240911121152eucas1p113445ce1ce6b6bd9c8f96322604bf517@eucas1p1.samsung.com>
2024-09-11 12:11 ` [PATCH v4 1/7] drivers/thermal/exynos: use DEFINE_SIMPLE_DEV_PM_OPS Mateusz Majewski
[not found] ` <CGME20240911121153eucas1p20f864c51746e6a7de55ecd0e52efe5c4@eucas1p2.samsung.com>
2024-09-11 12:11 ` [PATCH v4 2/7] drivers/thermal/exynos: use pm_sleep_ptr instead of conditional compilation Mateusz Majewski
[not found] ` <CGME20240911121154eucas1p1a429a565c446cdd968f565df1ffae42c@eucas1p1.samsung.com>
2024-09-11 12:11 ` [PATCH v4 3/7] drivers/thermal/exynos: improve sanitize_temp_error Mateusz Majewski
[not found] ` <CGME20240911121155eucas1p160223d89bff83a89a03c5447252f44a9@eucas1p1.samsung.com>
2024-09-11 12:11 ` [PATCH v4 4/7] drivers/thermal/exynos: reuse data->temp_mask in exynos_tmu_update_temp Mateusz Majewski
[not found] ` <CGME20240911121157eucas1p2a8ff398f0f30f141de974763680bb041@eucas1p2.samsung.com>
2024-09-11 12:11 ` [PATCH v4 5/7] dt-bindings: thermal: samsung,exynos: add exynos850-tmu string Mateusz Majewski
[not found] ` <CGME20240911121158eucas1p2ab3ec5b5b59351af22c5740e02236b16@eucas1p2.samsung.com>
2024-09-11 12:11 ` [PATCH v4 6/7] drivers/thermal/exynos: add initial Exynos850 support Mateusz Majewski
[not found] ` <CGME20240911121159eucas1p1ecfa41871f21953b49d90c287d121409@eucas1p1.samsung.com>
2024-09-11 12:11 ` [PATCH v4 7/7] dt-bindings: thermal: samsung,exynos: remove driver-specific information Mateusz Majewski
2026-02-18 18:29 ` [PATCH v4 0/7] Add initial Exynos850 support to the thermal driver Alexey Klimov
2026-03-03 9:02 ` Mateusz Majewski
2026-03-06 12:35 ` Mateusz Majewski
2026-03-10 2:56 ` Alexey Klimov [this message]
[not found] ` <CGME20260312160809eucas1p1fc79e4d333f3995a48b68ff07ec7726c@eucas1p1.samsung.com>
2026-03-12 16:07 ` Mateusz Majewski
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=DGYRJ0FUBJIB.3HUS30VW280A4@linaro.org \
--to=alexey.klimov@linaro.org \
--cc=alim.akhtar@samsung.com \
--cc=bzolnier@gmail.com \
--cc=conor+dt@kernel.org \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux.amoon@gmail.com \
--cc=lukasz.luba@arm.com \
--cc=m.majewski2@samsung.com \
--cc=m.szyprowski@samsung.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=rui.zhang@intel.com \
--cc=semen.protsenko@linaro.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
all inboxes | Powered by JetHome®