[chirp_devel] Patch to add support for the Intek HR-2040 dual band radio
Dan Smith
Thu Sep 19 16:02:02 PDT 2013
> +# 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!
--
Dan Smith
www.danplanet.com
KK7DS
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: OpenPGP digital signature
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20130919/c8b4c750/attachment-0001.bin
More information about the chirp_devel
mailing list