Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IC-706 set_mode failing #490

Closed
mdblack98 opened this issue Jan 6, 2021 · 3 comments
Closed

IC-706 set_mode failing #490

mdblack98 opened this issue Jan 6, 2021 · 3 comments
Labels
bug critical A problem for common operations with WSJT-X, GPredict, RigPi, etc. WSJTX Bugs affecting WSTJ-X operations

Comments

@mdblack98
Copy link
Contributor

[2021-01-06 21:41:53.595749][00:03:06.852069][RIGCTRL:trace] rig_set_mode mode=USB
[2021-01-06 21:41:53.595749][00:03:06.852081][RIGCTRL:debug] rig_set_mode called, vfo=currVFO, mode=USB, width=-1
[2021-01-06 21:41:53.595749][00:03:06.852090][RIGCTRL:debug] icom_set_mode called vfo=currVFO, mode=USB, width=-1
[2021-01-06 21:41:53.595749][00:03:06.852098][RIGCTRL:debug] rig_get_mode called
[2021-01-06 21:41:53.595749][00:03:06.852106][RIGCTRL:trace] elapsed_ms: start = 1609969313,493749700
[2021-01-06 21:41:53.595749][00:03:06.852115][RIGCTRL:trace] elapsed_ms: elapsed_msecs=102
[2021-01-06 21:41:53.595749][00:03:06.852124][RIGCTRL:trace] rig_get_mode: cache check age=102ms
[2021-01-06 21:41:53.595749][00:03:06.852132][RIGCTRL:trace] rig_get_mode: cache hit age=102ms
[2021-01-06 21:41:53.595749][00:03:06.852140][RIGCTRL:debug] icom_set_mode: icmode=1, icmode_ext=3
[2021-01-06 21:41:53.595749][00:03:06.852147][RIGCTRL:debug] icom_set_mode: #2 icmode=1, icmode_ext=3
[2021-01-06 21:41:53.595749][00:03:06.852160][RIGCTRL:debug] icom_transaction: cmd=0x06, subcmd=0x01, payload_len=1, data_len=80
[2021-01-06 21:41:53.595749][00:03:06.852168][RIGCTRL:trace] rig_flush: called for serial device
[2021-01-06 21:41:53.595749][00:03:06.852175][RIGCTRL:debug] serial_flush called
[2021-01-06 21:41:53.595749][00:03:06.852182][RIGCTRL:debug] tcflush
[2021-01-06 21:41:53.595749][00:03:06.852210][RIGCTRL:debug] write_block called
[2021-01-06 21:41:53.596749][00:03:06.852708][RIGCTRL:trace] write_block(): TX 8 bytes
[2021-01-06
21:41:53.596749][00:03:06.852730][RIGCTRL:trace] 0000 fe fe 4e e0 06
01 03 fd ..N.....
[2021-01-06 21:41:53.596749][00:03:06.852747][RIGCTRL:trace] read_string called, rxmax=80
[2021-01-06 21:41:53.616749][00:03:06.872809][RIGCTRL:trace] read_string(): RX 8 characters
[2021-01-06
21:41:53.616749][00:03:06.872852][RIGCTRL:trace] 0000 fe fe 4e e0 06
01 03 fd ..N.....
[2021-01-06 21:41:53.616749][00:03:06.872865][RIGCTRL:trace] read_string called, rxmax=80
[2021-01-06 21:41:53.636749][00:03:06.892892][RIGCTRL:trace] read_string(): RX 6 characters
[2021-01-06 21:41:53.636749][00:03:06.892939][RIGCTRL:trace] 0000 fe fe e0 4e fa fd ...N..
[2021-01-06 21:41:53.636749][00:03:06.892965][RIGCTRL:debug] icom_transaction: failed: Command rejected by the rig
[2021-01-06 21:41:53.636749][00:03:06.892990][RIGCTRL:trace] rig_set_mode: retcode after set_mode=-9
[2021-01-06 21:41:53.636749][00:03:06.893011][RIGCTRL:trace] elapsed_ms: start = 0,0
[2021-01-06 21:41:53.636749][00:03:06.893022][RIGCTRL:trace] elapsed_ms: after gettime, start = 1609969313,636749700
[2021-01-06 21:41:53.636749][00:03:06.893028][RIGCTRL:error] error: Command rejected by the rig
It should not be sending 0x06 0x01 0x03 to an IC-706 MkII, it will only accept 0x06 0x01 to set USB mode.

73
Bill
G4WJS.

@mdblack98 mdblack98 added bug critical A problem for common operations with WSJT-X, GPredict, RigPi, etc. WSJTX Bugs affecting WSTJ-X operations labels Jan 6, 2021
@mdblack98
Copy link
Contributor Author

RIG_PASSBAND_NOCHANGE was changed from 0 to -1 and icom.c was relying on filter=0 to be PASSBAND_NOCHANGE.
So we must initialize filter to RIG_PASSBAND_NOCHANGE in icom.c init()

mdblack98 added a commit that referenced this issue Jan 6, 2021
mdblack98 added a commit that referenced this issue Jan 7, 2021
@mdblack98
Copy link
Contributor Author

WSJT-X is using this logic
if (-RIG_ENAVAIL == rc || -RIG_ENIMPL == rc)
Where set_vfo is return RIG_ENTARGET instead of RIG_ENIMPL|RIG_ENAVAIL
So we are going to change hamlib set_vfo to return the more intuitive error codes
ENTARGET should be reserved for an error return when calling a function vfo-specific and the rig doesn't support it.

@mdblack98 mdblack98 added this to the 4.1 milestone Jan 8, 2021
mdblack98 added a commit that referenced this issue Jan 8, 2021
…fo==NULL

Add IC706* to rigs that don't do dsp settings
Remove dynamic detection of transceive status in icom/frame.c
#490
mdblack98 added a commit that referenced this issue Jan 8, 2021
@mdblack98
Copy link
Contributor Author

Working now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug critical A problem for common operations with WSJT-X, GPredict, RigPi, etc. WSJTX Bugs affecting WSTJ-X operations
Projects
None yet
Development

No branches or pull requests

1 participant