Skip to content

Conversation

@fpenard
Copy link
Contributor

@fpenard fpenard commented Aug 23, 2025

Summary of changes:

read_i2c_volume static function is defined but not used in targets/taranis/volume_i2c.cpp.

This one generates the next warning on build.

edgetx/radio/src/targets/taranis/volume_i2c.cpp:36:16: warning: 'int32_t read_i2c_volume()' defined but not used [-Wunused-function]
   36 | static int32_t read_i2c_volume()

@pfeerick pfeerick added the house keeping 🧹 Cleanup of code and house keeping label Aug 24, 2025
@pfeerick
Copy link
Member

@raphaelcoeffic Do we remove this, or revert #5863 and probably add && defined(AUDIO) so that debug building doesn't break? At present we can set, but not read the volume level on non-soft volume radios via cli.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

house keeping 🧹 Cleanup of code and house keeping

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants