From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com ([192.55.52.115]:21727 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751411AbaILTgt (ORCPT ); Fri, 12 Sep 2014 15:36:49 -0400 Date: Fri, 12 Sep 2014 12:36:45 -0700 From: Jacob Pan To: Maxime Ripard Cc: IIO , LKML , DEVICE TREE , Lee Jones , Carlo Caione , Srinivas Pandruvada , Aaron Lu , Alan Cox , Jean Delvare , Samuel Ortiz , Liam Girdwood , Mark Brown , Grant Likely , Greg Kroah-Hartman , Rob Herring , Lars-Peter Clausen , Hartmut Knaack , Fugang Duan , Arnd Bergmann , Zubair Lutfullah , Sebastian Reichel , Johannes Thumshirn , Philippe Reynes , Angelo Compagnucci , Doug Anderson , Ramakrishna Pallala , Peter Meerwald Subject: Re: [PATCH v3 0/5] Initial support for XPowers AXP288 PMIC Message-ID: <20140912123645.3bc6058f@ultegra> In-Reply-To: <20140912151824.GU31276@lukather> References: <1410477357-6407-1-git-send-email-jacob.jun.pan@linux.intel.com> <20140912151824.GU31276@lukather> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On Fri, 12 Sep 2014 17:18:24 +0200 Maxime Ripard wrote: > On Thu, Sep 11, 2014 at 04:15:52PM -0700, Jacob Pan wrote: > > XPowers AXP288 is a customized PMIC found on some Intel Baytrail-CR > > platforms. It comes with sub-functions such as USB charging, fuel > > gauge, ADC, and many LDO and BUCK channels. > > > > By extending the existing AXP20x driver, this patchset adds basic > > support for AXP288 PMIC with GPADC as one MFD cell device driver. > > It also adds hooks for ACPI opregion handler driver which can be > > used to handle ACPI requests. > > > > Currently, the PMIC driver in this patchset does not support > > platform data enumeration. But when ACPI _DSD and unified device > > properties become available, cell devices with platform data will > > be added. > > > > This patch does not use intel_soc_pmic core for i2c and regmap > > handling in that axp288 shares similar programming interface with > > other Xpower PMICs supported in axp20x.c. Therefore, extending > > axp20x.c to include axp288 makes more sense. > > > > Changes > > v3: - put all file rename changes in 1/5 > > The variables renaming are still not in 1/5.... > 1/5 is for file rename such that the follow up patches are more readable. There are so many details in variable rename, I think it belongs to the patch that expands the new device support.