Message ID | 20191115201037.44982-2-stewart.hildebrand@dornerworks.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | xen: arm: context switch vtimer PPI state | expand |
Hi, On 15/11/2019 20:10, Stewart Hildebrand wrote: > These will be used in a follow-up patch. > > Signed-off-by: Stewart Hildebrand <stewart.hildebrand@dornerworks.com> > --- > v3: new patch > --- > xen/include/asm-arm/irq.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/xen/include/asm-arm/irq.h b/xen/include/asm-arm/irq.h > index 3b37a21c06..367fe6269c 100644 > --- a/xen/include/asm-arm/irq.h > +++ b/xen/include/asm-arm/irq.h > @@ -33,7 +33,9 @@ struct arch_irq_desc { > unsigned int type; > }; > > -#define NR_LOCAL_IRQS 32 > +#define NR_SGIS 16 > +#define NR_PPIS 16 > +#define NR_LOCAL_IRQS (NR_SGIS + NR_PPIS) We have already NR_GIC_SGI (see include/asm-arm/gic.h) and VGIC_NR_SGIS (see include/asm-arm/new_vgic.h). So I would rather not want to define the same value (with the same meaning) a third time. Note that I am ok if the two existing one are dropped in favor of NR_SGIS. Cheers,
diff --git a/xen/include/asm-arm/irq.h b/xen/include/asm-arm/irq.h index 3b37a21c06..367fe6269c 100644 --- a/xen/include/asm-arm/irq.h +++ b/xen/include/asm-arm/irq.h @@ -33,7 +33,9 @@ struct arch_irq_desc { unsigned int type; }; -#define NR_LOCAL_IRQS 32 +#define NR_SGIS 16 +#define NR_PPIS 16 +#define NR_LOCAL_IRQS (NR_SGIS + NR_PPIS) /* * This only covers the interrupts that Xen cares about, so SGIs, PPIs and
These will be used in a follow-up patch. Signed-off-by: Stewart Hildebrand <stewart.hildebrand@dornerworks.com> --- v3: new patch --- xen/include/asm-arm/irq.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)