[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240529174843.13226-1-lsanche@lyndeno.ca>
Date: Wed, 29 May 2024 11:47:43 -0600
From: Lyndon Sanche <lsanche@...deno.ca>
To: lsanche@...deno.ca
Cc: mario.limonciello@....com,
pali@...nel.org,
W_Armin@....de,
srinivas.pandruvada@...ux.intel.com,
ilpo.jarvinen@...ux.intel.com,
lkp@...el.com,
hdegoede@...hat.com,
Yijun.Shen@...l.com,
Matthew Garrett <mjg59@...f.ucam.org>,
Vegard Nossum <vegard.nossum@...cle.com>,
Randy Dunlap <rdunlap@...radead.org>,
Heiner Kallweit <hkallweit1@...il.com>,
AceLan Kao <acelan.kao@...onical.com>,
linux-kernel@...r.kernel.org,
platform-driver-x86@...r.kernel.org,
Dell.Client.Kernel@...l.com
Subject: [PATCH v8 0/3] platform/x86: dell: Implement platform_profile
v8:
- Rebase all patches against master
- dell-pc.c:
- Separate <> and "" includes
- Align thermal_mode_bits with tabs
- Flatten nested ifs
- Directly return dell_send_request
- Correct some grammar
v7:
- Move platform_profile into new dell-pc module
- Add myself as maintainer of dell-pc
- Move smbios call and fill functions to dell-smbios-base
- Check for classes above 30 and return as not supported
- Rename dell_laptop_check_supported_cmds to
dell_smbios_class_is_supported
- Check for ENXIO and treat as no thermal modes supported
v6:
- Add ACPI dependency for dell-laptop
- Add and use helper symbol for checking supported commands
v5:
- Fix indent in smbios-thermal-ctl comment
- Remove linux/wmi.h include
- Add 'select ACPI_PLATFORM_PROFILE' to Dell KConfig
v4:
- Make thermal_init and thermal_cleanup static
- Rearrange order of added includes, did not edit current includes
- Include bits.h
- Switch comment style
- Return error if platform_profile registering failed
- Add thermal calls to call_blacklist
- Align defines with tabs
- Correct separation of function and error handling
- Propagate error codes up
v3:
- Convert smbios-thermal-ctl docs to multiline comment and wrap
- Change thermal_mode_bits enum to directly be BIT() values
- Convert related code to use this
- Use FIELD_GET/PREP and GENNMASK for getting/setting thermal modes
- Correct offset for getting current ACC mode, setting offset
unchanged
- Check if thermal_handler is allocated before freeing and
unregistering platform_profile
v2:
- Wrap smbios-thermal-ctl comment
- Return proper error code when invalid state returned
- Simplify platform_profile_get returns
- Propagate ENOMEM error
Lyndon Sanche (3):
platform/x86: dell-smbios: Add helper for checking supported class
platform/x86: dell-smbios: Move request functions for reuse
platform/x86: dell-pc: Implement platform_profile
MAINTAINERS | 6 +
drivers/platform/x86/dell/Kconfig | 13 +
drivers/platform/x86/dell/Makefile | 1 +
drivers/platform/x86/dell/dell-laptop.c | 23 --
drivers/platform/x86/dell/dell-pc.c | 307 +++++++++++++++++++
drivers/platform/x86/dell/dell-smbios-base.c | 35 +++
drivers/platform/x86/dell/dell-smbios.h | 7 +
7 files changed, 369 insertions(+), 23 deletions(-)
create mode 100644 drivers/platform/x86/dell/dell-pc.c
--
2.42.0
Powered by blists - more mailing lists