-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Fix uMesh RF POWER configuration error #9326
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
base: develop
Are you sure you want to change the base?
Conversation
|
|
|
Thanks. Since this uses a power amp, we should also set a TX_GAIN_LORA array in configuration.h , based on the charts at https://github.com/linser233/uMesh/blob/main/RF_Power.png |
I have checked the relevant For the 30 dBm SKU: The output power cannot simply be treated as a linear offset. In this case, is there a better way to handle it rather than just using |
|
@linser233 , see eg : Line 149 in d493f5f
for how to make an array of gains (first value is gain added when 1dBm does into the SX1262, second value is gain added when 2dBm into the SX1262, ... 22nd value is gain added when 22dBm into the SX1262) |
In that case, I would need four configuration files — 1262 and 1268, each with 30S and 33S. Would it be better to place them in a separate directory? |
Since the values are relatively close (at least for the different frequencies with the same device), maybe just take an average or a maximum to keep it simple ... |
What I mean is that my module is available in four SKUs: the SX1268-based versions rated at 30 dBm and 33 dBm, and the SX1262-based version.😂 |
|
@fifieldt I’ve considered that the 33 dBm SKU is unlikely to be produced in significant numbers, so it may be more practical to simply rename the current YAML files to reflect the 30 dBm SKU. |
Fix uMesh RF POWER configuration error
(see https://github.com/linser233/uMesh/blob/main/RF_Power.md)