[PATCH] asm-generic: Add msi.h

Thomas Gleixner tglx at linutronix.de
Tue Nov 18 02:59:39 PST 2014


On Tue, 18 Nov 2014, Arnd Bergmann wrote:

> On Tuesday 18 November 2014 11:34:37 Thomas Gleixner wrote:
> > To support MSI irq domains we want a generic data structure for
> > allocation, but we need the option to provide an architecture specific
> > version of it. So instead of playing #ifdef games in linux/msi.h we
> > add a generic header file and let architectures decide whether to
> > include it or to provide their own implementation and provide the
> > required typedef.
> 
> Acked-by: Arnd Bergmann <arnd at arndb.de>
> 
> for merging the asm-generic file
> 
> > I know that typedefs are not really nice, but in this case there are no
> > forward declarations required and it's the simplest solution.
> 
> I must be missing the obvious: what problem does the typedef solve
> that you would have with just a struct?

It's not obvious. :)

The irqdomain stuff is pretty device tree centric, but the new stacked
irqdomains are to be used by x86 as well. So we made some of the
interfaces opaque, i.e. void *allocation_arg.

Now MSI is a bit differnet as it cannot be decribed by DT, so we want
a proper generic data structure for it and of course we want to have a
type for it.

Now x86 has a bit more convoluted requirements where we prefer for
simplicity reasons to reuse the allocation data structure which we
have alredy for the non MSI cases, so it can be handed down to the
opaque interfaces as well.

So we have the generic:
 
struct msi_alloc_info {
       ....
};

and 

struct x86_alloc_info {
       ...
};

So we either can do in x86:

struct msi_alloc_info {
       struct x86_alloc_info info;
};

or use a typedef which maps x86_alloc_info to msi_alloc_info_t.

I think the typedef is more sane in that case.

Thanks,

	tglx







More information about the linux-arm-kernel mailing list