morransmith / sx127x

sx1276/7/8/9 library (mirror)
https://git.morran.xyz/morran/sx127x
18 stars 8 forks source link

Need help ! #2

Open ysn-ba opened 4 years ago

ysn-ba commented 4 years ago

Hi , thanks for sharing your work. I tried your codes with stm32F4 discovery with Keil IDE . I'm having error which stops me while debugging . ../Src/main.c(171): warning: #111-D: statement is unreachable sx127x_free(&sx1276_phy); in addition to many warnings with same notice ..\Src\sx127x_io.c(33): warning: #68-D: integer conversion resulted in a change of sign return -1; ..\Src\sx127x.c(422): warning: #68-D: integer conversion resulted in a change of sign return -1; ..\Src\sx127x_private.c(378):warning:#68-D: integer conversion resulted in a change of sign return -1; can you help me please .

morransmith commented 4 years ago

Hi ../Src/main.c(171): warning: #111-D: statement is unreachable sx127x_free(&sx1276_phy); is normal in this case because you never exit the main loop and this function will never be called. if your program never exits then you can just delete this line. warning: #68-D: integer conversion resulted in a change of sign return -1; appears because functions must return an unsigned 8-bit number, and the return number (-1) in them has a sign, which in fact will be 0xff you can ignore these warnings or replace all return -1 with return 0xff

morransmith commented 4 years ago

btw in the dev branch I replaced the dynamic allocation of sx127x_dev_t with static and fixed the pa_select behavior. so I recommend using it