Received-SPF: pass (google.com: domain of linux-kernel+bounces-46939-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <8552a795-9ce4-417a-bc71-593571a6b363@gmail.com>
Date: Wed, 31 Jan 2024 18:58:39 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v5 2/2] platform/surface: aggregator_registry: add entry
 for fan speed
To: Guenter Roeck <linux@roeck-us.net>, Ivor Wanders <ivor@iwanders.net>,
 =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>,
 Hans de Goede <hdegoede@redhat.com>
Cc: Jean Delvare <jdelvare@suse.com>, Jonathan Corbet <corbet@lwn.net>,
 Mark Gross <markgross@kernel.org>, linux-hwmon@vger.kernel.org,
 linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
 platform-driver-x86@vger.kernel.org
References: <20240131005856.10180-1-ivor@iwanders.net>
 <20240131005856.10180-3-ivor@iwanders.net>
 <7e392c1e-2cb2-43e4-804e-227551ed2dd7@roeck-us.net>
From: Maximilian Luz <luzmaximilian@gmail.com>
In-Reply-To: <7e392c1e-2cb2-43e4-804e-227551ed2dd7@roeck-us.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

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@iwanders.net>
>> Link: https://github.com/linux-surface/kernel/pull/144
>> Reviewed-by: Maximilian Luz <luzmaximilian@gmail.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.

Best regards,
Max