All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>
To: Grygorii Strashko
	<grygorii.strashko-l0cyMroinI0@public.gmane.org>,
	Prabhakar Lad
	<prabhakar.csengg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Linus Walleij
	<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org"
	<davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org>,
	Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	"linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	"linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Rob Landley <rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org>,
	Alex Elder <alex.elder-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Grant Likely
	<grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH v4 1/6] gpio: davinci: Fixed a check for unbanked gpio
Date: Wed, 6 Nov 2013 17:12:47 +0530	[thread overview]
Message-ID: <527A2B37.4040308@ti.com> (raw)
In-Reply-To: <527A28EF.9070601-l0cyMroinI0@public.gmane.org>

Hi Grygorii,

On Wednesday 06 November 2013 05:03 PM, Grygorii Strashko wrote:
> Hi Sekhar, Linus,
> 
> On 11/06/2013 12:49 PM, Sekhar Nori wrote:
>> On Wednesday 06 November 2013 03:45 PM, Prabhakar Lad wrote:
>>> Hi Linus,
>>>
>>> On Wed, Nov 6, 2013 at 3:26 PM, Linus Walleij
>>> <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
>>>> On Wed, Nov 6, 2013 at 10:33 AM, Prabhakar Lad
>>>> <prabhakar.csengg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>>> On Tue, Nov 5, 2013 at 6:09 PM, Linus Walleij
>>>>> <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote:
>>>>>> On Sat, Nov 2, 2013 at 4:39 PM, Lad, Prabhakar
>>>>>> <prabhakar.csengg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>>>>
>>>>>>> From: "Lad, Prabhakar" <prabhakar.csengg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>>>>>>
>>>>>>> This patch fixes the check for the offset in
>>>>>>> gpio_to_irq_unbanked() function.
>>>>>>>
>>>>>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>>>>>
>>>>>> Is this a regression that should go in right now?
>>>>>>
>>>>> Yes it needs too.
>>>>
>>>> But on top of *what* exactly?
>>>>
>>>> This does not apply to my gpio tree devel branch and
>>>> not even on the mainline kernel.
>>>>
>>> Looks like this needs to go via ARM tree as the earlier
>>> patches have  gone via ARM tree itself [1].
>>> If you can ACK it Sekhar can get it in via ARM tree.
>>
>> The dependent patches are all in linux-next through ARM SoC queued for
>> v3.13 merge. This fix can either be sent late in merge cycle once Linus
>> has pulled ARM SoC or after v3.13-rc1 comes out.
> 
> Pls, wait a bit - this fix is incomplete :( The below changes have to be
> done too:

Can either you or Prabhakar send a separate patch (series) with only the
fixes?

Thanks,
Sekhar

WARNING: multiple messages have this Message-ID (diff)
From: nsekhar@ti.com (Sekhar Nori)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 1/6] gpio: davinci: Fixed a check for unbanked gpio
Date: Wed, 6 Nov 2013 17:12:47 +0530	[thread overview]
Message-ID: <527A2B37.4040308@ti.com> (raw)
In-Reply-To: <527A28EF.9070601@ti.com>

Hi Grygorii,

