[PATCH v5 3/6] platform/x86: int3472: Map the VD55G0 supplies to "vddio" and "vana"

[Sergey Lebedev <lsa.uz@pm.me>]

The ST VD55G0 IR camera of the Surface Pro 8 and of the Surface Pro 11
with Intel (ACPI HID SMO55F0) is powered through two GPIOs of an INT3472
discrete device: the power enable, and AVDD. Which rail each one feeds
is not documented, and it cannot be measured here: the sensor answers on
I2C only with both on.

Map AVDD to "vana". The power enable takes "vddio"; whether it feeds
VDDIO or VCORE is not known either. VCORE is not described, so the
driver gets a dummy regulator for it. The vd55g0 driver enables all
three together, so the names do not change what the sensor gets.

Tested-by: Emfox Zhou <emfox@debian.org>
Signed-off-by: Sergey Lebedev <lsa.uz@pm.me>
---
v5: on media-committers next, where type 0x10 is INT3472_GPIO_TYPE_AVDD
(Hans de Goede's series), so the vana entry keys on it, and the message
no longer quotes Intel's name for it. Emfox's Tested-by, from the
Surface Pro 8, which this map covers as it stands.
New in v4, so that 2/6 uses the same supply names on DT and ACPI
(Sakari). 2/6 needs it on ACPI.

 drivers/platform/x86/intel/int3472/discrete.c | 24 +++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/drivers/platform/x86/intel/int3472/discrete.c b/drivers/platform/x86/intel/int3472/discrete.c
index ff94dc2ad8a..abd20345f33 100644
--- a/drivers/platform/x86/intel/int3472/discrete.c
+++ b/drivers/platform/x86/intel/int3472/discrete.c
@@ -150,6 +150,16 @@ static const char * const power_enable_hids_vdda[] = {
 	NULL
 };
 
+static const char * const power_enable_hids_vddio[] = {
+	"SMO55F0", /* vd55g0 on Microsoft Surface Pro 8 and 11 */
+	NULL
+};
+
+static const char * const avdd_hids_vana[] = {
+	"SMO55F0", /* vd55g0 on Microsoft Surface Pro 8 and 11 */
+	NULL
+};
+
 /**
  * struct int3472_gpio_map - Map GPIOs to whatever is expected by the
  * sensor driver (as in DT bindings)
@@ -199,6 +209,20 @@ static const struct int3472_gpio_map int3472_gpio_map[] = {
 		.con_id = "vana",
 		.enable_time_us = GPIO_REGULATOR_ENABLE_TIME,
 	},
+	{	/* Sensors which expect "vddio" as con_id for power enable */
+		.hids = power_enable_hids_vddio,
+		.type_from = INT3472_GPIO_TYPE_POWER_ENABLE,
+		.type_to = INT3472_GPIO_TYPE_POWER_ENABLE,
+		.con_id = "vddio",
+		.enable_time_us = GPIO_REGULATOR_ENABLE_TIME,
+	},
+	{	/* Sensors which expect "vana" as con_id for AVDD */
+		.hids = avdd_hids_vana,
+		.type_from = INT3472_GPIO_TYPE_AVDD,
+		.type_to = INT3472_GPIO_TYPE_AVDD,
+		.con_id = "vana",
+		.enable_time_us = GPIO_REGULATOR_ENABLE_TIME,
+	},
 };
 
 static bool int3472_gpio_map_hids_match(struct acpi_device *adev,
-- 
2.54.0