[chirp_devel] Re: Patch - [FT-60] Feature Request: Memory Bank Support #1945

[stickpatrick at gmail.com]

I'm looking at Kosta's now.  I took the same approach as the FT857 and lifted a lot of that code including the cache making it a bit harder to read.  I’ll see if there's a perf difference or not.


Dan - do you have any intention of using chirp_common.MTOBankModel in the future?  Right now it's equivalent to chirp_common.BankModel but seems to be a better description of what's in the FT60 since any memory can be stored in 1 or more banks.





Sent from Windows Mail





From: Dan Smith
Sent: ‎Saturday‎, ‎January‎ ‎10‎, ‎2015 ‎3‎:‎43‎ ‎PM
To: chirp_devel at intrepid.danplanet.com





> Can you guys look at each other's patches and decide which should be
> applied?

I applied Kosta's patch and it's failing one of the bank tests (see the
test report from a few minutes ago).

So, I'll revert it and let you guys work out which one should go in.
Note that you should be able to run the tests locally and verify that
the bank model(s) meet the requirements first.

Thanks!

--Dan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20150110/0de0ab54/attachment-0001.html