Skip to content
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

Need help ! #2

Open
ysn-ba opened this issue Mar 2, 2020 · 3 comments
Open

Need help ! #2

ysn-ba opened this issue Mar 2, 2020 · 3 comments

Comments

@ysn-ba
Copy link

ysn-ba commented Mar 2, 2020

No description provided.

@ysn-ba ysn-ba closed this as completed Mar 2, 2020
@ysn-ba ysn-ba reopened this Mar 2, 2020
@ysn-ba
Copy link
Author

ysn-ba commented Mar 2, 2020

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 .

@ysn-ba ysn-ba changed the title Hi thanks for sharing your work. Need help ! Mar 2, 2020
@morransmith
Copy link
Owner

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
Copy link
Owner

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants