From 197c25f359d6511a4e0ba803cbe12e6cba8fa747 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Mon, 25 Jul 2022 16:58:27 +0200
Subject: [PATCH 01/13] Add uw charging prio and charging profile hw io methods

---
 src/uniwill_keyboard.h | 59 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 59 insertions(+)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 24c0ea4..7050f6b 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -819,6 +819,65 @@ static int uw_lightbar_remove(struct platform_device *dev)
 	return 0;
 }
 
+/*
+ * charging_prio values
+ *     0 => charging priority
+ *     1 => performance priority
+ */
+static int uw_set_charging_priority(u8 charging_priority)
+{
+	u8 previous_data, next_data;
+	int result;
+
+	charging_priority = (charging_priority & 0x01) << 7;
+
+	result = uniwill_read_ec_ram(0x07cc, &previous_data);
+	if (result != 0)
+		return result;
+
+	next_data = (previous_data & ~(1 << 7)) | charging_priority;
+	result = uniwill_write_ec_ram(0x07cc, next_data);
+
+	return result;
+}
+
+static int uw_get_charging_priority(u8 *charging_priority)
+{
+	int result = uniwill_read_ec_ram(0x07cc, charging_priority);
+	*charging_priority = (*charging_priority >> 7) & 0x01;
+	return result;
+}
+
+/*
+ * charging_prio values
+ *     0 => high capacity
+ *     1 => balanced
+ *     2 => stationary
+ */
+static int uw_set_charging_profile(u8 charging_profile)
+{
+	u8 previous_data, next_data;
+	int result;
+
+	charging_profile = (charging_profile & 0x03) << 4;
+
+	result = uniwill_read_ec_ram(0x07a6, &previous_data);
+	if (result != 0)
+		return result;
+
+	next_data = (previous_data & ~(0x03 << 4)) | charging_profile;
+	result = uniwill_write_ec_ram(0x07a6, next_data);
+
+	return result;
+}
+
+static int uw_get_charging_profile(u8 *charging_profile)
+{
+	int result = uniwill_read_ec_ram(0x07a6, charging_profile);
+	*charging_profile = (*charging_profile >> 4) & 0x03;
+	return result;
+}
+
 static int uniwill_keyboard_probe(struct platform_device *dev)
 {
 	u32 i;

From f5916ddc34b719c25ebc35626fdb02544014400f Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Tue, 15 Nov 2022 18:53:30 +0100
Subject: [PATCH 02/13] Add interface definition for charging prio

---
 src/uniwill_keyboard.h | 97 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 97 insertions(+)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 7050f6b..e921a8f 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -878,6 +878,103 @@ static int uw_get_charging_profile(u8 *charging_profile)
 	return result;
 }
 
