MMDVM icon indicating copy to clipboard operation
MMDVM copied to clipboard

MMDVM stuck in FM mode on mqtt branch

Open ke5gdb opened this issue 1 year ago • 3 comments

This line causes a feedback loop with MMDVMHost. CFM::reset() is called when MMDVMHost asserts MMDVM_SET_MODE to IDLE, but serial.writeFMStatus() will cause MMDVMHost to assert the mode back into FM. Ultimately this locks the modem in FM mode.

https://github.com/g4klx/MMDVM/blame/a8bf99037ec31546321a6fc69e355f77d9d1aaef/FM.cpp#L391

I commented on the line here in the commit, but this comment is likely not the correct medium, hence the new issue.

https://github.com/g4klx/MMDVM/commit/7ef0d992d9190466bf13b7306b8867358898c9f1#r132809346

73 de KE5GDB

ke5gdb avatar Nov 17 '23 06:11 ke5gdb

Hi Andrew I think your fix is correct. I've incorporated it into the mqtt branch of the MMDVM firmware and I hope that the problem has now gone away. Thanks for finding it. Jonathan  G4KLX

On Friday, 17 November 2023 at 06:13:03 GMT, Andrew Koenig ***@***.***> wrote:  

This line causes a feedback loop with MMDVMHost. CFM::reset() is called when MMDVMHost asserts MMDVM_SET_MODE to IDLE, but serial.writeFMStatus() will cause MMDVMHost to assert the mode back into FM. Ultimately this locks the modem in FM mode.

https://github.com/g4klx/MMDVM/blame/a8bf99037ec31546321a6fc69e355f77d9d1aaef/FM.cpp#L391

I commented on the line here in the commit, but this comment is likely not the correct medium, hence the new issue.

7ef0d99#r132809346

73 de KE5GDB

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

g4klx avatar Nov 18 '23 16:11 g4klx

Hi @g4klx Jonathan , I got this problem "stuck in FM mode" when I Not set the configuration in mmdvmhost,

[FM] ... ModeHang=10

Do you think it is a good idea to add default ModeHang (Not Infinity), in firmware or in MMDVMHost?

Thank you very much!

DE BI7JTA

bi7jta avatar Jan 12 '24 10:01 bi7jta

Can you try the latest MMDVM firmware, this issue is fixed with it.

g4klx avatar Jan 12 '24 12:01 g4klx