[lkml]   [2018]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v3 3/3] b43: Use cordic algorithm from kernel library
On Wed, 14 Nov 2018 20:27:52 +0200
Priit Laes <> wrote:

> Kernel library has a common cordic algorithm which is identical
> to internally implementatd one, so use it and drop the duplicate
> implementation.

In v2 of the series it has been said that:

Arend van Spriel <> wrote:
> I recall doing a comparison between the algorithms and thought I put
> that in the original commit message. However, it is not there. It is not
> exactly the same as in b43 so there are difference for certain angles,
> most results are the same however. This implementation is slightly more
> accurate on the full scale.

That's not my definition of "identical".

Please do not apply this patch without doing a thorough regression test
on actual b43 LP hardware.

[unhandled content-type:application/pgp-signature]
 \ /
  Last update: 2018-11-14 19:48    [W:0.114 / U:3.516 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site