+struct char_to_u8_t {
+	char* descriptor;
+	u8 value;
+};
+
+/**
+ * Commonly used standard colors
+ */
+static struct char_to_u8_t charging_prio_options[] = {
+	{ .descriptor = "charge_battery", .value = 0x00 },
+	{ .descriptor = "performance",    .value = 0x01 }
+};
+
+static ssize_t uw_charging_prios_available_show(struct device *child,
+						struct device_attribute *attr,
+						char *buffer)
+{
+	int i;
+	for (i = 0; i < ARRAY_SIZE(charging_prio_options); ++i)
+		sprintf(buffer + strlen(buffer), " %s",
+			charging_prio_options[i].descriptor);
+	sprintf(buffer + strlen(buffer), "\n");
+	return strlen(buffer);
+}
+
+static ssize_t uw_charging_prio_show(struct device *child,
+				     struct device_attribute *attr, char *buffer)
+{
+	u8 charging_prio_value;
+	int i, result;
+
+	result = uw_get_charging_priority(&charging_prio_value);
+	if (result != 0)
+		return result;
+
+	for (i = 0; i < ARRAY_SIZE(charging_prio_options); ++i)
+		if (charging_prio_options[i].value == charging_prio_value) {
+			sprintf(buffer, "%s\n", charging_prio_options[i].descriptor);
+			return strlen(buffer);
+		}
+
+	pr_err("Read charging prio value not matched to a descriptor\n");
+
+	return -EIO;
+}
+
+static ssize_t uw_charging_prio_store(struct device *child,
+				      struct device_attribute *attr,
+				      const char *buffer, size_t size)
+{
+	u8 charging_prio_value;
+	int i, result;
+	char *buffer_copy;
+	char *charging_prio_descriptor;
+	buffer_copy = kmalloc(size + 1, GFP_KERNEL);
+	strcpy(buffer_copy, buffer);
+	charging_prio_descriptor = strstrip(buffer_copy);
+
+	for (i = 0; i < ARRAY_SIZE(charging_prio_options); ++i)
+		if (strcmp(charging_prio_options[i].descriptor, charging_prio_descriptor) == 0) {
+			charging_prio_value = charging_prio_options[i].value;
+			break;
+		}
+
+	kfree(buffer_copy);
+
+	if (i < ARRAY_SIZE(charging_prio_options)) {
+		// Option found try to set
+		result = uw_set_charging_priority(charging_prio_value);
+		if (result == 0)
+			return size;
+		else
+			return -EIO;
+	} else
+		// Invalid input, not matched to an option
+		return -EINVAL;
+}
+
+struct uw_charging_prio_attrs_t {
+	struct device_attribute charging_prios_available;
+	struct device_attribute charging_prio;
+} uw_charging_prio_attrs = {
+	.charging_prios_available = __ATTR(charging_prios_available, 0444, uw_charging_prios_available_show, NULL),
+	.charging_prio = __ATTR(charging_prio, 0644, uw_charging_prio_show, uw_charging_prio_store)
+};
+
+static struct attribute *uw_charging_prio_attrs_list[] = {
+	&uw_charging_prio_attrs.charging_prios_available.attr,
+	&uw_charging_prio_attrs.charging_prio.attr,
+	NULL
+};
+
+static struct attribute_group uw_charging_prio_attr_group = {
+	.name = "uw_charging_priority",
+	.attrs = uw_charging_prio_attrs_list
+};
+
 static int uniwill_keyboard_probe(struct platform_device *dev)
 {
 	u32 i;

From 0ba6d70c06c2a1b30ebacb0d466b0d5f048f5a3a Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Wed, 16 Nov 2022 11:30:40 +0100
Subject: [PATCH 03/13] Add charging prio capability check

- Add uw features property
- Add ec has charging prio func
- Move uw device id func def
---
 src/uniwill_interfaces.h |   1 +
 src/uniwill_keyboard.h   | 124 ++++++++++++++++++++++-----------------
 2 files changed, 71 insertions(+), 54 deletions(-)

diff --git a/src/uniwill_interfaces.h b/src/uniwill_interfaces.h
index ae95e39..c0a3ab0 100644
--- a/src/uniwill_interfaces.h
+++ b/src/uniwill_interfaces.h
@@ -68,6 +68,7 @@ struct uniwill_device_features_t {
 	bool uniwill_profile_v1_two_profs;
 	bool uniwill_profile_v1_three_profs;
 	bool uniwill_profile_v1_three_profs_leds_only;
+	bool uniwill_has_charging_prio;
 };
 
 u32 uniwill_add_interface(struct uniwill_interface_t *new_interface);
diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index e921a8f..9d987c3 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -208,60 +208,6 @@ u32 uniwill_get_active_interface_id(char **id_str)
 }
 EXPORT_SYMBOL(uniwill_get_active_interface_id);
 
