On 23.10.2023 09:23, Federico Serafini wrote: > Add missing parameter names. No functional change. > > Signed-off-by: Federico Serafini <[email protected]>
Acked-by: Jan Beulich <[email protected]> Albeit I can't resist to say that ... > --- a/xen/arch/x86/physdev.c > +++ b/xen/arch/x86/physdev.c > @@ -18,9 +18,9 @@ > #include <xsm/xsm.h> > #include <asm/p2m.h> > > -int physdev_map_pirq(domid_t, int type, int *index, int *pirq_p, > - struct msi_info *); > -int physdev_unmap_pirq(domid_t, int pirq); > +int physdev_map_pirq(domid_t domid, int type, int *index, int *pirq_p, > + struct msi_info *msi); > +int physdev_unmap_pirq(domid_t domid, int pirq); ... this is a good example where adding names to the so far unnamed parameters adds no new useful information at all, as their types were already sufficient to express all that needs expressing here. Jan
