Unverified Commit f3f67287 authored by Kurt Borja's avatar Kurt Borja Committed by Ilpo Järvinen
Browse files

platform/x86: alienware-wmi: Modify parse_rgb() signature 



parse_rgb() now takes struct color_platform instead of struct
platform_zone to support upcoming refactor.

Reviewed-by: default avatarArmin Wolf <W_Armin@gmx.de>
Signed-off-by: default avatarKurt Borja <kuurtb@gmail.com>
Reviewed-by: default avatarMario Limonciello <mario.limonciello@amd.com>
Link: https://lore.kernel.org/r/20250105153019.19206-5-kuurtb@gmail.com


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
parent 99675d25

drivers/platform/x86/dell/alienware-wmi.c

+3 −3
Original line number Diff line number Diff line
@@ -434,7 +434,7 @@ static u8 global_brightness; 
/*

 * Helpers used for zone control

 */

static int parse_rgb(const char *buf, struct platform_zone *zone)

static int parse_rgb(const char *buf, struct color_platform *colors)

{

	long unsigned int rgb;

	int ret;
@@ -454,7 +454,7 @@ static int parse_rgb(const char *buf, struct platform_zone *zone) 
	repackager.package = rgb & 0x0f0f0f0f;

	pr_debug("alienware-wmi: r: %d g:%d b: %d\n",

		 repackager.cp.red, repackager.cp.green, repackager.cp.blue);

	zone->colors = repackager.cp;

	*colors = repackager.cp;

	return 0;

}
@@ -538,7 +538,7 @@ static ssize_t zone_set(struct device *dev, struct device_attribute *attr, 
		pr_err("alienware-wmi: invalid target zone\n");

		return 1;

	}

	ret = parse_rgb(buf, target_zone);

	ret = parse_rgb(buf, &target_zone->colors);

	if (ret)

		return ret;

	ret = alienware_update_led(target_zone);