From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v4 4/5] mfd: tps65217: Add power button as subdevice Date: Tue, 30 Aug 2016 10:03:29 +0100 Message-ID: <20160830090329.GD1661@dell> References: <20160620105056.25843-1-m.niestroj@grinn-global.com> <20160620105056.25843-5-m.niestroj@grinn-global.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-pm-owner@vger.kernel.org To: Marcin Niestroj Cc: Tony Lindgren , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , Rob Herring , Pawel Moll , linux-omap@vger.kernel.org, linux-pm@vger.kernel.org, linux-input@vger.kernel.org, devicetree@vger.kernel.org, Grygorii Strashko List-Id: devicetree@vger.kernel.org On Mon, 29 Aug 2016, Marcin Niestroj wrote: > ping Don't do that! If you think the patch hasn't attracted attention in >2 weeks, then it's probably slipped through the gaps and you need to send a [RESEND]. However ... > On 20.06.2016 12:50, Marcin Niestroj wrote: > > Add tps65217 power buttor subdevice with assigned IRQ resources. > > > > Signed-off-by: Marcin Niestroj > > Acked-by: Lee Jones This patch has a maintainer Ack, so why are you pinging? > > --- > > Depends on patch 1 in series > > > > Changes v1 -> v4: none > > > > drivers/mfd/tps65217.c | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c > > index 41b5d59..57c8741 100644 > > --- a/drivers/mfd/tps65217.c > > +++ b/drivers/mfd/tps65217.c > > @@ -38,6 +38,10 @@ static struct resource charger_resources[] = { > > DEFINE_RES_IRQ_NAMED(TPS65217_IRQ_USB, "USB"), > > }; > > > > +static struct resource pb_resources[] = { > > + DEFINE_RES_IRQ_NAMED(TPS65217_IRQ_PB, "PB"), > > +}; > > + > > struct tps65217_irq { > > int mask; > > int interrupt; > > @@ -122,6 +126,12 @@ static struct mfd_cell tps65217s[] = { > > .resources = charger_resources, > > .of_compatible = "ti,tps65217-charger", > > }, > > + { > > + .name = "tps65217-pwrbutton", > > + .num_resources = ARRAY_SIZE(pb_resources), > > + .resources = pb_resources, > > + .of_compatible = "ti,tps65217-pwrbutton", > > + }, > > }; > > > > static irqreturn_t tps65217_irq_thread(int irq, void *data) > > > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog