[chirp_devel] Patch to add support for the Intek HR-2040 dual band radio
Ron Wellsted
Fri Sep 20 03:40:19 PDT 2013
On 20/09/13 00:02, Dan Smith wrote:
>> +# This SHOULD be a different model number but at present it returns the same as the AnyTone
>> +# expect this to change in future firmware releases.
>> + @classmethod
>> + def match_model(cls, filedata, filename):
>> + return filedata[0x21:0x28] == "QX588UV"
>> +# return filedata[0x21:0x28] == "HR-2040"
>
> This makes it intentionally mis-detect files, instead of letting the
> other one win. Doesn't this fail the detect test? I would think it should.
>
> Can you make this just return False so that it never matches anything
> (and thus the Anytone driver will always win)? Since this is
> functionally no different (especially from the testing-an-image
> perspective) I don't think we need this.
>
> Otherwise, I think this looks good.
>
> Thanks!
Hi,
I have improved the detection so that it will correctly determine a .img
file from an AnyTone or a Intek. A modified patch is attached.
--
Ron Wellsted
ron at wellsted.org.uk http://www.wellsted.org.uk
Call Sign: M0RNW, Linux Counter No. 202120
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Intek_HR-2040.patch
Type: text/x-patch
Size: 2704 bytes
Desc: not available
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20130920/05d7e37a/attachment-0002.bin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 263 bytes
Desc: OpenPGP digital signature
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20130920/05d7e37a/attachment-0003.bin
More information about the chirp_devel
mailing list