`SUBGRF_SetRfTxPower` interprets `power` argument incorrectly.

STMicroelectronics / STM32CubeWL

STM32Cube MCU Full FW Package for the STM32WL series - (HAL + LL Drivers, CMSIS Core, CMSIS Device, MW libraries plus a set of Projects running on boards provided by ST (Nucleo boards)

Other

110 stars 55 forks source link

`SUBGRF_SetRfTxPower` interprets `power` argument incorrectly. #64

Open Miceuz opened 1 year ago

Miceuz commented 1 year ago

The bug is in Middlewares/Third_Party/SubGHz_Phy/stm32_radio_driver/radio_driver.h:

SUBGRF_SetRfTxPower expects Tx power level [0..15]. It does checks on power > 15 to select RFO_HP and then it calls SUBGRF_SetTxParams that expects RF output power [-18..13] dBm.

So it interprets power argument incorrectly as it it's index, not dBm while later it's used as dBm. All the client code in https://github.com/stm32duino/STM32LoRaWAN/ uses SUBGRF_SetRfTxPower providing power in dBm.

RJMSTM commented 1 year ago

Hi @Miceuz,

Thank you for this report. We will get back to you as soon as we analyze it further. This may take some time. Thank you for your comprehension.

With regards, Rania

fpistm commented 1 year ago

Hi @Miceuz and thanks to submit this issue. @RJMSTM is there an internal BZ about this? Thanks.

ASELSTM commented 1 year ago

ST Internal Reference: 149717