On Thu, Apr 25, 2019 at 12:46:52PM +0530, Anup Patel wrote: > On Thu, Apr 25, 2019 at 11:28 AM Christoph Hellwig <h...@infradead.org> wrote: > > > > > +#ifdef CONFIG_64BIT > > > +#define SCAUSE_IRQ_FLAG _AC(0x8000000000000000, UL) > > > +#else > > > +#define SCAUSE_IRQ_FLAG _AC(0x80000000, UL) > > > +#endif > > > > Please keep the existing defintion that doesn't need an ifdef.. > > Sure, will do.
While we're at it: I don't think we need both INTERRUPT_CAUSE_FLAG and SCAUSE_IRQ_FLAG.
