On 31/07/2020 20:39, Jakub Kicinski wrote: > On Fri, 31 Jul 2020 13:59:04 +0100 Edward Cree wrote: >> +static inline efx_oword_t *ef100_tx_desc(struct efx_tx_queue *tx_queue, >> + unsigned int index) > Does this static inline make any difference? > > You know the general policy... Damn, I didn't spot that one.
Why doesn't checkpatch catch those? Is it just not smart enough to remember whether it's in a .c file or not? Or do I need to pass it some --strict --fascist --annoy-me-harder flags? Will remove 'inline' in v3, thanks. -ed