On Wed, 14 Nov 2018 20:27:52 +0200 Priit Laes <[email protected]> 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 <[email protected]> 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. -- Michael
pgpYX8DOrAA40.pgp
Description: OpenPGP digital signature
_______________________________________________ b43-dev mailing list [email protected] http://lists.infradead.org/mailman/listinfo/b43-dev
