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

Update MT99xx_ccnrf.ino #1025

Closed
wants to merge 1 commit into from
Closed

Update MT99xx_ccnrf.ino #1025

wants to merge 1 commit into from

Conversation

rdba2k
Copy link
Contributor

@rdba2k rdba2k commented Sep 28, 2024

add QF009 Su35 support to MT99XX protocol / H7 sub protocol

add QF009 Su35 support to MT99XX protocol / H7 sub protocol
@rdba2k rdba2k closed this Oct 2, 2024
@rdba2k
Copy link
Contributor Author

rdba2k commented Oct 2, 2024

It is going to be replaced by better code

@pascallanger
Copy link
Owner

The code has proposed won't work anymore for the H7...
packet[6] |= FLAG_MT_RATE1; -> it's taking the packet[6] previous flags which you are overwriting with your new code

@rdba2k
Copy link
Contributor Author

rdba2k commented Oct 2, 2024

I realized the issue. I will work on adding a new sub protocol Su35 but my code is not working yet.

I am thinking in MT99xx_ccnrf.ino

add
#define MT99XX2_SUB_BASE 0x10

so this will change to
if(protocol == PROTO_MT99XX2)
sub_protocol|=0x08; -> sub_protocol |=MT99XX2_SUB_BASE;

and all
PA18+8 -> PA18 | MT99XX2_SUB_BASE

Thus leaving subprotocol "8" to Su35.

Or you can made the change and I will learn what are the changes needed.

Thanks.

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

Successfully merging this pull request may close these issues.

2 participants