-struct uniwill_device_features_t *uniwill_get_device_features(void)
-{
-	struct uniwill_device_features_t *uw_feats = &uniwill_device_features;
-	u32 status;
-
-	status = uniwill_read_ec_ram(0x0740, &uw_feats->model);
-	if (status != 0)
-		uw_feats->model = 0;
-
-	uw_feats->uniwill_profile_v1_two_profs = false
-		|| dmi_match(DMI_BOARD_NAME, "PF5PU1G")
-		|| dmi_match(DMI_BOARD_NAME, "PULSE1401")
-		|| dmi_match(DMI_BOARD_NAME, "PULSE1501")
-	;
-
-	uw_feats->uniwill_profile_v1_three_profs = false
-	// Devices with "classic" profile support
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501A1650TI")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501A2060")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501I1650TI")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501I2060")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701A1650TI")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701A2060")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701I1650TI")
-		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701I2060")
-		// Note: XMG Fusion removed for now, seem to have
-		// neither same power profile control nor TDP set
-		//|| dmi_match(DMI_BOARD_NAME, "LAPQC71A")
-		//|| dmi_match(DMI_BOARD_NAME, "LAPQC71B")
-		//|| dmi_match(DMI_PRODUCT_NAME, "A60 MUV")
-	;
-
-	uw_feats->uniwill_profile_v1_three_profs_leds_only = false
-	// Devices where profile mainly controls power profile LED status
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 18, 0)
-		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XA02")
-		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XI02")
-		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XA03")
-		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XI03")
-		|| dmi_match(DMI_PRODUCT_SKU, "STELLARIS1XI03")
-		|| dmi_match(DMI_PRODUCT_SKU, "STELLARIS1XA03")
-		|| dmi_match(DMI_PRODUCT_SKU, "STELLARIS1XI04")
-		|| dmi_match(DMI_PRODUCT_SKU, "STEPOL1XA04")
-#endif
-	;
-
-	uw_feats->uniwill_profile_v1 =
-		uw_feats->uniwill_profile_v1_two_profs ||
-		uw_feats->uniwill_profile_v1_three_profs;
-
-	return uw_feats;
-}
-EXPORT_SYMBOL(uniwill_get_device_features);
-
 static void key_event_work(struct work_struct *work)
 {
 	sparse_keymap_report_known_event(
@@ -848,6 +794,20 @@ static int uw_get_charging_priority(u8 *charging_priority)
 	return result;
 }
 
+static int uw_has_charging_priority(bool *status)
+{
+	u8 data;
+	int result;
+	result = uniwill_read_ec_ram(0x0742, &data);
+
+	if (data & (1 << 5))
+		*status = true;
+	else
+		*status = false;
+
+	return result;
+}
+
 /*
  * charging_prio values
  *     0 => high capacity
@@ -975,6 +935,62 @@ static struct attribute_group uw_charging_prio_attr_group = {
 	.attrs = uw_charging_prio_attrs_list
 };
 
+struct uniwill_device_features_t *uniwill_get_device_features(void)
+{
+	struct uniwill_device_features_t *uw_feats = &uniwill_device_features;
+	u32 status;
+
+	status = uniwill_read_ec_ram(0x0740, &uw_feats->model);
+	if (status != 0)
+		uw_feats->model = 0;
+
+	uw_feats->uniwill_profile_v1_two_profs = false
+		|| dmi_match(DMI_BOARD_NAME, "PF5PU1G")
+		|| dmi_match(DMI_BOARD_NAME, "PULSE1401")
+		|| dmi_match(DMI_BOARD_NAME, "PULSE1501")
+	;
+
+	uw_feats->uniwill_profile_v1_three_profs = false
+	// Devices with "classic" profile support
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501A1650TI")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501A2060")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501I1650TI")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1501I2060")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701A1650TI")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701A2060")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701I1650TI")
+		|| dmi_match(DMI_BOARD_NAME, "POLARIS1701I2060")
+		// Note: XMG Fusion removed for now, seem to have
+		// neither same power profile control nor TDP set
+		//|| dmi_match(DMI_BOARD_NAME, "LAPQC71A")
+		//|| dmi_match(DMI_BOARD_NAME, "LAPQC71B")
+		//|| dmi_match(DMI_PRODUCT_NAME, "A60 MUV")
+	;
+
+	uw_feats->uniwill_profile_v1_three_profs_leds_only = false
+	// Devices where profile mainly controls power profile LED status
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 18, 0)
+		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XA02")
+		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XI02")
+		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XA03")
+		|| dmi_match(DMI_PRODUCT_SKU, "POLARIS1XI03")
+		|| dmi_match(DMI_PRODUCT_SKU, "STELLARIS1XI03")
+		|| dmi_match(DMI_PRODUCT_SKU, "STELLARIS1XA03")
+		|| dmi_match(DMI_PRODUCT_SKU, "STELLARIS1XI04")
+		|| dmi_match(DMI_PRODUCT_SKU, "STEPOL1XA04")
+#endif
+	;
+
+	uw_feats->uniwill_profile_v1 =
+		uw_feats->uniwill_profile_v1_two_profs ||
+		uw_feats->uniwill_profile_v1_three_profs;
+
+	uw_has_charging_priority(&uw_feats->uniwill_has_charging_prio);
+
+	return uw_feats;
+}
+EXPORT_SYMBOL(uniwill_get_device_features);
+
 static int uniwill_keyboard_probe(struct platform_device *dev)
 {
 	u32 i;

From acc12d579f4a905ed9c657578bf1de531491cd35 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 18 Nov 2022 18:20:04 +0100
Subject: [PATCH 04/13] Add charging prio init

---
 src/uniwill_keyboard.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 9d987c3..95608df 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -765,6 +765,8 @@ static int uw_lightbar_remove(struct platform_device *dev)
 	return 0;
 }
 
+static bool uw_charging_prio_loaded = false;
+
 /*
  * charging_prio values
  *     0 => charging priority
@@ -1026,11 +1028,16 @@ static int uniwill_keyboard_probe(struct platform_device *dev)
 	status = uw_lightbar_init(dev);
 	uw_lightbar_loaded = (status >= 0);
 
+	if (uw_feats->uniwill_has_charging_prio)
+		uw_charging_prio_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_prio_attr_group) == 0;
+
 	return 0;
 }
 
 static int uniwill_keyboard_remove(struct platform_device *dev)
 {
+	if (uw_charging_prio_loaded)
+		sysfs_remove_group(&dev->dev.kobj, &uw_charging_prio_attr_group);
 
 	if (uniwill_kbd_bl_type_rgb_single_color) {
 		sysfs_remove_group(&dev->dev.kobj, &uw_kbd_bl_color_attr_group);

From dcf9ec030cb8fce9b9f228ac591f587c82ee15a8 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 18 Nov 2022 18:36:40 +0100
Subject: [PATCH 05/13] Fix list output whitespace

---
 src/uniwill_keyboard.h | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 95608df..e25c2ec 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -857,11 +857,17 @@ static ssize_t uw_charging_prios_available_show(struct device *child,
 						struct device_attribute *attr,
 						char *buffer)
 {
-	int i;
-	for (i = 0; i < ARRAY_SIZE(charging_prio_options); ++i)
-		sprintf(buffer + strlen(buffer), " %s",
+	int i, n;
+	n = ARRAY_SIZE(charging_prio_options);
+	for (i = 0; i < n; ++i) {
+		sprintf(buffer + strlen(buffer), "%s",
 			charging_prio_options[i].descriptor);
-	sprintf(buffer + strlen(buffer), "\n");
+		if (i < n - 1)
+			sprintf(buffer + strlen(buffer), " ");
+		else
+			sprintf(buffer + strlen(buffer), "\n");
+	}
+
 	return strlen(buffer);
 }
 

From 7e03e3dfd9924774da1bc70ce7bd38a3dac9c2be Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 18 Nov 2022 18:38:03 +0100
Subject: [PATCH 06/13] Remove old copy comment

---
 src/uniwill_keyboard.h | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index e25c2ec..ccb1eeb 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -845,9 +845,6 @@ struct char_to_u8_t {
 	u8 value;
 };
 
-/**
- * Commonly used standard colors
- */
 static struct char_to_u8_t charging_prio_options[] = {
 	{ .descriptor = "charge_battery", .value = 0x00 },
 	{ .descriptor = "performance",    .value = 0x01 }

From ae58c0ddcbbb19ff273c27d255730844cae8e8fd Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 18 Nov 2022 18:58:01 +0100
Subject: [PATCH 07/13] Add uw charging profiles interface structures

---
 src/uniwill_keyboard.h | 96 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 96 insertions(+)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index ccb1eeb..1ee8ee7 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -845,6 +845,102 @@ struct char_to_u8_t {
 	u8 value;
 };
 
+static struct char_to_u8_t charging_profile_options[] = {
+	{ .descriptor = "high_capacity", .value = 0x00 },
+	{ .descriptor = "balanced",	 .value = 0x01 },
+	{ .descriptor = "stationary",	 .value = 0x02 }
+};
+
+static ssize_t uw_charging_profiles_available_show(struct device *child,
+						   struct device_attribute *attr,
+						   char *buffer)
+{
+	int i, n;
+	n = ARRAY_SIZE(charging_profile_options);
+	for (i = 0; i < n; ++i) {
+		sprintf(buffer + strlen(buffer), "%s",
+			charging_profile_options[i].descriptor);
+		if (i < n - 1)
+			sprintf(buffer + strlen(buffer), " ");
+		else
+			sprintf(buffer + strlen(buffer), "\n");
+	}
+
+	return strlen(buffer);
+}
+
+static ssize_t uw_charging_profile_show(struct device *child,
+					struct device_attribute *attr, char *buffer)
+{
+	u8 charging_profile_value;
+	int i, result;
+
+	result = uw_get_charging_profile(&charging_profile_value);
+	if (result != 0)
+		return result;
+
+	for (i = 0; i < ARRAY_SIZE(charging_profile_options); ++i)
+		if (charging_profile_options[i].value == charging_profile_value) {
+			sprintf(buffer, "%s\n", charging_profile_options[i].descriptor);
+			return strlen(buffer);
+		}
+
+	pr_err("Read charging profile value not matched to a descriptor\n");
+
+	return -EIO;
+}
+
+static ssize_t uw_charging_profile_store(struct device *child,
+					 struct device_attribute *attr,
+					 const char *buffer, size_t size)
+{
+	u8 charging_profile_value;
+	int i, result;
+	char *buffer_copy;
+	char *charging_profile_descriptor;
+	buffer_copy = kmalloc(size + 1, GFP_KERNEL);
+	strcpy(buffer_copy, buffer);
+	charging_profile_descriptor = strstrip(buffer_copy);
+
+	for (i = 0; i < ARRAY_SIZE(charging_profile_options); ++i)
+		if (strcmp(charging_profile_options[i].descriptor, charging_profile_descriptor) == 0) {
+			charging_profile_value = charging_profile_options[i].value;
+			break;
+		}
+
+	kfree(buffer_copy);
+
+	if (i < ARRAY_SIZE(charging_profile_options)) {
+		// Option found try to set
+		result = uw_set_charging_profile(charging_profile_value);
+		if (result == 0)
+			return size;
+		else
+			return -EIO;
+	} else
+		// Invalid input, not matched to an option
+		return -EINVAL;
+}
+
+struct uw_charging_profile_attrs_t {
+	struct device_attribute charging_profiles_available;
+	struct device_attribute charging_profile;
+} uw_charging_profile_attrs = {
+	.charging_profiles_available = __ATTR(charging_profiles_available, 0444, uw_charging_profiles_available_show, NULL),
+	.charging_profile = __ATTR(charging_profile, 0644, uw_charging_profile_show, uw_charging_profile_store)
+};
+
+static struct attribute *uw_charging_profile_attrs_list[] = {
+	&uw_charging_profile_attrs.charging_profiles_available.attr,
+	&uw_charging_profile_attrs.charging_profile.attr,
+	NULL
+};
+
+static struct attribute_group uw_charging_profile_attr_group = {
+	.name = "uw_charging_profile",
+	.attrs = uw_charging_profile_attrs_list
+};
+
 static struct char_to_u8_t charging_prio_options[] = {
 	{ .descriptor = "charge_battery", .value = 0x00 },
 	{ .descriptor = "performance",    .value = 0x01 }

From 32e16b49f3414d7a78c7fbf0bc0516200720a012 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 18 Nov 2022 19:01:27 +0100
Subject: [PATCH 08/13] Fix comment typo

---
 src/uniwill_keyboard.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 1ee8ee7..e0963c6 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -811,7 +811,7 @@ static int uw_has_charging_priority(bool *status)
 }
 
 /*
- * charging_prio values
+ * charging_profile values
  *     0 => high capacity
  *     1 => balanced
  *     2 => stationary

From d6f32f72701cc6993fe780f70216c30390e721b8 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 18 Nov 2022 19:45:03 +0100
Subject: [PATCH 09/13] Add uw charging profile init

---
 src/uniwill_keyboard.h | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index e0963c6..ba8b1f7 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -810,6 +810,8 @@ static int uw_has_charging_priority(bool *status)
 	return result;
 }
 
+static bool uw_charging_profile_loaded = false;
+
 /*
  * charging_profile values
  *     0 => high capacity
@@ -1130,6 +1132,8 @@ static int uniwill_keyboard_probe(struct platform_device *dev)
 	if (uw_feats->uniwill_has_charging_prio)
 		uw_charging_prio_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_prio_attr_group) == 0;
 
+	uw_charging_profile_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_profile_attr_group) == 0;
+
 	return 0;
 }
 
@@ -1138,6 +1142,9 @@ static int uniwill_keyboard_remove(struct platform_device *dev)
 	if (uw_charging_prio_loaded)
 		sysfs_remove_group(&dev->dev.kobj, &uw_charging_prio_attr_group);
 
+	if (uw_charging_profile_loaded)
+		sysfs_remove_group(&dev->dev.kobj, &uw_charging_profile_attr_group);
+
 	if (uniwill_kbd_bl_type_rgb_single_color) {
 		sysfs_remove_group(&dev->dev.kobj, &uw_kbd_bl_color_attr_group);
 	}

From 97ddaed7820dd93c5934551004ad19d09684f265 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Mon, 21 Nov 2022 14:47:55 +0100
Subject: [PATCH 10/13] Add id check for charging profile

---
 src/uniwill_interfaces.h |  1 +
 src/uniwill_keyboard.h   | 18 +++++++++++++++++-
 2 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/src/uniwill_interfaces.h b/src/uniwill_interfaces.h
index c0a3ab0..f8895e4 100644
--- a/src/uniwill_interfaces.h
+++ b/src/uniwill_interfaces.h
@@ -69,6 +69,7 @@ struct uniwill_device_features_t {
 	bool uniwill_profile_v1_three_profs;
 	bool uniwill_profile_v1_three_profs_leds_only;
 	bool uniwill_has_charging_prio;
+	bool uniwill_has_charging_profile;
 };
 
 u32 uniwill_add_interface(struct uniwill_interface_t *new_interface);
diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index ba8b1f7..73b28a3 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -842,6 +842,20 @@ static int uw_get_charging_profile(u8 *charging_profile)
 	return result;
 }
 
+static int uw_has_charging_profile(bool *status)
+{
+	u8 data;
+	int result;
+	result = uniwill_read_ec_ram(0x078e, &data);
+
+	if (data & (1 << 3))
+		*status = true;
+	else
+		*status = false;
+
+	return result;
+}
+
 struct char_to_u8_t {
 	char* descriptor;
 	u8 value;
@@ -1089,6 +1103,7 @@ struct uniwill_device_features_t *uniwill_get_device_features(void)
 		uw_feats->uniwill_profile_v1_three_profs;
 
 	uw_has_charging_priority(&uw_feats->uniwill_has_charging_prio);
+	uw_has_charging_profile(&uw_feats->uniwill_has_charging_profile);
 
 	return uw_feats;
 }
@@ -1132,7 +1147,8 @@ static int uniwill_keyboard_probe(struct platform_device *dev)
 	if (uw_feats->uniwill_has_charging_prio)
 		uw_charging_prio_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_prio_attr_group) == 0;
 
-	uw_charging_profile_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_profile_attr_group) == 0;
+	if (uw_feats->uniwill_has_charging_prio)
+		uw_charging_profile_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_profile_attr_group) == 0;
 
 	return 0;
 }

From 362e16e703d20b2d00fbd105324957049df5bc79 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Mon, 21 Nov 2022 17:00:04 +0100
Subject: [PATCH 11/13] Remove attribute group name prefix

---
 src/uniwill_keyboard.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 73b28a3..2296c1e 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -953,7 +953,7 @@ static struct attribute *uw_charging_profile_attrs_list[] = {
 };
 
 static struct attribute_group uw_charging_profile_attr_group = {
-	.name = "uw_charging_profile",
+	.name = "charging_profile",
 	.attrs = uw_charging_profile_attrs_list
 };
 
@@ -1048,7 +1048,7 @@ static struct attribute *uw_charging_prio_attrs_list[] = {
 };
 
 static struct attribute_group uw_charging_prio_attr_group = {
-	.name = "uw_charging_priority",
+	.name = "charging_priority",
 	.attrs = uw_charging_prio_attrs_list
 };
 

From bb4885cb571efcce4cdf2d0eb342c84415a3aee4 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Mon, 21 Nov 2022 17:47:28 +0100
Subject: [PATCH 12/13] Fix feature check

---
 src/uniwill_keyboard.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 2296c1e..9729a3b 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -1147,7 +1147,7 @@ static int uniwill_keyboard_probe(struct platform_device *dev)
 	if (uw_feats->uniwill_has_charging_prio)
 		uw_charging_prio_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_prio_attr_group) == 0;
 
-	if (uw_feats->uniwill_has_charging_prio)
+	if (uw_feats->uniwill_has_charging_profile)
 		uw_charging_profile_loaded = sysfs_create_group(&dev->dev.kobj, &uw_charging_profile_attr_group) == 0;
 
 	return 0;

From f2c37fcc06cd55860b86a8d2abff9d92bd76ca08 Mon Sep 17 00:00:00 2001
From: Christoffer Sandberg <cs@tuxedo.de>
Date: Fri, 16 Dec 2022 18:05:09 +0100
Subject: [PATCH 13/13] Exclude devices where interface is known to be
 incompatible with feature

---
 src/uniwill_keyboard.h | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/src/uniwill_keyboard.h b/src/uniwill_keyboard.h
index 9729a3b..392b271 100644
--- a/src/uniwill_keyboard.h
+++ b/src/uniwill_keyboard.h
@@ -800,6 +800,17 @@ static int uw_has_charging_priority(bool *status)
 {
 	u8 data;
 	int result;
+
+	bool not_supported_device = false
+		|| dmi_match(DMI_BOARD_NAME, "PF5PU1G")
+		|| dmi_match(DMI_BOARD_NAME, "LAPQC71A")
+		|| dmi_match(DMI_BOARD_NAME, "LAPQC71B")
+		|| dmi_match(DMI_PRODUCT_NAME, "A60 MUV")
+	;
+
+	if (not_supported_device)
+		return false;
+
 	result = uniwill_read_ec_ram(0x0742, &data);
 
 	if (data & (1 << 5))
@@ -846,6 +857,17 @@ static int uw_has_charging_profile(bool *status)
 {
 	u8 data;
 	int result;
+
+	bool not_supported_device = false
+		|| dmi_match(DMI_BOARD_NAME, "PF5PU1G")
+		|| dmi_match(DMI_BOARD_NAME, "LAPQC71A")
+		|| dmi_match(DMI_BOARD_NAME, "LAPQC71B")
+		|| dmi_match(DMI_PRODUCT_NAME, "A60 MUV")
+	;
+
+	if (not_supported_device)
+		return false;
+
 	result = uniwill_read_ec_ram(0x078e, &data);
 
 	if (data & (1 << 3))