Messages in this thread | | | Date | Thu, 24 Oct 2013 13:41:33 +0200 | From | Alexander Gordeev <> | Subject | Re: [PATCH RFC v2 12/29] PCI/MSI: Introduce pcim_enable_msi*() family helpers |
| |
On Thu, Oct 24, 2013 at 11:57:40AM +0100, David Laight wrote: > The one case it doesn't work is where the driver either > wants the full number or the minimum number - but not > a value in between. > > Might be worth adding an extra parameter so that this > (and maybe other) odd requirements can be met.
IMHO its not worth it, since it is not possible to generalize all odd requirements out there. I do not think we should blow the API in this case.
Having said that, the min-or-max interface is probably the only worth considering. But again, I would prefer to put its semantics to function name rather than to extra parameters, i.e.
pcim_enable_msix_min_max(struct pci_dev *dev, struct msix_entry *entries, unsigned int minvec, unsigned int maxvec);
> Some static inline functions could be used for the common cases. > > David
-- Regards, Alexander Gordeev agordeev@redhat.com
| |