From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933165AbbENKdV (ORCPT ); Thu, 14 May 2015 06:33:21 -0400 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:47043 "EHLO opensource.wolfsonmicro.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932627AbbENKdT (ORCPT ); Thu, 14 May 2015 06:33:19 -0400 Date: Thu, 14 May 2015 11:33:14 +0100 From: Richard Fitzgerald To: Lee Jones Cc: linux-kernel@vger.kernel.org, patches@opensource.wolfsonmicro.com Subject: Re: [PATCH v4] mfd: arizona: Add support for WM8998 and WM1814 Message-ID: <20150514103314.GA2832@opensource.wolfsonmicro.com> References: <1431595499-12344-1-git-send-email-rf@opensource.wolfsonmicro.com> <20150514100833.GH22418@x1> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150514100833.GH22418@x1> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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. 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. > > 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