IC-R6 Initial Driver

[jbradsha]

Icom IC-R6 Initial version

CHIRP PR Guidelines

The following must be true before PRs can be merged:

1. All tests must be passing. The "PR Checks" job is speculative and failure doesn't always indicate a critial problem, but generally it needs to pass as well.
2. Commits should be rebased (or simply rebase-able in the web UI) on current master. Do not put merge commits in a PR.
3. Commits in a single PR should be related. Squash intermediate commits into logical units (i.e. "fix tests" commits need not survive on their own). Keep cleanup commits separate from functional changes.
4. Major new features or bug fixes should reference a CHIRP issue in the commit message. Do this with the pattern Fixes #1234 or Related to #1234 so that the ticket system links the commit to the issue.
5. Please write a reasonable commit message, especially if making some change that isn't totally obvious (such as adding a new model, adding a feature, etc). The first line of every commit is emailed to the users' list after each build. It should be short, but meaningful for regular users (examples: "thd74: Fixed tone decoding" or "uv5r: Added settings support").
6. New drivers should be accompanied by a test image in tests/images (except for thin aliases where the driver is sufficiently tested already). All new drivers must use MemoryMapBytes. New drivers and radio models will affect the Python3 test matrix. You should regenerate this file with tox -emakesupported and include it in your commit.
7. All files must be GPLv3 licensed or contain no license verbiage. No additional restrictions can be placed on the usage (i.e. such as noncommercial).
8. Do not add new py2-compatibility code (No new uses of six, future, etc).

[kk7ds]

Your commit is titled something that won't make sense for people when it gets sent to the mailing list. Please change that to something like "Add Icom IC-R6 driver" per item 5 above. Also, please include a test image per item 6.

Thanks!

[jbradsha]

Closed - will create a new PR once most of the above is fixed (some queries around endian - radio uses both big and small endian in different places).

[kk7ds]

Please don't open a new one, just re-open this and push to it. That keeps the discussion history in tact and makes it easier to catch back up when you have another version.

Thanks!

[jbradsha]

Thanks Dan. Will upload fresh copy shortly.. just running it through tox again. Changed not all done (the different endians still need looked at. Not sure why Icom even did that, even if they used legacy firmware from another IC-Rx model would have expected it to be consistent) and the warp_bytes for names not working as expected so not added to the PR code.

[helmick43270]

Hello, just dropping in after reading the Service Manual and instruction book. Was toying with the idea of trying my frankensteined Kenwood level converter on my new (to me) IC-R6 and having a hexadecimal conversation with it until I read this. Hope everyone is doing okay. This just seems like a lot of trouble for something not many people are requesting, which is exactly why it’s interesting to me.

[jbradsha]

Hi,

There's a working "test module" attached to https://chirpmyradio.com/issues/2795 . It'd be great to get some feedback, had a spike of interest earlier last year but nobody provided any feedback on the module. Code should be fairly similar to this pull request. Even getting a bit of activity on that feature request ticket would be great, may encourage others to test and feedback.

Most of the device memory is mapped out, including the auto-scan channels (which are the same as the first 1300 channels), I haven't raised a PR for a while so that's only in the local version.

Bit of tidy up needed as the IC-R6 uses different endianness in different places which has complicated the Python code a little. I've a bit more work to do on the GUI for the device settings too.

Regards, John Mi0SYN