ACPI / PMIC: Stop xpower OPRegion handler relying on IIO
authorHans de Goede <hdegoede@redhat.com>
Wed, 19 Apr 2017 13:07:00 +0000 (15:07 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 20 Apr 2017 10:43:43 +0000 (12:43 +0200)
commit2e5a7f71095be27064c140faf6cecdab585ff198
treef21022fecf177df983db08b2ede5312e522271ac
parentac2c4936e9ec76f1d5c4cd2afdc8258769635b7a
ACPI / PMIC: Stop xpower OPRegion handler relying on IIO

The intel_pmic_xpower code provides an OPRegion handler, which must be
available before other drivers using it are loaded, which can only be
ensured if both the mfd and opregion drivers are built in, which is why
the Kconfig option for intel_pmic_xpower is a bool.

The use of IIO is causing trouble for generic distro configs here as
distros will typically want to build IIO drivers as modules and there
really is no reason to use IIO here. The reading of the ADC value is a
single regmap_bulk_read, which is already protected against races by
the regmap-lock.

This commit removes the use of IIO, allowing distros to enable the
driver without needing to built IIO in and also actually simplifies
the code.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/Kconfig
drivers/acpi/pmic/intel_pmic_xpower.c