Skip to content

Add new packet proposal for heartbeat and system information packet - #3341

Open
ThadHouse wants to merge 3 commits into
wpilibsuite:mainfrom
ThadHouse:newpacketproposal
Open

Add new packet proposal for heartbeat and system information packet#3341
ThadHouse wants to merge 3 commits into
wpilibsuite:mainfrom
ThadHouse:newpacketproposal

Conversation

@ThadHouse

Copy link
Copy Markdown
Member

This is a proposal for an updated packet setup. This does the following things.

  1. Most information is removed from the heartbeat, available for future use.
  2. The heartbeat changes to be an FD frame when the device is configured for FD. This allows devices to know if they can send an FD frame by if this frame is received as FD or not. If there is a 2.0 device on the bus, this will automatically bring down the bus with error frames, instead of delaying until a vendor actually sends an FD frame.
  3. Moves all of the information that was into the heartbeat into an information packet. This is send at a much slower rate. Additionally, this packet encodes the bus number. This will be used by the system to determine if users accidentally cross their buses.

@ThadHouse

Copy link
Copy Markdown
Member Author

I realized we removed match time from the heartbeat, but that actually might be something useful. I can add that back in easily enough.

@ThadHouse

Copy link
Copy Markdown
Member Author

We should also create a low priority device type, and use that for the information packet.

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.

1 participant