[chirp_devel] New Model #6421 TYT TH-350
Zhaofeng Li
Thu Jan 31 15:51:38 PST 2019
My apologies, but I found a silly error that I made during refactoring
that broke VFO memories. Attached is the fixed changeset.
On Thu, Jan 31, 2019 at 1:40 PM Zhaofeng Li <hello at zhaofeng.li> wrote:
>
> Hi Dan,
>
> That sounds reasonable to me. I've made a couple of changes to the
> UV-B5 driver to make overriding easier, and the revised patchset is
> attached to this email.
>
> Thanks,
> Zhaofeng Li
>
> On Thu, Jan 31, 2019 at 11:02 AM Dan Smith via chirp_devel
> <chirp_devel at intrepid.danplanet.com> wrote:
> >
> > > Hi Zhaofeng,
> > >
> > >> Just wrote a driver for TYT TH-350, based on the code for Baofeng
> > >> UV-B5. Attached is the patch.
> > >
> > > How similar is this driver to the UV-B5 itself? In general it's much better to subclass a driver if you have just a few changes. I haven't tried applying this and diff'ing them myself, but wanted to ask you first.
> >
> > Heh, right after I sent, I re-read your comment that there are more details in the issue. It sounds like there are some differences, but are they just around tone encoding and the clone protocol? If so, those are also easy to subclass and override. If many things are different, then it might make the most sense to just add a whole new driver. I just want to avoid senseless duplication _when_possible_.
> >
> > Thanks!
> >
> > --Dan
> > _______________________________________________
> > chirp_devel mailing list
> > chirp_devel at intrepid.danplanet.com
> > http://intrepid.danplanet.com/mailman/listinfo/chirp_devel
> > Developer docs: http://chirp.danplanet.com/projects/chirp/wiki/Developers
-------------- next part --------------
A non-text attachment was scrubbed...
Name: th350.patch
Type: text/x-patch
Size: 13620 bytes
Desc: not available
Url : http://intrepid.danplanet.com/pipermail/chirp_devel/attachments/20190131/6755e68b/attachment-0001.bin
More information about the chirp_devel
mailing list