-
Notifications
You must be signed in to change notification settings - Fork 183
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
Idea about implementing the Fine timestamp functionality in a Basic Station (Corecell design and SX1303 or SX1302 chip) #177
Comments
Also interested having fine-timestamps implemented for SX1303. I was expecting to see this added last year. |
I've been testing the proposed basic station code with the fine timestamp changes (+ ChirpStack) for couple weeks with no problems;) |
@beitler could you provide any feedback on this issue? |
Radio silence continues... Is this repository still active? @beitler or anyone there, could you please give some feedback on this matter. |
Hi @LouneCode , I'm sorry for the silence. I saw your patch and I think you did a great job there. Thank you very much for your contribution. It's an interesting idea to replace the sub-second part of the rxtime with the precise timestamp. However, generally @brocaar the |
Thank You @beitler for your feedback. "So, I'm wondering, do you have a particular use case in mind for this behavior?" : No, I don't have a specific use case for this behavior. in the patch, UTC part of the Do you have any other idea for this UTC part of the fine timestamp. Perhaps there no need to use UTC part at all? |
Sorry, I messed a things in the previous comment. Please forget this comment:) Do you have any other idea for this UTC part of the fine timestamp. Perhaps there no need to use UTC part at all? You mean the following patch code in src/s2e.c file and the rxtime field :
This fine timestamp fractal part in the rxtime field has been there for testing purposes only. I Think this change is not needed and can be reverted back as follows.
|
It's so needed! |
Hi,
I have an idea how to enable the Fine timestamp functionality on Basic Station. The idea only applies to the Corecell design (std, stdn variants) and the latest chips SX1303 and SX1302. An example of the concept can be found in the LouneCode/gls-basicstation repository.
It would be nice if someone from Semtech would review the code changes and give feedback on this proposal. The easiest way to investigate the required code changes is to look differences of this patcht.
Code changes shortly:
s2e_flushRxjobs(), changes in loggin. Changes in uj_encKVn() function call arguments.
Cheers
LouneCode - Only husky in the village
The text was updated successfully, but these errors were encountered: