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

Feature: SF Audio amplifier mute - 2.9 edition #3741

Merged
merged 9 commits into from
Jul 14, 2023

Conversation

mha1
Copy link
Contributor

@mha1 mha1 commented Jul 1, 2023

This PR adds "Audio amplifier mute" to the SF/GF list. Possible use cases:

  • total silence, no noise, not only volume down
  • home brew BT Audio module picking up the audio signal at the amplifier input to use wireless ear buds. Silencing the speakers is necessary. Using SF makes that easy

mha1 added 2 commits July 1, 2023 19:55
 - All color lcd radios (including NV14) except Horus X12S
 - TX12, TX12MK2, ZORRO, BOXER, T8, TLITE, TPRO, LR3PRO, COMMANDO8
@Eldenroot
Copy link
Contributor

Eldenroot commented Jul 2, 2023

CZ:
#define TR_SF_DISABLE_AUDIO_AMP "Vypnutí zesilovače zvuku"

ENG original text:
#define TR_SF_DISABLE_AUDIO_AMP "Audio Amp Off"

@HThuren
Copy link
Contributor

HThuren commented Jul 2, 2023

DA
#define TR_SF_DISABLE_AUDIO_AMP TR("Sluk audio amp", "Sluk audio amplifier")

@Pat6874
Copy link
Contributor

Pat6874 commented Jul 2, 2023 via email

- updated radio languages CZ, DA, FR
- updated companion languages CZ, DA, FR
@mha1
Copy link
Contributor Author

mha1 commented Jul 2, 2023

I have been transaulted :-)

@elecpower: Companion done, I'd appreciate your review

  • updated companion for new SF
  • updated radio languages CZ, DA, FR
  • updated companion languages CZ, DA, FR

@robustini
Copy link
Contributor

IT
#define TR_SF_DISABLE_AUDIO_AMP TR("Amp. audio off", "Amplificatore audio off")

@ParkerEde
Copy link
Contributor

DE
#define TR_SF_DISABLE_AUDIO_AMP "Audio Amp AUS"

@raphaelcoeffic
Copy link
Member

raphaelcoeffic commented Jul 3, 2023

@mha1 has this already been implemented in main? Normally we implement new things in main, and back-port / cherry-pick if really necessary into a stable branch (ex: 2.9), not the opposite.

@zyren
Copy link
Contributor

zyren commented Jul 3, 2023

CN

#define TR_SF_DISABLE_AUDIO_AMP        "关闭音频功放"

TW

#define TR_SF_DISABLE_AUDIO_AMP        "關閉音頻功放"

@mha1
Copy link
Contributor Author

mha1 commented Jul 3, 2023

@raphaelcoeffic I know, but main was in flux and changing yaml datastructs logic. Find PR ported to main here: #3753

@mha1 mha1 changed the title Feature: SF Audio amplifier mute Feature: SF Audio amplifier mute - 2.9 edition Jul 3, 2023
@pfeerick
Copy link
Member

pfeerick commented Jul 3, 2023

The other thing you should have kept in mind is 2.9 has been essentially in feature freeze, and is now in RC, so basing this on 2.9 is more than a bit naughty ;)

@mha1
Copy link
Contributor Author

mha1 commented Jul 4, 2023

what? why? me naughty? I plead total ignorance your honor.

@ajjjjjjjj
Copy link
Contributor

PL

#define TR_SF_DISABLE_AUDIO_AMP TR("Wycisz wzm.", "Wycisz wzmacniacz audio")

@mha1
Copy link
Contributor Author

mha1 commented Jul 4, 2023

@pfeerick Seriously, say the word and I'll close it. Until then I'll keep it alive and updated.

@ulfhedlund
Copy link
Contributor

ulfhedlund commented Jul 4, 2023

SE
#define TR_SF_DISABLE_AUDIO_AMP TR("Audioförst. av","Audioförstärkare av")

@pfeerick
Copy link
Member

pfeerick commented Jul 5, 2023

@pfeerick Seriously, say the word and I'll close it. Until then I'll keep it alive and updated.

Nah, keep it alive... I'm inclined to let this infraction slide 🤪. Might as well round that feature off a bit more since you've already done the 2.9 version.

@pfeerick
Copy link
Member

pfeerick commented Jul 5, 2023

btw, I think you should be able to knock the type="unfinished" flag off the companion translations that are translated ;)

@pfeerick pfeerick added this to the 2.9 milestone Jul 5, 2023
@mha1
Copy link
Contributor Author

mha1 commented Jul 5, 2023

I accept the sentence as hard as it may be, your Honor

- removed type unfinished for translated languages
@ToshihiroMakuuchi
Copy link
Contributor

ToshihiroMakuuchi commented Jul 8, 2023

JP
#define TR_SF_DISABLE_AUDIO_AMP "オーディオアンプ OFF"

@pfeerick pfeerick added the enhancement ✨ New feature or request label Jul 14, 2023
@pfeerick pfeerick merged commit 5193225 into EdgeTX:2.9 Jul 14, 2023
pfeerick pushed a commit that referenced this pull request Jul 28, 2023
confused FUNCTION_* with FUNC_*
@brainbubblersbest
Copy link

brainbubblersbest commented Aug 21, 2023

sorry. But why not for the X12S?

FROS mutes the Audio Ampflifier too when using external r9m to prevent the humming noise.
So it does exactly the same on fros as the other radios that are supported here on Edgetx.

MCU PI09 can be changed to Audio_Mute_pin in hal instead of the actual definition Shutdown_Pin.
MCU PI09 is traced to The Amplifier Shutdown Pin like it is on the T16. And I think it is not used under the actual hal Definition somewhere in the code.

Here is the circuit documentation from the T16.
https://user-images.githubusercontent.com/48190425/102207842-ed477380-3ece-11eb-936c-3139c995007c.jpg
Similar to x12s

@mha1
Copy link
Contributor Author

mha1 commented Aug 22, 2023

@3djc your word on this?

@3djc
Copy link
Collaborator

3djc commented Aug 22, 2023

To my knowledge, fros uses the x12s audio chip in a completely different way that allows them to mute, but would not work at all with EdgeTx (and the audio chip mute pin is soldered to the ground)

@mha1
Copy link
Contributor Author

mha1 commented Aug 22, 2023

🙏

@brainbubblersbest
Copy link

brainbubblersbest commented Aug 22, 2023

To my knowledge, fros uses the x12s audio chip in a completely different way that allows them to mute, but would not work at all with EdgeTx (and the audio chip mute pin is soldered to the ground)

Yes it is soldered to Ground.
And Yes FROS uses a second Chip for mp3 decoding. (VLS1)
Both Audio ICs together with mcu
But that Chip was never used at all with opentx and edgetx.
The Shutdown Pin from apa2603a is traced in the same way like the fm8002 SD Pin on T16 Mainboard.

Im sure about that. Thats why I desoldered my defective old Mainboard and measured the Layout to compare with T16/X10 Tech Documentation.
THIS mainboard

The X10 Hal definition may be Named audio mute pin. But if you would measure the pcb, you would notice that it in fact controls the Shutdown Pin on the audio Amplifier. NOT the Mute In pin.
Fm8002 doesnt even have such an Input

Just take a closer Look into THIS:
T16_X10Copy Wiring Diagramm

Anyway: I would definite Test this on my Radio for Validation. 😉

@3djc
Copy link
Collaborator

3djc commented Aug 22, 2023

@brainbubblersbest
Copy link

brainbubblersbest commented Aug 22, 2023 via email

@raphaelcoeffic
Copy link
Member

raphaelcoeffic commented Aug 23, 2023

@brainbubblersbest the real question is whether or not the MUTE pin (marked blue in your picture) is somehow wired to other things as well like shutdown. Regarding start-up time, it is specified in the datasheet as 1.2s (up to 2s):
Screenshot 2023-08-23 at 07 42 17

That being said, if you guys want to reverse engineer how it is used in FrOS, go ahead! We can see how that can be implemented then.

That being said, the X12S audio circuit is fairly noise-free, so that it can be implemented in a different way, for instance by stopping data transfer to the VS1053b (maybe just dropping the buffers that would otherwise be transferred instead).

@3djc
Copy link
Collaborator

3djc commented Aug 23, 2023

@brainbubblersbest the real question is whether or not the MUTE pin (marked blue in your picture) is somehow wired to other things as well like shutdown.

Mute is connected to GND directly

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ✨ New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.