Commit d8139f63 authored by Aaron Lu's avatar Aaron Lu Committed by Rafael J. Wysocki
Browse files

ACPI / PMIC: support PMIC operation region for XPower AXP288 



The Baytrail-T-CR platform firmware has defined two customized operation
regions for PMIC chip Dollar Cove XPower - one is for power resource
handling and one is for thermal just like the CrystalCove one. This patch
adds support for them on top of the common PMIC opregion region code.

Signed-off-by: default avatarAaron Lu <aaron.lu@intel.com>
Acked-by: Lee Jones <lee.jones@linaro.org> for the MFD part
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent b1eea857

drivers/acpi/Kconfig

+6 −0
Original line number Diff line number Diff line
@@ -409,6 +409,12 @@ config CRC_PMIC_OPREGION 
	help

	  This config adds ACPI operation region support for CrystalCove PMIC.



config XPOWER_PMIC_OPREGION

	bool "ACPI operation region support for XPower AXP288 PMIC"

	depends on AXP288_ADC = y

	help

	  This config adds ACPI operation region support for XPower AXP288 PMIC.



endif



endif	# ACPI

drivers/acpi/Makefile

+1 −0
Original line number Diff line number Diff line
@@ -90,3 +90,4 @@ obj-$(CONFIG_ACPI_EXTLOG) += acpi_extlog.o 


obj-$(CONFIG_PMIC_OPREGION)	+= pmic/intel_pmic.o

obj-$(CONFIG_CRC_PMIC_OPREGION) += pmic/intel_pmic_crc.o

obj-$(CONFIG_XPOWER_PMIC_OPREGION) += pmic/intel_pmic_xpower.o

drivers/acpi/pmic/intel_pmic_xpower.c

0 → 100644
+246 −0
Original line number Diff line number Diff line 
/*

 * intel_pmic_xpower.c - XPower AXP288 PMIC operation region driver

 *

 * Copyright (C) 2014 Intel Corporation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or

 * modify it under the terms of the GNU General Public License version

 * 2 as published by the Free Software Foundation.

 *

 * This program is distributed in the hope that it will be useful,

 * but WITHOUT ANY WARRANTY; without even the implied warranty of

 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

 * GNU General Public License for more details.

 */



#include <linux/module.h>

#include <linux/acpi.h>

#include <linux/mfd/axp20x.h>

#include <linux/regmap.h>

#include <linux/platform_device.h>

#include <linux/iio/consumer.h>

#include "intel_pmic.h"



#define XPOWER_GPADC_LOW	0x5b



static struct pmic_table power_table[] = {

	{

		.address = 0x00,

		.reg = 0x13,

		.bit = 0x05,

	},

	{

		.address = 0x04,

		.reg = 0x13,

		.bit = 0x06,

	},

	{

		.address = 0x08,

		.reg = 0x13,

		.bit = 0x07,

	},

	{

		.address = 0x0c,

		.reg = 0x12,

		.bit = 0x03,

	},

	{

		.address = 0x10,

		.reg = 0x12,

		.bit = 0x04,

	},

	{

		.address = 0x14,

		.reg = 0x12,

		.bit = 0x05,

	},

	{

		.address = 0x18,

		.reg = 0x12,

		.bit = 0x06,

	},

	{

		.address = 0x1c,

		.reg = 0x12,

		.bit = 0x00,

	},

	{

		.address = 0x20,

		.reg = 0x12,

		.bit = 0x01,

	},

	{

		.address = 0x24,

		.reg = 0x12,

		.bit = 0x02,

	},

	{

		.address = 0x28,

		.reg = 0x13,

		.bit = 0x02,

	},

	{

		.address = 0x2c,

		.reg = 0x13,

		.bit = 0x03,

	},

	{

		.address = 0x30,

		.reg = 0x13,

		.bit = 0x04,

	},

	{

		.address = 0x38,

		.reg = 0x10,

		.bit = 0x03,

	},

	{

		.address = 0x3c,

		.reg = 0x10,

		.bit = 0x06,

	},

	{

		.address = 0x40,

		.reg = 0x10,

		.bit = 0x05,

	},

	{

		.address = 0x44,

		.reg = 0x10,

		.bit = 0x04,

	},

	{

		.address = 0x48,

		.reg = 0x10,

		.bit = 0x01,

	},

