Hi Jim,

Thank you for the feedback. If you're able to test this against your own KG-UV920P-A, that would be great.

Unless I hear otherwise, I'll create a new patch with those two corrections when I have a chance this evening. Would the convention be to share it in this same email thread, or start a new one?

Thanks,
Matt
KF7TAL

On Mon, Feb 14, 2022 at 7:59 AM Jim Unroe <rock.unroe@gmail.com> wrote:
Matthew and Chirp-Devel,

Nice! Thank you for developing this driver. I am going to try to find
time today to dig my Wouxun KG-UV920P-A out of mothballs and give this
a go. It looks great!

Below are a couple of things that you might have to address before it
gets accepted (unless Dan takes care of it for you):

1. I think it was requested that the commit message be broken up into
lines of less than 80 characters. I could be wrong but something got
me into the habit of doing it. ;-)

2. I don't see where the driver was added to the cpep8.manifest. It
needs to be listed in the manifest so that the build system will know
to perform a pep8 style check on it during the build process. The good
news is that your driver passes the style check with flying colors.

Thank you again for your work developing this driver.

Jim KC9HI



On Sun, Feb 13, 2022 at 9:46 PM Matthew Handley via chirp_devel
<chirp_devel@intrepid.danplanet.com> wrote:
>
>
> _______________________________________________
> 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