linux-kernel - Re: [PATCH v5 2/2] platform/surface: aggregator

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]

Date: Tue, 6 Feb 2024 12:17:21 +0200 (EET)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Maximilian Luz <luzmaximilian@...il.com>, Ivor Wanders <ivor@...nders.net>
cc: Guenter Roeck <linux@...ck-us.net>, Hans de Goede <hdegoede@...hat.com>, 
    Jean Delvare <jdelvare@...e.com>, Jonathan Corbet <corbet@....net>, 
    Mark Gross <markgross@...nel.org>, linux-hwmon@...r.kernel.org, 
    linux-doc@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>, 
    platform-driver-x86@...r.kernel.org
Subject: Re: [PATCH v5 2/2] platform/surface: aggregator_registry: add entry
 for fan speed

On Thu, 1 Feb 2024, Ilpo Järvinen wrote:

> On Wed, 31 Jan 2024, Maximilian Luz wrote:
> 
> > Am 1/31/2024 um 2:24 PM schrieb Guenter Roeck:
> > > On Tue, Jan 30, 2024 at 07:58:56PM -0500, Ivor Wanders wrote:
> > > > Add an entry for the fan speed function.
> > > > Add this new entry to the Surface Pro 9 group.
> > > > 
> > > > Signed-off-by: Ivor Wanders <ivor@...nders.net>
> > > > Link: https://github.com/linux-surface/kernel/pull/144
> > > > Reviewed-by: Maximilian Luz <luzmaximilian@...il.com>
> > > 
> > > I wasn't sure if the Reviewed-by: tag means that I should apply the patch
> > > through the hwmon subsystem. If so, please let me know. For now I'll
> > > assume that it will be applied through a platform tree.
> > 
> > I think it would make more sense for it to go through pdx86 (as usual
> > for platform/surface). That would avoid any potential merge conflicts
> > if we get more changes to the surface_aggregator_registry later on.
> > 
> > Hans, Ilpo, could you please take this?
> > 
> > Also I just noticed that Ilpo wasn't CCd, I assume because of an older
> > MAINTAINERS list. Ivor, please add him for any next submissions to
> > platform/surface.
> 
> Okay, thanks for letting me know (I assumed the opposite). I'll take it 
> through pdx86.

Patch 2/2 applied to review-ilpo.

-- 
 i.