Re: [PATCH v2 1/3] lib: cordic: Move cordic macros and defines to header file

[Arend van Spriel]

On 11/5/2018 8:37 PM, Priit Laes wrote:
> Also append CORDIC_ prefix to nonprefixed macros.

Bit annoying that you made me look at LKML for this patch. Can you just 
post the entire series to linux-wireless so Kalle can take it all 
through his tree. Also the commit message is a bit thin. Please explain 
why these are moved to the header file.

Reviewed-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx>
> Signed-off-by: Priit Laes <plaes@xxxxxxxxx>
> ---
>  include/linux/cordic.h |  9 +++++++++
>  lib/cordic.c           | 23 +++++++----------------
>  2 files changed, 16 insertions(+), 16 deletions(-)