From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH 0/2] Support CrystalCove PMIC ACPI operation region Date: Mon, 15 Sep 2014 23:43:17 +0100 Message-ID: <20140915224317.GF25162@lee--X1> References: <1410229968-11638-1-git-send-email-aaron.lu@intel.com> <20140915025727.GA14730@aaronlu.sh.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <20140915025727.GA14730@aaronlu.sh.intel.com> Sender: linux-kernel-owner@vger.kernel.org To: Aaron Lu Cc: Linus Walleij , Alexandre Courbot , Samuel Ortiz , Arnd Bergmann , linux-gpio@vger.kernel.org, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, Jacob Pan , Lejun Zhu , Radivoje Jovanovic , Daniel =?iso-8859-1?Q?Gl=F6ckner?= , linux-acpi@vger.kernel.org, "Rafael J. Wysocki" List-Id: linux-acpi@vger.kernel.org On Mon, 15 Sep 2014, Aaron Lu wrote: > I wonder if anyone has looked at this? >=20 > Note that this is for CrystalCove PMIC operation region support and t= he > CrystalCove driver is already in mainline, not for the DollarCove PMI= C > driver that Jacob is currently upstreaming, so the patch doesn't have > any dependency. Not yet. It is on my list of TODOs though. > On Tue, Sep 09, 2014 at 10:32:46AM +0800, Aaron Lu wrote: > > The two patches add support for CrystalCove PMIC ACPI operation reg= ion. > > The PMIC chip has two customized operation regions: one for power r= ail > > manipulation and one for thermal purpose: sensor temperature readin= g > > and trip point value reading/setting. > >=20 > > For an example ASL code on ASUS T100 with CrystalCove PMIC, see her= e: > > https://gist.github.com/aaronlu/f5f65771a6c3251fae5d > >=20 > > Aaron Lu (2): > > gpio / CrystalCove: support virtual GPIO > > PMIC / opregion: support PMIC customized operation region for > > CrystalCove > >=20 > > drivers/gpio/gpio-crystalcove.c | 19 +- > > drivers/mfd/Kconfig | 11 + > > drivers/mfd/Makefile | 1 + > > drivers/mfd/intel_soc_pmic_crc.c | 3 + > > drivers/mfd/intel_soc_pmic_crc_opregion.c | 229 ++++++++++++++++++= + > > drivers/mfd/intel_soc_pmic_opregion.c | 350 ++++++++++++++++++= ++++++++++++ > > drivers/mfd/intel_soc_pmic_opregion.h | 35 +++ > > include/asm-generic/gpio.h | 2 +- > > 8 files changed, 646 insertions(+), 4 deletions(-) > > create mode 100644 drivers/mfd/intel_soc_pmic_crc_opregion.c > > create mode 100644 drivers/mfd/intel_soc_pmic_opregion.c > > create mode 100644 drivers/mfd/intel_soc_pmic_opregion.h > >=20 --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog