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

Alarm can transmit everywhere #179

Closed
ea7hjb opened this issue Dec 3, 2023 · 7 comments
Closed

Alarm can transmit everywhere #179

ea7hjb opened this issue Dec 3, 2023 · 7 comments
Labels
bug Something isn't working

Comments

@ea7hjb
Copy link

ea7hjb commented Dec 3, 2023

Hi,

Even if TX is not fully unlocked, ALARM can do TX everywhere (including airband) if menu option 45 is set to "TONE".

Just for the info, 1750 Hz does show "TX DISABLE" as expected, so that function is OK.

Thank you for your effort and time and best regards.

@egzumer
Copy link
Owner

egzumer commented Dec 3, 2023

I don't use that option, but if I remember correctly the tone option is simply radio playing the tone through a speaker. It doesn't transmit anything, does it?

@ea7hjb
Copy link
Author

ea7hjb commented Dec 3, 2023

Hello. If option is set to "SITE" it will only play on the speaker. If set to "TONE" it will do both TX and Speaker (switching between the two option each cycle).

If set to "TONE", it will not obey the TX table and will emit everywhere.

I remember I posted this exact same bug on 1of11 GitHub, and she fixed it, in case you can check that there and save yourself some time :)

@egzumer
Copy link
Owner

egzumer commented Dec 3, 2023

Ok thank, I'll check that then.

@ea7hjb
Copy link
Author

ea7hjb commented Dec 3, 2023

Found it!:
OneOfEleven/uv-k5-firmware-custom@b385b53

In case it helps.

@egzumer egzumer added the bug Something isn't working label Dec 3, 2023
@MachoEnLaNoche

This comment was marked as off-topic.

@Arikshten
Copy link

This alarm function (IMVHO) is one to be removed and its memory used for better functions.
I have never found a use case where it is appropriate/sensible/helpful for this to even exist. Although it might be simpler to fix than remove.

@egzumer
Copy link
Owner

egzumer commented Dec 7, 2023

Fixed in cd032c3

@egzumer egzumer closed this as completed Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants