Re: [PATCH] platform/x86: dell-laptop: Implement platform_profile

From: Lyndon Sanche <lsanche@lyndeno.ca>
To: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: "Mario Limonciello" <mario.limonciello@amd.com>,
	"Matthew Garrett" <mjg59@srcf.ucam.org>,
	"Pali Rohár" <pali@kernel.org>,
	"Hans de Goede" <hdegoede@redhat.com>,
	"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
	platform-driver-x86@vger.kernel.org,
	linux-kernel@vger.kernel.org, Dell.Client.Kernel@dell.com
Subject: Re: [PATCH] platform/x86: dell-laptop: Implement platform_profile
Date: Thu, 25 Apr 2024 18:38:35 -0600	[thread overview]
Message-ID: <BSXICS.THI25R7A1N202@lyndeno.ca> (raw)
In-Reply-To: <3cf80056-e62d-4be1-9716-4c843ae2f88c@linux.intel.com>

On Thu, Apr 25 2024 at 02:51:42 PM -07:00:00, Srinivas Pandruvada 
<srinivas.pandruvada@linux.intel.com> wrote:
> 
> On 4/25/24 13:28, Mario Limonciello wrote:
>> Yeah it's not say it's a "new" conflict, it would just become a lot 
>> \x7fmore prevalent since software like GNOME and KDE use 
>> \x7fpower-profiles-daemon to manipulate the new power profile you're 
>> \x7fexporting from the driver.
>> 
>> If there really is no conflict, then great!
>> If there is a conflict then I was just wondering if there needs to 
>> be \x7fan easy way to turn on/off the profile support when thermald is 
>> in use.
> 
> This shouldn't be in conflict as this should be directly changing 
> some settings in BIOS. BIOS should send some notification, if it 
> wants some changes in thermal tables used by thermald.
> 
> 
> Thanks,
> 
> Srinivas
> 

If we do not think there is a conflict I can leave it without a toggle.

Lyndon