On Sat, Aug 8, 2020 at 2:22 AM Crt Mori <cmo@xxxxxxxxxxx> wrote: > > TAdut4 was calculated each iteration although it did not change. In light > of near future additions of the Extended range DSP calculations, this > function refactoring will help reduce unrelated changes in that series as > well as reduce the number of new functions needed. I see you provided a helper for this calculus, but I think we may reduce a lot of churn if you do the following: - introduce helper and convert one/current user (no functional change!) - do what you did in this patch, but it will move less LOCs - do what you have done in the next one using the helper -- With Best Regards, Andy Shevchenko
