[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <5fe4c524-ced6-4666-b8fe-ab95c8d1cff1@gmail.com>
Date: Sun, 11 Aug 2024 01:23:27 +0200
From: Maximilian Luz <luzmaximilian@...il.com>
To: Guenter Roeck <linux@...ck-us.net>
Cc: Jean Delvare <jdelvare@...e.com>, Hans de Goede <hdegoede@...hat.com>,
Ivor Wanders <ivor@...nders.net>, linux-kernel@...r.kernel.org,
linux-hwmon@...r.kernel.org
Subject: Re: [PATCH v3] hwmon: Add thermal sensor driver for Surface
Aggregator Module
On 8/10/24 11:47 PM, Maximilian Luz wrote:
[...]
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index e4d9a035a57a..5ac9ce8fa0a9 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -2080,6 +2080,17 @@ config SENSORS_SURFACE_FAN
>
> Select M or Y here, if you want to be able to read the fan's speed.
>
> +config SENSORS_SURFACE_TEMP
> + tristate "Microsoft Surface Thermal Sensor Driver"
> + depends on SURFACE_AGGREGATOR
> + select SURFACE_AGGREGATOR_BUS
It seems that for some reason, this creates a recurrence in Kconfig. I should
have properly build-tested this before sending it in, sorry. I'll switch to
"depends on" for now as there seem to be more changes required to make
"select" work.
Best regards,
Max
Powered by blists - more mailing lists