linux-kernel - Re: FW: Patch for supprting thermal of using drivetemp module

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: <d089cd3e-4115-a984-d1ce-599e0dd1160e@roeck-us.net>
Date:   Wed, 15 Mar 2023 07:12:57 -0700
From:   Guenter Roeck <linux@...ck-us.net>
To:     Phinex Hung <phinex@...ltek.com>,
        "jdelvare@...e.com" <jdelvare@...e.com>
Cc:     "linux-hwmon@...r.kernel.org" <linux-hwmon@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: FW: Patch for supprting thermal of using drivetemp module

On 3/15/23 03:49, Phinex Hung wrote:
> Hi Jean,
> 
> I have a patch to support thermal zone so that we can just rely on dts to describe a thermal zone and do the cooling operations.
> 
> To do this, I first modify drivetemp.c to support platform driver depending on THERMAL_OF and then try to support multiple sensors using a single thermal zone.
> 

First of all, this is not how to submit patches into the Linux kernel.
Please consult the documentation.

Second, the driver registers its sensors with the hwmon core using
HWMON_C_REGISTER_TZ, which should already result in its registration
as thermal zone. If that does not work, please figure out why and
fix it instead of re-implementing thermal zone registration in
the driver.

Thanks,
Guenter