Re: [External] Re: [PATCH 2/3] ACPI: platform-profile: Add platform profile support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rafael

On 10/11/2020 07:53, Rafael J. Wysocki wrote:
On Tue, Nov 10, 2020 at 4:32 AM Mark Pearson <markpearson@xxxxxxxxxx> wrote:

This is the initial implementation of the platform-profile feature.
It provides the details discussed and outlined in the
sysfs-platform_profile document.

Many modern systems have the ability to modify the operating profile to
control aspects like fan speed, temperature and power levels. This
module provides a common sysfs interface that platform modules can register
against to control their individual profile options.

Signed-off-by: Mark Pearson <markpearson@xxxxxxxxxx>
---
  MAINTAINERS                      |   8 ++
  drivers/acpi/Kconfig             |  19 ++++
  drivers/acpi/Makefile            |   1 +
  drivers/acpi/platform_profile.c  | 172 +++++++++++++++++++++++++++++++
  include/linux/platform_profile.h |  36 +++++++
  5 files changed, 236 insertions(+)
  create mode 100644 drivers/acpi/platform_profile.c
  create mode 100644 include/linux/platform_profile.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 9a54806ebf02..e731ac1c4447 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -436,6 +436,14 @@ S: Orphan
  F:     drivers/platform/x86/wmi.c
  F:     include/uapi/linux/wmi.h

+ACPI PLATFORM PROFILE DRIVER
+M:     Mark Pearson <markpearons@xxxxxxxxxx>
+L:     linux-acpi@xxxxxxxxxxxxxxx
+S:     Supported
+W:     https://01.org/linux-acpi
+B:     https://bugzilla.kernel.org
+F:     drivers/acpi/platform_profile.c

IMO it is premature to add a MAINTAINERS entry for this until it turns
out to be really necessary.

Not a problem - I'll remove this. I wasn't particularly sure about the etiquette here and I ran checkpatch and it suggested it.

I'm obviously happy to help with fixes etc. My plan is not to dump and run :)

Mark



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux