* [PATCH] irqchip/irq-csky-mpintc: Fixup mask/unmask un-implementation
@ 2021-10-25 9:12 guoren
2021-10-25 10:34 ` Marc Zyngier
0 siblings, 1 reply; 3+ messages in thread
From: guoren @ 2021-10-25 9:12 UTC (permalink / raw)
To: guoren, maz, tglx; +Cc: linux-kernel, linux-csky, Guo Ren, Marc Zyngier
From: Guo Ren <guoren@linux.alibaba.com>
The mask/unmask must be implemented, and enable/disable supplement
them if the HW requires something different at startup time. When
irq source is disabled by mask, mpintc could complete irq normally.
So just replace the with mask/unmask function.
Tested-by: Guo Ren <guoren@linux.alibaba.com>
Signed-off-by: Guo Ren <guoren@linux.alibaba.com>
Cc: Marc Zyngier <marc.zyngier@arm.com>
---
drivers/irqchip/irq-csky-mpintc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-csky-mpintc.c b/drivers/irqchip/irq-csky-mpintc.c
index a1534edef7fa..f169600dbde6 100644
--- a/drivers/irqchip/irq-csky-mpintc.c
+++ b/drivers/irqchip/irq-csky-mpintc.c
@@ -164,8 +164,8 @@ static int csky_irq_set_affinity(struct irq_data *d,
static struct irq_chip csky_irq_chip = {
.name = "C-SKY SMP Intc",
.irq_eoi = csky_mpintc_eoi,
- .irq_enable = csky_mpintc_enable,
- .irq_disable = csky_mpintc_disable,
+ .irq_unmask = csky_mpintc_enable,
+ .irq_mask = csky_mpintc_disable,
.irq_set_type = csky_mpintc_set_type,
#ifdef CONFIG_SMP
.irq_set_affinity = csky_irq_set_affinity,
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] irqchip/irq-csky-mpintc: Fixup mask/unmask un-implementation
2021-10-25 9:12 [PATCH] irqchip/irq-csky-mpintc: Fixup mask/unmask un-implementation guoren
@ 2021-10-25 10:34 ` Marc Zyngier
2021-10-25 13:14 ` Guo Ren
0 siblings, 1 reply; 3+ messages in thread
From: Marc Zyngier @ 2021-10-25 10:34 UTC (permalink / raw)
To: guoren; +Cc: tglx, linux-kernel, linux-csky, Guo Ren, Marc Zyngier
On Mon, 25 Oct 2021 10:12:19 +0100,
guoren@kernel.org wrote:
>
> From: Guo Ren <guoren@linux.alibaba.com>
>
> The mask/unmask must be implemented, and enable/disable supplement
> them if the HW requires something different at startup time. When
> irq source is disabled by mask, mpintc could complete irq normally.
>
> So just replace the with mask/unmask function.
>
> Tested-by: Guo Ren <guoren@linux.alibaba.com>
This only makes sense if tested by a third party. It is assumed that
the author of a patch has tested it.
> Signed-off-by: Guo Ren <guoren@linux.alibaba.com>
> Cc: Marc Zyngier <marc.zyngier@arm.com>
This hasn't been my email address for over two years now. I'm sure my
ex manager is happy to hear from you, but I doubt he'll bother replying.
> ---
> drivers/irqchip/irq-csky-mpintc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/irqchip/irq-csky-mpintc.c b/drivers/irqchip/irq-csky-mpintc.c
> index a1534edef7fa..f169600dbde6 100644
> --- a/drivers/irqchip/irq-csky-mpintc.c
> +++ b/drivers/irqchip/irq-csky-mpintc.c
> @@ -164,8 +164,8 @@ static int csky_irq_set_affinity(struct irq_data *d,
> static struct irq_chip csky_irq_chip = {
> .name = "C-SKY SMP Intc",
> .irq_eoi = csky_mpintc_eoi,
> - .irq_enable = csky_mpintc_enable,
> - .irq_disable = csky_mpintc_disable,
> + .irq_unmask = csky_mpintc_enable,
> + .irq_mask = csky_mpintc_disable,
> .irq_set_type = csky_mpintc_set_type,
> #ifdef CONFIG_SMP
> .irq_set_affinity = csky_irq_set_affinity,
Please rename the functions to match the fields they are assigned to.
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] irqchip/irq-csky-mpintc: Fixup mask/unmask un-implementation
2021-10-25 10:34 ` Marc Zyngier
@ 2021-10-25 13:14 ` Guo Ren
0 siblings, 0 replies; 3+ messages in thread
From: Guo Ren @ 2021-10-25 13:14 UTC (permalink / raw)
To: Marc Zyngier
Cc: Thomas Gleixner, Linux Kernel Mailing List, linux-csky, Guo Ren,
Marc Zyngier
On Mon, Oct 25, 2021 at 6:34 PM Marc Zyngier <maz@kernel.org> wrote:
>
> On Mon, 25 Oct 2021 10:12:19 +0100,
> guoren@kernel.org wrote:
> >
> > From: Guo Ren <guoren@linux.alibaba.com>
> >
> > The mask/unmask must be implemented, and enable/disable supplement
> > them if the HW requires something different at startup time. When
> > irq source is disabled by mask, mpintc could complete irq normally.
> >
> > So just replace the with mask/unmask function.
> >
> > Tested-by: Guo Ren <guoren@linux.alibaba.com>
>
> This only makes sense if tested by a third party. It is assumed that
> the author of a patch has tested it.
Thx for pointing it out.
>
> > Signed-off-by: Guo Ren <guoren@linux.alibaba.com>
> > Cc: Marc Zyngier <marc.zyngier@arm.com>
>
> This hasn't been my email address for over two years now. I'm sure my
> ex manager is happy to hear from you, but I doubt he'll bother replying.
I'll fix up your email in the next patch, thx for still reviewing.
>
> > ---
> > drivers/irqchip/irq-csky-mpintc.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/irqchip/irq-csky-mpintc.c b/drivers/irqchip/irq-csky-mpintc.c
> > index a1534edef7fa..f169600dbde6 100644
> > --- a/drivers/irqchip/irq-csky-mpintc.c
> > +++ b/drivers/irqchip/irq-csky-mpintc.c
> > @@ -164,8 +164,8 @@ static int csky_irq_set_affinity(struct irq_data *d,
> > static struct irq_chip csky_irq_chip = {
> > .name = "C-SKY SMP Intc",
> > .irq_eoi = csky_mpintc_eoi,
> > - .irq_enable = csky_mpintc_enable,
> > - .irq_disable = csky_mpintc_disable,
> > + .irq_unmask = csky_mpintc_enable,
> > + .irq_mask = csky_mpintc_disable,
> > .irq_set_type = csky_mpintc_set_type,
> > #ifdef CONFIG_SMP
> > .irq_set_affinity = csky_irq_set_affinity,
>
> Please rename the functions to match the fields they are assigned to.
okay
>
> Thanks,
>
> M.
>
> --
> Without deviation from the norm, progress is not possible.
--
Best Regards
Guo Ren
ML: https://lore.kernel.org/linux-csky/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-10-25 13:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-25 9:12 [PATCH] irqchip/irq-csky-mpintc: Fixup mask/unmask un-implementation guoren
2021-10-25 10:34 ` Marc Zyngier
2021-10-25 13:14 ` Guo Ren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).