Adds support for producer/consumer identified message in the CallbackEventHandler #743
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…EventHandler. When the user registers this callback, we proxy the producer and consumer identified messages to it. This is useful for a node that needs to discover remote state and sends out identify producer / consumer queries for doing so.
* master: Fix test makefiles and test target (#736) Moves the bootloader hook so that the hook implementation gets a chance to look at incoming frames. Fixes missing includes. Fixes missing symbol error. Adds stm32f072 HAL drivers to bare.armv6m compilation target. Update HASSERT and DIE to use ets_printf() on ESP32 (#744) Adds a useful command to generate release notes. Fix MultiConfiguredPC.hxx compilation with C++20 (#737) Esp32 TWAI updates based on comments (#720) Fork spiffs/stm32f7/Stm32SPIFFS.cxx to st/Stm32Flash.hxx step 4 Fork spiffs/stm32f7/Stm32SPIFFS.cxx to st/Stm32Flash.hxx step 2 Fork spiffs/stm32f7/Stm32SPIFFS.cxx to st/Stm32Flash.hxx Fix compile error in railcom board definitions. Stm32 pwm f7xx update (#735) Add L4xx HAL include for Stm32PWM (#734) Fix missing ifdef for reboot because macos build is broken.
bakerstu
approved these changes
Nov 11, 2023
| /// message arrives. | ||
| /// | ||
| /// @param mti MTI_PRODUCER_IDENTIFIED_UNKNOWN or | ||
| /// MTI_CONSUMER_IDENTIFIED_UNKNOWN. The state bits can be read from |
There was a problem hiding this comment.
Can we not pass in the correct MTI based on pre-decoding the state of the report?
There was a problem hiding this comment.
I think we don't want to. This way you can test for producer / consumer with a single ==. Otherwise the test is much more complicated.
balazsracz
added a commit
that referenced
this pull request
Nov 12, 2023
* master: Unaligned read/write support in TivaEEPROM storage (#741) Adds support for producer/consumer identified message in the CallbackEventHandler. (#743) Improve error handling in BroadcastTimeDefs::string_to_date() (#745) Fix test makefiles and test target (#736) Moves the bootloader hook so that the hook implementation gets a chance to look at incoming frames. Fixes missing includes. Fixes missing symbol error. Adds stm32f072 HAL drivers to bare.armv6m compilation target. Update HASSERT and DIE to use ets_printf() on ESP32 (#744) Adds a useful command to generate release notes.
balazsracz
added a commit
that referenced
this pull request
Dec 23, 2023
…t-hub-router * 'master' of github.com:bakerstu/openmrn: (152 commits) Switches over make tests at toplevel to use the tests target instead of the cov target. Ensures that openmrn and application directory builds can run in parallel. (#754) Ensures that 'make clean' empties lib directory of symlinks. (#753) Add an input/output GPIO type. This helps support bit banged I2C. (#752) Adds TinyUsb based CDC virtual Com port driver with an STM32 port. (#748) Limit the amount of input we read into memory in arduino sketches. (#746) Updates to send_datagram cmdline utility (#740) Adds lflash command to the bracz-railcom board support makefile. Do not crash on an incoming message with 0 alias. (#751) Add common app test coverage rules. (#750) Fix symlinking of library files to the lib directory. (#747) Refactors the SPIFFS flash driver for the F7 into a generic stm32 flash driver. (#742) Unaligned read/write support in TivaEEPROM storage (#741) Adds support for producer/consumer identified message in the CallbackEventHandler. (#743) Improve error handling in BroadcastTimeDefs::string_to_date() (#745) Fix test makefiles and test target (#736) Moves the bootloader hook so that the hook implementation gets a chance to look at incoming frames. Fixes missing includes. Fixes missing symbol error. Adds stm32f072 HAL drivers to bare.armv6m compilation target. ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When the user registers this callback, we proxy the producer and consumer identified messages to it. This is useful for a node that needs to discover remote state and sends out identify producer / consumer queries for doing so.