[PATCH 2/4] irqchip/ast2700-intcx: Add AST2700 INTC0/INTC1 support

[Ryan Chen]

> Subject: RE: [PATCH 2/4] irqchip/ast2700-intcx: Add AST2700 INTC0/INTC1
> support
> 
> On Sat, Feb 07 2026 at 03:50, Ryan Chen wrote:
> >> -----Original Message-----
> >> From: Thomas Gleixner <tglx at linutronix.de>
> >> Sent: Friday, February 6, 2026 7:34 PM
> >> To: Ryan Chen <ryan_chen at aspeedtech.com>; Rob Herring
> >> <robh at kernel.org>; Krzysztof Kozlowski <krzk+dt at kernel.org>; Conor
> >> Dooley <conor+dt at kernel.org>; Joel Stanley <joel at jms.id.au>; Andrew
> >> Jeffery <andrew at codeconstruct.com.au>; Paul Walmsley
> >> <pjw at kernel.org>; Palmer Dabbelt <palmer at dabbelt.com>; Albert Ou
> >> <aou at eecs.berkeley.edu>; Alexandre Ghiti <alex at ghiti.fr>
> >> Cc: linux-kernel at vger.kernel.org; devicetree at vger.kernel.org;
> >> linux-arm-kernel at lists.infradead.org; linux-aspeed at lists.ozlabs.org;
> >> linux-riscv at lists.infradead.org; Ryan Chen <ryan_chen at aspeedtech.com>
> >> Subject: Re: [PATCH 2/4] irqchip/ast2700-intcx: Add AST2700
> >> INTC0/INTC1 support
> 
> Please fix your email client to not copy all mail headers. If you can't fix it,
> remove them manually and replace it with the usual:

Sorry, This is my mistake. 
> 
> On Fri, Feb 06 2026 at ...., Thomas Gleixner wrote:
> >> > +	guard(raw_spinlock_irqsave)(&intc0->intc_lock);
> >>
> >> s/_irqsave// Interrupts are disabled when this is invoked.
> > Sorry, do you mean when this function been call.
> > Ther have desc->lock to do the raw_spin_lock_irqsave.
> 
> The core always holds irq_desc::lock with interrupts disabled when it invokes
> those irqchip callbacks. So trying to disable interrupts in them again is a
> pointless exercise.
> 
Thanks your gyudabce , understood.