[chirp_devel] IC2730 Issue #7031 patch
Patch for issue #7031 to fix corrupted bank index values after download from IC-2730A. Possible cause of the bad values is use of an audio extension cable with the standard USB cable.
Patch for issue #7031 to fix corrupted bank index values after download from IC-2730A. Possible cause of the bad values is use of an audio extension cable with the standard USB cable.
If this is the case, then anything could be corrupted, so spot fixes like this are ill-fated. It looks to me like we're not actually checking the checksum on received data frames on the icom base driver. We should do that and abort the process if one fails instead of silently ingesting the bad data and trying to sanitize random places.
Any interest in trying to address that?
--Dan
OK, I will try to drill down through the icf.py file and see what it does.
Rick DeWitt AA0RD Sequim, Washington, USA 360-681-3494
On 9/4/2019 11:28 AM, Dan Smith via chirp_devel wrote:
Patch for issue #7031 to fix corrupted bank index values after download from IC-2730A. Possible cause of the bad values is use of an audio extension cable with the standard USB cable.
If this is the case, then anything could be corrupted, so spot fixes like this are ill-fated. It looks to me like we're not actually checking the checksum on received data frames on the icom base driver. We should do that and abort the process if one fails instead of silently ingesting the bad data and trying to sanitize random places.
Any interest in trying to address that?
--Dan _______________________________________________ chirp_devel mailing list chirp_devel@intrepid.danplanet.com http://intrepid.danplanet.com/mailman/listinfo/chirp_devel Developer docs: http://chirp.danplanet.com/projects/chirp/wiki/Developers
participants (3)
-
Dan Smith -
Rick DeWitt -
Rick DeWitt AA0RD