Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1220812 > unrolled thread

Re: [patch 15/22] pinctrl: Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [patch 15/22] pinctrl: Use irq_desc_get_xxx() to avoid redundant  lookup of irq_desc Linus Walleij <linus.walleij@linaro.org> - 2015-09-08 16:10 +0200
    Re: [patch 15/22] pinctrl: Use irq_desc_get_xxx() to avoid redundant  lookup of irq_desc Thomas Gleixner <tglx@linutronix.de> - 2015-09-08 16:20 +0200
      Re: [patch 15/22] pinctrl: Use irq_desc_get_xxx() to avoid redundant  lookup of irq_desc Linus Walleij <linus.walleij@linaro.org> - 2015-09-08 17:00 +0200

#1220812 — Re: [patch 15/22] pinctrl: Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc

On Mon, Aug 31, 2015 at 11:28 PM, Paul Gortmaker
<paul.gortmaker@windriver.com> wrote:
> On Mon, Jul 13, 2015 at 4:47 PM, Thomas Gleixner <tglx@linutronix.de> wrote:
>> From: Jiang Liu <jiang.liu@linux.intel.com>
>>
>> Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc while we
>> already have a pointer to corresponding irq_desc.
>
> Hi Linus,
>
> It seems that linux-next has this copy of the patch as 5663bb27dec1, which
> has a stray irq_desc_get_irq_chip in s3c24xx ; (2nd instance of "irq" is
> bogus looking.)   I first assumed it was from tglx since it did not have any
> additional SOB lines, but on further investigation I traced it back to sfr's
> merge of pinctrl/for-next.

Hm, saw this a bit late I guess.

Torvalds has merged my trees, is there a problem in mainline
that need to be addressed or did this fix itself?

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1220829

On Tue, 8 Sep 2015, Linus Walleij wrote:
> On Mon, Aug 31, 2015 at 11:28 PM, Paul Gortmaker
> <paul.gortmaker@windriver.com> wrote:
> > On Mon, Jul 13, 2015 at 4:47 PM, Thomas Gleixner <tglx@linutronix.de> wrote:
> >> From: Jiang Liu <jiang.liu@linux.intel.com>
> >>
> >> Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc while we
> >> already have a pointer to corresponding irq_desc.
> >
> > Hi Linus,
> >
> > It seems that linux-next has this copy of the patch as 5663bb27dec1, which
> > has a stray irq_desc_get_irq_chip in s3c24xx ; (2nd instance of "irq" is
> > bogus looking.)   I first assumed it was from tglx since it did not have any
> > additional SOB lines, but on further investigation I traced it back to sfr's
> > merge of pinctrl/for-next.
> 
> Hm, saw this a bit late I guess.
> 
> Torvalds has merged my trees, is there a problem in mainline
> that need to be addressed or did this fix itself?

No, it's buggered in Linus tree as well.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1220907

On Tue, Sep 8, 2015 at 4:10 PM, Thomas Gleixner <tglx@linutronix.de> wrote:
> On Tue, 8 Sep 2015, Linus Walleij wrote:
>> On Mon, Aug 31, 2015 at 11:28 PM, Paul Gortmaker
>> <paul.gortmaker@windriver.com> wrote:
>> > On Mon, Jul 13, 2015 at 4:47 PM, Thomas Gleixner <tglx@linutronix.de> wrote:
>> >> From: Jiang Liu <jiang.liu@linux.intel.com>
>> >>
>> >> Use irq_desc_get_xxx() to avoid redundant lookup of irq_desc while we
>> >> already have a pointer to corresponding irq_desc.
>> >
>> > Hi Linus,
>> >
>> > It seems that linux-next has this copy of the patch as 5663bb27dec1, which
>> > has a stray irq_desc_get_irq_chip in s3c24xx ; (2nd instance of "irq" is
>> > bogus looking.)   I first assumed it was from tglx since it did not have any
>> > additional SOB lines, but on further investigation I traced it back to sfr's
>> > merge of pinctrl/for-next.
>>
>> Hm, saw this a bit late I guess.
>>
>> Torvalds has merged my trees, is there a problem in mainline
>> that need to be addressed or did this fix itself?
>
> No, it's buggered in Linus tree as well.

OK fixing it.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web