-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Enabling Raw Frame Support and Promiscuous #54
Comments
Thanks for using QNEthernet and for the suggestion. I'm curious, can you point me to where you read about using quotes? |
Also, I'm curious: what did your non-working attempts look like? Would you mind listing them? I find it's sometimes good to put "what not to do" in addition "what to do". It appears this might be helpful here. |
The forum article that suggested quotes was this https://forum.arduino.cc/t/how-to-specify-multiple-extra-compiler-flags/994083 Here are some of the variants I tried:
I have been trying to receive LLDP packets, in the end I did not need to enable it as the reason it was not working was because of the line
in the RawFrameMonitor example. This ignores any raw packets over about 41 bytes and the LLDP packets where 491 bytes. |
[The reply appears to be missing the forum link.] Thank you for those incorrect attempts. I'm going to think about how I'd like to improve the documentation. That code in RawFrameMonitor checks for less than 60 bytes (64 - 4) and prints "SHORT Frame"; it prints the whole frame if >= 60 bytes. I don't think it ignores 491-byte frames? |
I have updated my comment to include the forum link. I had over simplified the RawFrameMonitor example as I just wanted to see the raw data printed to serial. The example code as written is fine. |
What do you think of these modifications to the README:
|
See commit 5484704. |
It took me a while to work out how you enable both raw frame support and promiscuous together as the readme only gives an example for enabling one macro. Can I suggest updating the example to include enabling two macros as bellow.
It all seems obvious now but many of the forum posts I read suggested quotes which lead me down the garden path. In the end I had omitted the -D from QNETHERNET_ENABLE_PROMISCUOUS_MODE as I had not spotted in the readme that this had been appended to QNETHERNET_ENABLE_RAW_FRAME_SUPPORT.
Amazing library though, thank you so much for your work on this.
The text was updated successfully, but these errors were encountered: