Re: [PATCH 0/5] Use common cordic algorithm for b43

From: Arend van Spriel
Date: Mon Nov 05 2018 - 03:24:21 EST


On 11/3/2018 10:59 AM, Priit Laes wrote:
b43 wireless driver included internal implementation of cordic
algorithm which has now been removed in favor of library
implementation.

During the process, brcmfmac was driver was also cleaned.

You actually touched the *brcmsmac* driver, not brcmfmac. Please fix the driver prefix where appropriate in this series, ie. patches 2 and 3.

Please note that this series is only compile-tested, as I
do not have access to the hardware.

I can/will verify brcmsmac. As Kalle mentioned it makes more sense to push the 'lib: cordic:' patch through the wireless tree as well as it only is used by wireless drivers right now.

Regards,
Arend