Lengths of frequencies (CHG_FREQUENCY and OUT_FREQUENCY)

[dacwe]

Setting the (defaults) to parameters CHG_FREQUENCY (333kHz) and OUT_FREQUENCY (300kHz) it is read back as something else.

The code says that this is a "integer of length 1": https://github.com/mini-box/ups/blame/8c6db6732181958accb73bc6a405dacf99dde89c/src/lib/HIDOpenUPS.cpp#L73

Is the length (1) correct here? Seems like values above 255 is truncated?

[dacwe]

Super weird behaviour of the OpenUPS when these values are set by this repo. It didn't start properly (no vout) for example.

Had to restore the values using the Windows software to get everything working again.

[avg-I]

It's not documented anywhere, but the values of those parameters are actually enumerations.
Specific integers map to specific frequencies.
The mapping seems to exist only in the Windows software and, obviously, the firmware.
openups tool is not aware of it and just passes the value as-is.