On Wednesday 06 November 2013 05:03 PM, Grygorii Strashko wrote:
> Hi Sekhar, Linus,
> 
> On 11/06/2013 12:49 PM, Sekhar Nori wrote:
>> On Wednesday 06 November 2013 03:45 PM, Prabhakar Lad wrote:
>>> Hi Linus,
>>>
>>> On Wed, Nov 6, 2013 at 3:26 PM, Linus Walleij
>>> <linus.walleij@linaro.org> wrote:
>>>> On Wed, Nov 6, 2013 at 10:33 AM, Prabhakar Lad
>>>> <prabhakar.csengg@gmail.com> wrote:
>>>>> On Tue, Nov 5, 2013 at 6:09 PM, Linus Walleij
>>>>> <linus.walleij@linaro.org> wrote:
>>>>>> On Sat, Nov 2, 2013 at 4:39 PM, Lad, Prabhakar
>>>>>> <prabhakar.csengg@gmail.com> wrote:
>>>>>>
>>>>>>> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>>>>>>>
>>>>>>> This patch fixes the check for the offset in
>>>>>>> gpio_to_irq_unbanked() function.
>>>>>>>
>>>>>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>>>>>>
>>>>>> Is this a regression that should go in right now?
>>>>>>
>>>>> Yes it needs too.
>>>>
>>>> But on top of *what* exactly?
>>>>
>>>> This does not apply to my gpio tree devel branch and
>>>> not even on the mainline kernel.
>>>>
>>> Looks like this needs to go via ARM tree as the earlier
>>> patches have  gone via ARM tree itself [1].
>>> If you can ACK it Sekhar can get it in via ARM tree.
>>
>> The dependent patches are all in linux-next through ARM SoC queued for
>> v3.13 merge. This fix can either be sent late in merge cycle once Linus
>> has pulled ARM SoC or after v3.13-rc1 comes out.
> 
> Pls, wait a bit - this fix is incomplete :( The below changes have to be
> done too:

Can either you or Prabhakar send a separate patch (series) with only the
fixes?

Thanks,
Sekhar

WARNING: multiple messages have this Message-ID (diff)
From: Sekhar Nori <nsekhar@ti.com>
To: Grygorii Strashko <grygorii.strashko@ti.com>,
	Prabhakar Lad <prabhakar.csengg@gmail.com>,
	Linus Walleij <linus.walleij@linaro.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"davinci-linux-open-source@linux.davincidsp.com" 
	<davinci-linux-open-source@linux.davincidsp.com>,
	Russell King <linux@arm.linux.org.uk>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
	Pawel Moll <pawel.moll@arm.com>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Rob Herring <rob.herring@calxeda.com>,
	"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
	Rob Landley <rob@landley.net>, Alex Elder <alex.elder@linaro.org>,
	Grant Likely <grant.likely@linaro.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v4 1/6] gpio: davinci: Fixed a check for unbanked gpio
Date: Wed, 6 Nov 2013 17:12:47 +0530	[thread overview]
Message-ID: <527A2B37.4040308@ti.com> (raw)
In-Reply-To: <527A28EF.9070601@ti.com>

Hi Grygorii,

