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


Groups > linux.kernel > #1600561 > unrolled thread

Re: [PATCH] pinctrl: rockchip: add irq_enable & irq_disable ops

Started byHeiko Stübner <heiko@sntech.de>
First post2017-03-14 16:10 +0100
Last post2017-03-14 16:10 +0100
Articles 1 — 1 participant

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] pinctrl: rockchip: add irq_enable & irq_disable ops Heiko Stübner <heiko@sntech.de> - 2017-03-14 16:10 +0100

#1600561 — Re: [PATCH] pinctrl: rockchip: add irq_enable & irq_disable ops

FromHeiko Stübner <heiko@sntech.de>
Date2017-03-14 16:10 +0100
SubjectRe: [PATCH] pinctrl: rockchip: add irq_enable & irq_disable ops
Message-ID<tkWfh-1T2-61@gated-at.bofh.it>
Hi Linus,

Am Dienstag, 14. März 2017, 16:03:48 CET schrieb Linus Walleij:
> On Thu, Mar 2, 2017 at 6:56 AM, Jeffy Chen <jeffy.chen@rock-chips.com> 
wrote:
> > Currently we are trying to enable/disable the clk of irq's gpio bank when
> > unmask/mask irq. But the kernel's "lazy disable approach" will skip
> > masking
> > irq when the irq chip doesn't support irq_disable ops.
> > 
> > So we may hit this case:
> >         irq_enable-> enable clk
> >         irq_disable-> noop
> >         irq_enable-> enable clk again
> >         irq_disable-> noop
> > 
> > Signed-off-by: Jeffy Chen <jeffy.chen@rock-chips.com>
> 
> Heiko, can you look at this patch?

This (and all the other pending pinctrl patches) are in my queue, but right 
now I'm still hunting rockchip regressions in 4.11-rc, so haven't found the 
time to review yet.


Heiko

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web