On Mon, 11 Apr 2016, Grygorii Strashko wrote: > On 04/11/2016 01:37 PM, Lee Jones wrote: > >On Mon, 11 Apr 2016, Grygorii Strashko wrote: > > > >>On 04/11/2016 12:44 PM, Lee Jones wrote: > >>>On Mon, 04 Apr 2016, Marcin Niestroj wrote: > >>> > >>>>Add tps65217 power buttor subdevice with assigned IRQ resources. > >>>> > >>>>Signed-off-by: Marcin Niestroj <m.niestroj@xxxxxxxxxxxxxxxx> > >>>>--- > >>>> drivers/mfd/tps65217.c | 10 ++++++++++ > >>>> 1 file changed, 10 insertions(+) > >>> > >>>Applied, thanks. > >> > >>Seems there is dependency from patch 1 :( > > > >What is the dependency? > > TPS65217_IRQ_PB definition added in patch 1. Very well. Unappiled. > >>>>diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c > >>>>index a20980f..fd3b5d8 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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html