Typo for fractional part of `lposc_freq_khz`?

[tommy-gilligan]

Currently:

pico-sdk/src/rp2_common/hardware_powman/powman.c

Line 72 in ee68c78

uint32_t lposc_freq_khz_frac16 = (lposc_freq_khz % 1000) * 65536 / 1000;

I think this line should be referencing lposc_freq_hz instead of lposc_freq_khz

Given an input value of lposc_freq_hz = 32768
lposc_freq_khz will be 32768 / 1000 = 32
This is the integer part
Fractional part cannot be derived from this

I am not sure if the rest of the maths makes sense either.

edit: The maths is correct but it does seem the wrong variable is being referenced.

[tommy-gilligan]

65536 / 1000 should be (65536 / 1000) imho

[kilograham]

Please feel free to make a PR (for the khz->hz), i think the 65536 / 1000 is good as is, because otherwise you'd be multiplying by 655 not 655.36

[tommy-gilligan]

Misread the size of variable. Without parentheses is better.

[lurch]

Fix merged into develop