From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933158AbbENMlj (ORCPT ); Thu, 14 May 2015 08:41:39 -0400 Received: from mail-wg0-f50.google.com ([74.125.82.50]:35041 "EHLO mail-wg0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932498AbbENMlh (ORCPT ); Thu, 14 May 2015 08:41:37 -0400 Date: Thu, 14 May 2015 13:41:32 +0100 From: Lee Jones To: Richard Fitzgerald Cc: linux-kernel@vger.kernel.org, patches@opensource.wolfsonmicro.com Subject: Re: [PATCH v4] mfd: arizona: Add support for WM8998 and WM1814 Message-ID: <20150514124132.GJ22418@x1> References: <1431595499-12344-1-git-send-email-rf@opensource.wolfsonmicro.com> <20150514100833.GH22418@x1> <20150514103314.GA2832@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20150514103314.GA2832@opensource.wolfsonmicro.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 14 May 2015, Richard Fitzgerald wrote: > On Thu, May 14, 2015 at 11:08:33AM +0100, Lee Jones wrote: > > On Thu, 14 May 2015, Richard Fitzgerald wrote: > > > > > Signed-off-by: Richard Fitzgerald > > > --- > > > > This is v4, so where's the change log? > > > > I looked though my own history of the patch and I see that I already > > Acked v3, so I'm pretty unsure what's happening. > > > > I'm unsure too. Last I have from you on the v3 of this patch is your > question of whether it's orthogonal enough for you to just take it. You've > acked it, but not said that you've applied it and your for-mfd-next tree > doesn't contain it yet so I assumed that you haven't applied it yet. > Therefore... I've done v4 which fixes the incorrect makefile conditional > that was in the v3 patch. Right, so unless there are major changes, the Ack should be applied to this patch. If there are changes, a change-log is always helpful. Without one I have to re-review the whole 2k lines, which for obvious reasons I'd like to avoid. > If you are busy applying the v3 patch and want to take that as-is I can > do a separate bugfix patch on top of v3 to fix the makefile. I haven't applied the patch, only Acked it. > > > drivers/mfd/Kconfig | 6 + > > > drivers/mfd/Makefile | 3 + > > > drivers/mfd/arizona-core.c | 105 ++- > > > drivers/mfd/arizona-i2c.c | 8 + > > > drivers/mfd/arizona-irq.c | 9 + > > > drivers/mfd/arizona.h | 5 + > > > drivers/mfd/wm8998-tables.c | 1592 +++++++++++++++++++++++++++++++++ > > > include/linux/mfd/arizona/core.h | 3 + > > > include/linux/mfd/arizona/pdata.h | 2 + > > > include/linux/mfd/arizona/registers.h | 220 +++++ > > > 10 files changed, 1945 insertions(+), 8 deletions(-) > > > create mode 100644 drivers/mfd/wm8998-tables.c > > > > > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > > > index 048f5bf..d2e549b 100644 > > > --- a/drivers/mfd/Kconfig > > > +++ b/drivers/mfd/Kconfig > > > @@ -1359,6 +1359,12 @@ config MFD_WM8997 > > > help > > > Support for Wolfson Microelectronics WM8997 low power audio SoC > > > > > > +config MFD_WM8998 > > > + bool "Wolfson Microelectronics WM8998" > > > + depends on MFD_ARIZONA > > > + help > > > + Support for Wolfson Microelectronics WM8998 low power audio SoC > > > + > > > config MFD_WM8400 > > > bool "Wolfson Microelectronics WM8400" > > > select MFD_CORE > > > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > > > index ca6d184..d244afc 100644 > > > --- a/drivers/mfd/Makefile > > > +++ b/drivers/mfd/Makefile > > > @@ -50,6 +50,9 @@ endif > > > ifneq ($(CONFIG_MFD_WM8997),n) > > > obj-$(CONFIG_MFD_ARIZONA) += wm8997-tables.o > > > endif > > > +ifeq ($(CONFIG_MFD_WM8998),y) > > > +obj-$(CONFIG_MFD_ARIZONA) += wm8998-tables.o > > > +endif -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog