You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When writing to the volume register, values without a decimal point do not get parsed correctly. For example, writing -20 to the volume register results in
[ 914.313055] Maximum attenuation exceeded.
[ 914.313062] Setting attenuation to -100 dB.
being printed to the kernel log.
In light of this, perhaps all of our drivers should use shared (and tested) string parsing/conversion functions.
The text was updated successfully, but these errors were encountered:
This would be stemming from a bug in the set_fixed_num function, correct? As you are seeing, it looks like the function just doesn't handle there not being a decimal point.
Yes, that should be the case. set_fixed_num in custom_functions.h should have a working implementation. I'd vote for including that header in this driver to avoid duplicating code.
When writing to the volume register, values without a decimal point do not get parsed correctly. For example, writing -20 to the volume register results in
being printed to the kernel log.
In light of this, perhaps all of our drivers should use shared (and tested) string parsing/conversion functions.
The text was updated successfully, but these errors were encountered: