On 2014/11/14 4:23, Marc Zyngier wrote: > On 13/11/14 11:43, Jiang Liu wrote: >> Introduce msi_domain_{alloc|free}_irqs() to alloc/free interrupts >> from generic MSI irqdomain. >> >> Signed-off-by: Jiang Liu <jiang.liu@xxxxxxxxxxxxxxx> >> --- >> include/linux/msi.h | 35 ++++++++++++++++++++++++++++++ >> kernel/irq/msi.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++ >> 2 files changed, 94 insertions(+) >> >> +/* >> + * Default structure for MSI interrupt allocation. >> + * Arch may overwrite it by define msi_alloc_info_t. >> + */ >> +struct msi_alloc_info { >> + struct msi_desc *desc; >> + irq_hw_number_t hwirq; >> + long scratchpad[NUM_MSI_ALLOC_SCRATCHPAD_REGS]; > > As much as I'm relieved to see a generic structure here, could > scratchpad be slightly less awkward to use? Something like: > > struct msi_alloc_info { > [...] > union { > unsigned long ul; > void *ptr; > } scratchpad[NUM_MSI_ALLOC_SCRATCHPAD_REGS]; > }; > > That would avoid some very ugly casting. Great, will changed to this way in next version:) > > Thanks, > > M. > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html