| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <a16d0785-d136-47a6-ae0b-07e5b7a6ec19@gmx.de>
Date: Thu, 2 Jan 2025 16:50:49 +0100
From: Armin Wolf <W_Armin@....de>
To: Kurt Borja <kuurtb@...il.com>, platform-driver-x86@...r.kernel.org
Cc: Dell.Client.Kernel@...l.com, hdegoede@...hat.com,
ilpo.jarvinen@...ux.intel.com, linux-kernel@...r.kernel.org,
mario.limonciello@....com
Subject: Re: [PATCH v2 06/20] alienware_wmi: General cleanup of WMAX methods
Am 29.12.24 um 20:44 schrieb Kurt Borja:
> Reorder variable declaration from longest to shortest. Standarize
> show/store method names of WMAX's sysfs groups.
Reviewed-by: Armin Wolf <W_Armin@....de>
>
> Signed-off-by: Kurt Borja <kuurtb@...il.com>
> ---
> drivers/platform/x86/dell/alienware-wmi.c | 67 +++++++++++------------
> 1 file changed, 32 insertions(+), 35 deletions(-)
>
> diff --git a/drivers/platform/x86/dell/alienware-wmi.c b/drivers/platform/x86/dell/alienware-wmi.c
> index d97e5e15a8f2..2c17160473a6 100644
> --- a/drivers/platform/x86/dell/alienware-wmi.c
> +++ b/drivers/platform/x86/dell/alienware-wmi.c
> @@ -722,14 +722,14 @@ static acpi_status alienware_wmax_command(void *in_args, size_t in_size,
> * The HDMI mux sysfs node indicates the status of the HDMI input mux.
> * It can toggle between standard system GPU output and HDMI input.
> */
> -static ssize_t show_hdmi_cable(struct device *dev,
> - struct device_attribute *attr, char *buf)
> +static ssize_t cable_show(struct device *dev, struct device_attribute *attr,
> + char *buf)
> {
> - acpi_status status;
> - u32 out_data;
> struct wmax_basic_args in_args = {
> .arg = 0,
> };
> + acpi_status status;
> + u32 out_data;
> status =
> alienware_wmax_command(&in_args, sizeof(in_args),
> WMAX_METHOD_HDMI_CABLE, &out_data);
> @@ -743,14 +743,14 @@ static ssize_t show_hdmi_cable(struct device *dev,
> return sysfs_emit(buf, "unconnected connected [unknown]\n");
> }
>
> -static ssize_t show_hdmi_source(struct device *dev,
> - struct device_attribute *attr, char *buf)
> +static ssize_t source_show(struct device *dev, struct device_attribute *attr,
> + char *buf)
> {
> - acpi_status status;
> - u32 out_data;
> struct wmax_basic_args in_args = {
> .arg = 0,
> };
> + acpi_status status;
> + u32 out_data;
> status =
> alienware_wmax_command(&in_args, sizeof(in_args),
> WMAX_METHOD_HDMI_STATUS, &out_data);
> @@ -765,12 +765,11 @@ static ssize_t show_hdmi_source(struct device *dev,
> return sysfs_emit(buf, "input gpu [unknown]\n");
> }
>
> -static ssize_t toggle_hdmi_source(struct device *dev,
> - struct device_attribute *attr,
> - const char *buf, size_t count)
> +static ssize_t source_store(struct device *dev, struct device_attribute *attr,
> + const char *buf, size_t count)
> {
> - acpi_status status;
> struct wmax_basic_args args;
> + acpi_status status;
> if (strcmp(buf, "gpu\n") == 0)
> args.arg = 1;
> else if (strcmp(buf, "input\n") == 0)
> @@ -788,9 +787,8 @@ static ssize_t toggle_hdmi_source(struct device *dev,
> return count;
> }
>
> -static DEVICE_ATTR(cable, S_IRUGO, show_hdmi_cable, NULL);
> -static DEVICE_ATTR(source, S_IRUGO | S_IWUSR, show_hdmi_source,
> - toggle_hdmi_source);
> +static DEVICE_ATTR_RO(cable);
> +static DEVICE_ATTR_RW(source);
>
> static bool hdmi_group_visible(struct kobject *kobj)
> {
> @@ -815,14 +813,14 @@ static const struct attribute_group hdmi_attribute_group = {
> * - Currently supports reading cable status
> * - Leaving expansion room to possibly support dock/undock events later
> */
> -static ssize_t show_amplifier_status(struct device *dev,
> - struct device_attribute *attr, char *buf)
> +static ssize_t status_show(struct device *dev, struct device_attribute *attr,
> + char *buf)
> {
> - acpi_status status;
> - u32 out_data;
> struct wmax_basic_args in_args = {
> .arg = 0,
> };
> + acpi_status status;
> + u32 out_data;
> status =
> alienware_wmax_command(&in_args, sizeof(in_args),
> WMAX_METHOD_AMPLIFIER_CABLE, &out_data);
> @@ -836,7 +834,7 @@ static ssize_t show_amplifier_status(struct device *dev,
> return sysfs_emit(buf, "unconnected connected [unknown]\n");
> }
>
> -static DEVICE_ATTR(status, S_IRUGO, show_amplifier_status, NULL);
> +static DEVICE_ATTR_RO(status);
>
> static bool amplifier_group_visible(struct kobject *kobj)
> {
> @@ -859,14 +857,14 @@ static const struct attribute_group amplifier_attribute_group = {
> * Deep Sleep Control support
> * - Modifies BIOS setting for deep sleep control allowing extra wakeup events
> */
> -static ssize_t show_deepsleep_status(struct device *dev,
> - struct device_attribute *attr, char *buf)
> +static ssize_t deepsleep_show(struct device *dev, struct device_attribute *attr,
> + char *buf)
> {
> - acpi_status status;
> - u32 out_data;
> struct wmax_basic_args in_args = {
> .arg = 0,
> };
> + acpi_status status;
> + u32 out_data;
> status = alienware_wmax_command(&in_args, sizeof(in_args),
> WMAX_METHOD_DEEP_SLEEP_STATUS, &out_data);
> if (ACPI_SUCCESS(status)) {
> @@ -881,12 +879,11 @@ static ssize_t show_deepsleep_status(struct device *dev,
> return sysfs_emit(buf, "disabled s5 s5_s4 [unknown]\n");
> }
>
> -static ssize_t toggle_deepsleep(struct device *dev,
> - struct device_attribute *attr,
> - const char *buf, size_t count)
> +static ssize_t deepsleep_store(struct device *dev, struct device_attribute *attr,
> + const char *buf, size_t count)
> {
> - acpi_status status;
> struct wmax_basic_args args;
> + acpi_status status;
>
> if (strcmp(buf, "disabled\n") == 0)
> args.arg = 0;
> @@ -905,7 +902,7 @@ static ssize_t toggle_deepsleep(struct device *dev,
> return count;
> }
>
> -static DEVICE_ATTR(deepsleep, S_IRUGO | S_IWUSR, show_deepsleep_status, toggle_deepsleep);
> +static DEVICE_ATTR_RW(deepsleep);
>
> static bool deepsleep_group_visible(struct kobject *kobj)
> {
> @@ -953,13 +950,13 @@ static bool is_wmax_thermal_code(u32 code)
>
> static int wmax_thermal_information(u8 operation, u8 arg, u32 *out_data)
> {
> - acpi_status status;
> struct wmax_u32_args in_args = {
> .operation = operation,
> .arg1 = arg,
> .arg2 = 0,
> .arg3 = 0,
> };
> + acpi_status status;
>
> status = alienware_wmax_command(&in_args, sizeof(in_args),
> WMAX_METHOD_THERMAL_INFORMATION,
> @@ -976,13 +973,13 @@ static int wmax_thermal_information(u8 operation, u8 arg, u32 *out_data)
>
> static int wmax_thermal_control(u8 profile)
> {
> - acpi_status status;
> struct wmax_u32_args in_args = {
> .operation = WMAX_OPERATION_ACTIVATE_PROFILE,
> .arg1 = profile,
> .arg2 = 0,
> .arg3 = 0,
> };
> + acpi_status status;
> u32 out_data;
>
> status = alienware_wmax_command(&in_args, sizeof(in_args),
> @@ -1000,13 +997,13 @@ static int wmax_thermal_control(u8 profile)
>
> static int wmax_game_shift_status(u8 operation, u32 *out_data)
> {
> - acpi_status status;
> struct wmax_u32_args in_args = {
> .operation = operation,
> .arg1 = 0,
> .arg2 = 0,
> .arg3 = 0,
> };
> + acpi_status status;
>
> status = alienware_wmax_command(&in_args, sizeof(in_args),
> WMAX_METHOD_GAME_SHIFT_STATUS,
> @@ -1075,11 +1072,11 @@ static int thermal_profile_set(struct platform_profile_handler *pprof,
>
> static int create_thermal_profile(struct platform_device *platform_device)
> {
> - u32 out_data;
> + enum platform_profile_option profile;
> + enum wmax_thermal_mode mode;
> u8 sys_desc[4];
> u32 first_mode;
> - enum wmax_thermal_mode mode;
> - enum platform_profile_option profile;
> + u32 out_data;
> int ret;
>
> ret = wmax_thermal_information(WMAX_OPERATION_SYS_DESCRIPTION,
Powered by blists - more mailing lists