lkml.org 
[lkml]   [2016]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] pinctrl: exynos: remove duplicate calls in irq handler


Sent using Zoho Mail


---- On Mon, 15 Aug 2016 09:59:41 -0700 Krzysztof Kozlowski <k.kozlowski@samsung.com> wrote ----
> On Mon, Aug 15, 2016 at 06:17:07PM +0800, perr wrote:
> > Because chained_irq_enter() has already called chip->irq_mask() and
> > chip->irq_ack(), also chained_irq_exit() will call chip->irq_unmask(),
> > so it's not necessary to call chip->irq_*() here.
> >
> > Signed-off-by: perr <strongbox8@zoho.com>
> >
> > ---
> > drivers/pinctrl/samsung/pinctrl-exynos.c | 6 +-----
> > 1 file changed, 1 insertion(+), 5 deletions(-)
> >
>
>
> Looks correct so for the patch itself:
> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>
> However, your name "Perr" confuses me a little bit. For the
> Signed-off-by statement, as mentioned in
> Documentation/SubmittingPatches, a real name is required. The name
> should match the author.
>
> Would you be so kind as to use a full, real name? Unless Perr is your
> full, real name... then sorry for the trouble.
Perr Zhang <strongbox8@zoho.com> is OK?
I'm from China. Perr is my first name in English.
>
> Best regards,
> Krzysztof
>
>

\
 
 \ /
  Last update: 2016-09-17 09:57    [W:0.044 / U:0.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site