On 2011-12-10 16:54, Blue Swirl wrote: > On Sat, Dec 10, 2011 at 15:51, Jan Kiszka <jan.kiszka@xxxxxx> wrote: >> On 2011-12-10 16:49, Blue Swirl wrote: >>>> >>>> +ISADevice *pit_init(int base, qemu_irq irq) >>> >>> Please retain this function in pc.h, or even better, introduce i8254.h. >> >> No concerns about i8254.h, but this function does not qualify for static >> inline. > > The function is static inline in a header file not for performance > reasons, but to keep the instantiation separate from device internals. Not performance, footprint and header dependencies. You need to pull in all the stuff the inline function needs for everyone including the header that contains this function. That's messy. Even if the instantiation helper should not poke into the device model internals (and I don't want this to change as well), it belongs to the module that implements the device. We do the same with other fabric functions. Jan
Attachment:
signature.asc
Description: OpenPGP digital signature