On Wednesday 06 November 2013 05:03 PM, Grygorii Strashko wrote:
> Hi Sekhar, Linus,
> 
> On 11/06/2013 12:49 PM, Sekhar Nori wrote:
>> On Wednesday 06 November 2013 03:45 PM, Prabhakar Lad wrote:
>>> Hi Linus,
>>>
>>> On Wed, Nov 6, 2013 at 3:26 PM, Linus Walleij
>>> <linus.walleij@linaro.org> wrote:
>>>> On Wed, Nov 6, 2013 at 10:33 AM, Prabhakar Lad
>>>> <prabhakar.csengg@gmail.com> wrote:
>>>>> On Tue, Nov 5, 2013 at 6:09 PM, Linus Walleij
>>>>> <linus.walleij@linaro.org> wrote:
>>>>>> On Sat, Nov 2, 2013 at 4:39 PM, Lad, Prabhakar
>>>>>> <prabhakar.csengg@gmail.com> wrote:
>>>>>>
>>>>>>> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>>>>>>>
>>>>>>> This patch fixes the check for the offset in
>>>>>>> gpio_to_irq_unbanked() function.
>>>>>>>
>>>>>>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>>>>>>
>>>>>> Is this a regression that should go in right now?
>>>>>>
>>>>> Yes it needs too.
>>>>
>>>> But on top of *what* exactly?
>>>>
>>>> This does not apply to my gpio tree devel branch and
>>>> not even on the mainline kernel.
>>>>
>>> Looks like this needs to go via ARM tree as the earlier
>>> patches have  gone via ARM tree itself [1].
>>> If you can ACK it Sekhar can get it in via ARM tree.
>>
>> The dependent patches are all in linux-next through ARM SoC queued for
>> v3.13 merge. This fix can either be sent late in merge cycle once Linus
>> has pulled ARM SoC or after v3.13-rc1 comes out.
> 
> Pls, wait a bit - this fix is incomplete :( The below changes have to be
> done too:

Can either you or Prabhakar send a separate patch (series) with only the
fixes?

Thanks,
Sekhar

  parent reply	other threads:[~2013-11-06 11:42 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-02 15:39 [PATCH v4 0/6] gpio: daVinci: Fixes and feature enhancement Lad, Prabhakar
2013-11-02 15:39 ` Lad, Prabhakar
2013-11-02 15:39 ` [PATCH v4 1/6] gpio: davinci: Fixed a check for unbanked gpio Lad, Prabhakar
2013-11-02 15:39   ` Lad, Prabhakar
2013-11-05 12:39   ` Linus Walleij
2013-11-05 12:39     ` Linus Walleij
2013-11-06  9:33     ` Prabhakar Lad
2013-11-06  9:33       ` Prabhakar Lad
2013-11-06  9:56       ` Linus Walleij
2013-11-06  9:56         ` Linus Walleij
2013-11-06 10:15         ` Prabhakar Lad
2013-11-06 10:15           ` Prabhakar Lad
     [not found]           ` <CA+V-a8tZTm9bh+arQ+uFyTVHxfdf07bWaM_Sdqo08TO=UCMMUg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-11-06 10:49             ` Sekhar Nori
2013-11-06 10:49               ` Sekhar Nori
2013-11-06 10:49               ` Sekhar Nori
     [not found]               ` <527A1EC7.20400-l0cyMroinI0@public.gmane.org>
2013-11-06 11:33                 ` Grygorii Strashko
2013-11-06 11:33                   ` Grygorii Strashko
2013-11-06 11:33                   ` Grygorii Strashko
     [not found]                   ` <527A28EF.9070601-l0cyMroinI0@public.gmane.org>
2013-11-06 11:42                     ` Sekhar Nori [this message]
2013-11-06 11:42                       ` Sekhar Nori
2013-11-06 11:42                       ` Sekhar Nori
2013-11-06 11:36           ` Linus Walleij
2013-11-06 11:36             ` Linus Walleij
2013-11-02 15:39 ` [PATCH v4 2/6] gpio: davinci: remove unnecessary printk Lad, Prabhakar
2013-11-02 15:39   ` Lad, Prabhakar
     [not found] ` <1383406775-14902-1-git-send-email-prabhakar.csenng-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-11-02 15:39   ` [PATCH v4 3/6] gpio: davinci: use irqdomain Lad, Prabhakar
2013-11-02 15:39     ` Lad, Prabhakar
2013-11-02 15:39     ` Lad, Prabhakar
     [not found]     ` <1383406775-14902-4-git-send-email-prabhakar.csenng-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-11-04 18:27       ` Grygorii Strashko
2013-11-04 18:27         ` Grygorii Strashko
2013-11-04 18:27         ` Grygorii Strashko
2013-11-05  8:00         ` Prabhakar Lad
2013-11-05  8:00           ` Prabhakar Lad
2013-11-02 15:39   ` [PATCH v4 5/6] ARM: davinci: da850: add GPIO DT node Lad, Prabhakar
2013-11-02 15:39     ` Lad, Prabhakar
2013-11-02 15:39     ` Lad, Prabhakar
2013-11-02 15:39 ` [PATCH v4 4/6] gpio: davinci: add OF support Lad, Prabhakar
2013-11-02 15:39   ` Lad, Prabhakar
     [not found]   ` <1383406775-14902-5-git-send-email-prabhakar.csenng-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-11-04 18:28     ` Grygorii Strashko
2013-11-04 18:28       ` Grygorii Strashko
2013-11-04 18:28       ` Grygorii Strashko
2013-11-05  8:53       ` Prabhakar Lad
2013-11-05  8:53         ` Prabhakar Lad
     [not found]         ` <CA+V-a8uV0g36LoQtxFjR56SaMxj7KOE2ztC+pfT_Pn1wGcqEmw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-11-05 16:59           ` Grygorii Strashko
2013-11-05 16:59             ` Grygorii Strashko
2013-11-05 16:59             ` Grygorii Strashko
2013-11-06 10:08             ` Prabhakar Lad
2013-11-06 10:08               ` Prabhakar Lad
2013-11-06 10:08               ` Prabhakar Lad
     [not found]               ` <CA+V-a8tJ7yDJOPamoCiaX6Twnnqxp_o_L0rzo2dEeSDS2VoYLw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-11-06 11:55                 ` Grygorii Strashko
2013-11-06 11:55                   ` Grygorii Strashko
2013-11-06 11:55                   ` Grygorii Strashko
2013-11-02 15:39 ` [PATCH v4 6/6] ARM: davinci: da850 evm: add GPIO pinumux entries DT node Lad, Prabhakar
2013-11-02 15:39   ` Lad, Prabhakar

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=527A2B37.4040308@ti.com \
    --to=nsekhar-l0cymroini0@public.gmane.org \
    --cc=alex.elder-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=grygorii.strashko-l0cyMroinI0@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-gpio-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=prabhakar.csengg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org \
    --cc=rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.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.