On Wed, 15 Jun 2016, Marcin Niestroj wrote:
> Hi,
>
> On 15.06.2016 16:12, Lee Jones wrote:
> >On Tue, 14 Jun 2016, Marcin Niestroj wrote:
> >
> >>Add tps65217 power buttor subdevice with assigned IRQ resources.
> >>
> >>Signed-off-by: Marcin Niestroj <m.niestroj@xxxxxxxxxxxxxxxx>
> >>---
> >>depends on patch 1 in series
> >>
> >> drivers/mfd/tps65217.c | 10 ++++++++++
> >> 1 file changed, 10 insertions(+)
> >
> >Applied, thanks.
>
> Did you also apply patch 1? As I stated above, this patch depends on
> patch 1.
Nope. It looks like you still have changes to make.
What does it rely on? The IRQ define?
> >>diff --git a/drivers/mfd/tps65217.c b/drivers/mfd/tps65217.c
> >>index d49f94e..68cf431 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 devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
