linux-kernel - Re: [PATCH v7 RESEND 2/3] thermal: exynos_tmu: Support new hardware and update TMU interface

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>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <1704827d-f273-4360-bcd5-c18818a85034@linaro.org>
Date: Thu, 27 Nov 2025 11:04:51 +0200
From: Tudor Ambarus <tudor.ambarus@...aro.org>
To: 손신 <shin.son@...sung.com>,
 'Bartlomiej Zolnierkiewicz' <bzolnier@...il.com>,
 'Krzysztof Kozlowski' <krzk@...nel.org>,
 "'Rafael J . Wysocki'" <rafael@...nel.org>,
 'Daniel Lezcano' <daniel.lezcano@...aro.org>,
 'Zhang Rui' <rui.zhang@...el.com>, 'Lukasz Luba' <lukasz.luba@....com>,
 'Rob Herring' <robh@...nel.org>, 'Conor Dooley' <conor+dt@...nel.org>,
 'Alim Akhtar' <alim.akhtar@...sung.com>, youngmin.nam@...sung.com
Cc: 'Henrik Grimler' <henrik@...mler.se>, linux-pm@...r.kernel.org,
 linux-samsung-soc@...r.kernel.org, devicetree@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
 'Peter Griffin' <peter.griffin@...aro.org>,
 'André Draszik' <andre.draszik@...aro.org>,
 'William McVicker' <willmcvicker@...gle.com>, jyescas@...gle.com
Subject: Re: [PATCH v7 RESEND 2/3] thermal: exynos_tmu: Support new hardware
 and update TMU interface



On 11/27/25 5:07 AM, 손신 wrote:
> Hello, Tudor Ambarus

Hi!

> 
>> -----Original Message-----
>> From: Tudor Ambarus [mailto:tudor.ambarus@...aro.org]
>> Sent: Wednesday, November 26, 2025 6:22 PM
>> To: 손신 <shin.son@...sung.com>; 'Bartlomiej Zolnierkiewicz'
>> <bzolnier@...il.com>; 'Krzysztof Kozlowski' <krzk@...nel.org>; 'Rafael J .
>> Wysocki' <rafael@...nel.org>; 'Daniel Lezcano' <daniel.lezcano@...aro.org>;
>> 'Zhang Rui' <rui.zhang@...el.com>; 'Lukasz Luba' <lukasz.luba@....com>;
>> 'Rob Herring' <robh@...nel.org>; 'Conor Dooley' <conor+dt@...nel.org>;
>> 'Alim Akhtar' <alim.akhtar@...sung.com>; youngmin.nam@...sung.com
>> Cc: 'Henrik Grimler' <henrik@...mler.se>; linux-pm@...r.kernel.org; linux-
>> samsung-soc@...r.kernel.org; devicetree@...r.kernel.org; linux-arm-
>> kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; 'Peter Griffin'
>> <peter.griffin@...aro.org>; 'André Draszik' <andre.draszik@...aro.org>;
>> 'William McVicker' <willmcvicker@...gle.com>; jyescas@...gle.com
>> Subject: Re: [PATCH v7 RESEND 2/3] thermal: exynos_tmu: Support new
>> hardware and update TMU interface
>>
>> Hi, Shin Son,
>>
>> On 11/26/25 9:19 AM, 손신 wrote:
>>>> Looking at the exynosautov9 registers that you described and
>>>> comparing them with
>>>> gs101 I see just 2 differences:
>>>> 1/ exnosautov2 has a TRIMINFO_CONFIG2 register, while gs101 doesn't
>>>> 2/ EXYNOSAUTOV920_PEND register fields differ from GS101
>>> TRIMINFO_CONFIG2 doesn't exist on eav920 either; I simply misnamed it.
>>> However, the PEND register indeed differs from GS101.
>>>
>>>> Given the similarities, and considering the EXYNOS9_ registers rename
>> from:
>>>> https://lore.kernel.org/linux-samsung-soc/20251117074140.4090939-5-
>>>> youngmin.nam@...sung.com/
>>>> would it make sense to use the SoC-era name instead of specific SoC,
>> i.e.
>>>> s/EXYNOSAUTOV920_/EXYNOS9_ and use the latter for both exynosautov9
>>>> and gs101?
>>>>
>>> First of all, as far as I know, EXYNOS9 is not the same as exynosautov9,
>> and exynosautov920 also differs from exynosautov9.
>>
>> See also see this patch, or maybe the entire patch set:
>> https://lore.kernel.org/linux-samsung-soc/20251117074140.4090939-2-
>> youngmin.nam@...sung.com/
>>
>> It's not just autov9 and gs101 that have similar TMU registers (with the
>> two exceptions AFAICT), it's also exynos850 that seems identical with
>> autov9.
> 
> Yes, Do you have any plans to upstream the GS101 TMU code? From what I understand,

Yes, I'm currently working on upstreaming the GS101 TMU code. My plan is to
do the acpm tmu helpers and then integrate the gs101 TMU support with what
will be the new exynos TMU driver.

> Autov9 and exynos850 are unlikely to be upstreamed in their current form.

>From what I understand from your email exchanges with Daniel, you're going to
propose a new driver. Is my understanding correct? Do you have a timeline for it?
I'll then follow with the gs101 support.

> 
>> All seem to be part of the same "Exynos9-era" SoCs. Let's think about how
>> gs101/exynos850 TMU addition will follow. Shall one use the EXYNOSAUTOV920
>> registers? That seems misleading. Shall one redefine the entire register
>> set?
>> That won't fly because of the code duplication.
> 
> I kind of admit that.
> 
>> Thus I propose to use the EXYNOS9 prefix for the register definitions, and
>> if there are SoCs with slight differences, that can be handled with
>> compatible match data and specific SoC definitions, but only where things
>> differ.
> 
> However, I am not sure whether Exynos2200, 7885, 990, 9810, 8890, 8895, or FSD share the same TMU hardware layout as exynosautov920.
> So I’m wondering whether the EXYNOS9 prefix should be limited to GS101 and eav920, or if we should consider a different prefix that better reflects the grouping.

exynos850 has the same reg layout as eav920 and gs101 too. If Exynos9-era is
a common terminology used inside Samsung then we should be good to go with
Exynos9 prefix I think. Especially since we have a predecessor, the renaming
tried in pinctrl. But if you're not sure about it, use just EXYNOS_ then.

Cheers,
ta