From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Hans de Goede <hdegoede@redhat.com>,
Linus Walleij <linus.walleij@linaro.org>,
Alexandre Courbot <gnurou@gmail.com>
Cc: linux-gpio@vger.kernel.org
Subject: Re: [PATCH 0/2] gpio: acpi: acpi_dev_gpio_irq_get: ignore the status of unselected irqs
Date: Mon, 13 Mar 2017 21:19:13 +0200 [thread overview]
Message-ID: <1489432753.20145.226.camel@linux.intel.com> (raw)
In-Reply-To: <3133c2fe-427e-f742-9647-d1f45eeaed4a@redhat.com>
On Mon, 2017-03-13 at 18:56 +0100, Hans de Goede wrote:
> On 13-03-17 18:23, Andy Shevchenko wrote:
> > On Fri, 2017-03-10 at 21:58 +0100, Hans de Goede wrote:
> > I will incorporate slightly different version of this (due to
> > changes
> > happened) into my branch. Please verify if everything is okay.
>
> I just noticed that my patch was botched up a bit, it is
> taking a more complex approach then needed and it contained a bug
> (it did lookup->index = index, which should be lookup->index = i).
>
> I've just finished writing a v2 (sorry).
NP.
Does it mean this one replaces both from v2? (to me looks like)
> I've attached the v2 here,
> note this is against an unmodified v4.11-rc2 rather then your branch
> as I needed a clean base to debug some problems. But it should be
> easy to adapt to your branch I think.
>
> Note this can go upstream either way (through your branch or
> directly since it is based on a clean v4.11-rc2 now) but it is
> probably easier to take it upstream through your branch to
> avoid conflicts.
I will rebase my series on your patch, but I'm going to slightly modify
it anyway (basically squash one of my patches).
--
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy
next prev parent reply other threads:[~2017-03-13 19:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-10 20:58 [PATCH 0/2] gpio: acpi: acpi_dev_gpio_irq_get: ignore the status of unselected irqs Hans de Goede
2017-03-10 20:58 ` [PATCH 1/2] gpio: acpi: acpi_gpio_resource_lookup: return info even if there is no desc Hans de Goede
2017-03-15 10:17 ` Linus Walleij
2017-03-15 10:19 ` Hans de Goede
2017-03-10 20:58 ` [PATCH 2/2] gpio: acpi: acpi_dev_gpio_irq_get: ignore the status of unselected irqs Hans de Goede
2017-03-13 17:23 ` [PATCH 0/2] " Andy Shevchenko
2017-03-13 17:56 ` Hans de Goede
2017-03-13 19:19 ` Andy Shevchenko [this message]
2017-03-13 19:23 ` Hans de Goede
2017-03-13 19:31 ` Andy Shevchenko
2017-03-13 19:38 ` Andy Shevchenko
2017-03-13 22:02 ` Hans de Goede
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1489432753.20145.226.camel@linux.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=gnurou@gmail.com \
--cc=hdegoede@redhat.com \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.