	{

		.address = 0x4c,

		.reg = 0x10,

		.bit = 0x00

	},

};



/* TMP0 - TMP5 are the same, all from GPADC */

static struct pmic_table thermal_table[] = {

	{

		.address = 0x00,

		.reg = XPOWER_GPADC_LOW

	},

	{

		.address = 0x0c,

		.reg = XPOWER_GPADC_LOW

	},

	{

		.address = 0x18,

		.reg = XPOWER_GPADC_LOW

	},

	{

		.address = 0x24,

		.reg = XPOWER_GPADC_LOW

	},

	{

		.address = 0x30,

		.reg = XPOWER_GPADC_LOW

	},

	{

		.address = 0x3c,

		.reg = XPOWER_GPADC_LOW

	},

};



static int intel_xpower_pmic_get_power(struct regmap *regmap, int reg,

				       int bit, u64 *value)

{

	int data;



	if (regmap_read(regmap, reg, &data))

		return -EIO;



	*value = (data & BIT(bit)) ? 1 : 0;

	return 0;

}



static int intel_xpower_pmic_update_power(struct regmap *regmap, int reg,

					  int bit, bool on)

{

	int data;



	if (regmap_read(regmap, reg, &data))

		return -EIO;



	if (on)

		data |= BIT(bit);

	else

		data &= ~BIT(bit);



	if (regmap_write(regmap, reg, data))

		return -EIO;



	return 0;

}



/**

 * intel_xpower_pmic_get_raw_temp(): Get raw temperature reading from the PMIC

 *

 * @regmap: regmap of the PMIC device

 * @reg: register to get the reading

 *

 * We could get the sensor value by manipulating the HW regs here, but since

 * the axp288 IIO driver may also access the same regs at the same time, the

 * APIs provided by IIO subsystem are used here instead to avoid problems. As

 * a result, the two passed in params are of no actual use.

 *

 * Return a positive value on success, errno on failure.

 */

static int intel_xpower_pmic_get_raw_temp(struct regmap *regmap, int reg)

{

	struct iio_channel *gpadc_chan;

	int ret, val;



	gpadc_chan = iio_channel_get(NULL, "axp288-system-temp");

	if (IS_ERR_OR_NULL(gpadc_chan))

		return -EACCES;



	ret = iio_read_channel_raw(gpadc_chan, &val);

	if (ret < 0)

		val = ret;



	iio_channel_release(gpadc_chan);

	return val;

}



static struct intel_pmic_opregion_data intel_xpower_pmic_opregion_data = {

	.get_power = intel_xpower_pmic_get_power,

	.update_power = intel_xpower_pmic_update_power,

	.get_raw_temp = intel_xpower_pmic_get_raw_temp,

	.power_table = power_table,

	.power_table_count = ARRAY_SIZE(power_table),

	.thermal_table = thermal_table,

	.thermal_table_count = ARRAY_SIZE(thermal_table),

};





static int intel_xpower_pmic_opregion_probe(struct platform_device *pdev)

{

	struct axp20x_dev *axp20x = dev_get_drvdata(pdev->dev.parent);

	return intel_pmic_install_opregion_handler(&pdev->dev,

			ACPI_HANDLE(pdev->dev.parent), axp20x->regmap,

			&intel_xpower_pmic_opregion_data);

}



static struct platform_driver intel_xpower_pmic_opregion_driver = {

	.probe = intel_xpower_pmic_opregion_probe,

	.driver = {

		.name = "axp288_pmic_acpi",

	},

};



static int __init intel_xpower_pmic_opregion_driver_init(void)

{

	return platform_driver_register(&intel_xpower_pmic_opregion_driver);

}

module_init(intel_xpower_pmic_opregion_driver_init);



MODULE_DESCRIPTION("XPower AXP288 ACPI operation region driver");

MODULE_LICENSE("GPL");

drivers/mfd/axp20x.c

+3 −0
Original line number Diff line number Diff line
@@ -354,6 +354,9 @@ static struct mfd_cell axp288_cells[] = { 
		.num_resources = ARRAY_SIZE(axp288_battery_resources),

		.resources = axp288_battery_resources,

	},

	{

		.name = "axp288_pmic_acpi",

	},

};



static struct axp20x_dev *axp20x_pm_power_off;

CRA Git | Maintained and supported by SUSTech CRA and CCSE