diff options
| author | Hans de Goede <hdegoede@redhat.com> | 2021-12-03 13:28:51 +0300 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2021-12-13 13:44:48 +0300 |
| commit | 71102bc7964342f0aaf1faf7aa384678b1207848 (patch) | |
| tree | 29a0b3d0c479b82aedffeb19e10d847392de64ab /drivers/platform/x86/intel/int3472/common.c | |
| parent | a2f9fbc247eea0ad1b0b59bc29bec144c5ead03c (diff) | |
| download | linux-71102bc7964342f0aaf1faf7aa384678b1207848.tar.xz | |
platform/x86: int3472: Add get_sensor_adev_and_name() helper
The discrete.c code is not the only code which needs to lookup the
acpi_device and device-name for the sensor for which the INT3472
ACPI-device is a GPIO/clk/regulator provider.
The tps68470.c code also needs this functionality, so factor this
out into a new get_sensor_adev_and_name() helper.
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20211203102857.44539-9-hdegoede@redhat.com
Diffstat (limited to 'drivers/platform/x86/intel/int3472/common.c')
| -rw-r--r-- | drivers/platform/x86/intel/int3472/common.c | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/drivers/platform/x86/intel/int3472/common.c b/drivers/platform/x86/intel/int3472/common.c index 350655a9515b..77cf058e4168 100644 --- a/drivers/platform/x86/intel/int3472/common.c +++ b/drivers/platform/x86/intel/int3472/common.c @@ -52,3 +52,31 @@ out_free_obj: kfree(obj); return ret; } + +/* sensor_adev_ret may be NULL, name_ret must not be NULL */ +int skl_int3472_get_sensor_adev_and_name(struct device *dev, + struct acpi_device **sensor_adev_ret, + const char **name_ret) +{ + struct acpi_device *adev = ACPI_COMPANION(dev); + struct acpi_device *sensor; + int ret = 0; + + sensor = acpi_dev_get_first_consumer_dev(adev); + if (!sensor) { + dev_err(dev, "INT3472 seems to have no dependents.\n"); + return -ENODEV; + } + + *name_ret = devm_kasprintf(dev, GFP_KERNEL, I2C_DEV_NAME_FORMAT, + acpi_dev_name(sensor)); + if (!*name_ret) + ret = -ENOMEM; + + if (ret == 0 && sensor_adev_ret) + *sensor_adev_ret = sensor; + else + acpi_dev_put(sensor); + + return ret; +} |