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

2018-11-05 Thread Kalle Valo
Arend van Spriel writes: > On 11/5/2018 9:02 AM, Kalle Valo wrote: >> Also I don't see MAINTAINERS entry for cordic.[c|h], that would be good >> to have as well. > > We added the cordic library functions during brcm80211 staging > cleanup. We can add it to MAINTAINERS file. Great, thanks. -- K

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

2018-11-05 Thread Arend van Spriel
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. P

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

2018-11-05 Thread Arend van Spriel
On 11/5/2018 9:02 AM, Kalle Valo wrote: Also I don't see MAINTAINERS entry for cordic.[c|h], that would be good to have as well. We added the cordic library functions during brcm80211 staging cleanup. We can add it to MAINTAINERS file. Regards, Arend

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

2018-11-05 Thread Kalle Valo
Kalle Valo writes: > Priit Laes writes: > >> 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. >> >> Please note that this series is only co

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

2018-11-05 Thread Kalle Valo
Priit Laes writes: > 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. > > Please note that this series is only compile-tested, as I > do not hav

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

2018-11-03 Thread Priit Laes
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. Please note that this series is only compile-tested, as I do not have access to the hardware. Priit La