On Sat, 20 Feb 2016 07:58:31 -0800
<harish.patil at qlogic.com> wrote:

> +unsigned long log2_align(unsigned long n)
> +{

Common code is good, but you need to practice good function name
hygiene on public functions to avoid any namespace clashes when
using static linking.

The application might define a function with log2_align.
Either make the functions static, or use a common prefix for all qede
internal functions.

Reply via email to