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

[WIP] Baud autodetect #58

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

andreagilardoni
Copy link
Contributor

@andreagilardoni andreagilardoni commented Jul 29, 2024

This PR aims to implement an autodetect for UART, so that it is possible to change the baudrate from renesas mcu.
This PR is the continuation on the discussion performed here: #56
Missing features

  • detect when the baudrate needs to be changed
    - discover it by adding a CRC, ECC or whatever (open to discussion)
    - discover by the size of the incoming message
    - others
  • Fix E (5553) uart: uart_get_buffered_data_len(1342): uart driver error
  • code cleaning
  • retro compatibility with previous version of FW

@andreagilardoni
Copy link
Contributor Author

Hi @matthieuvigne and @JAndrassy, I am trying to implement via FW the baudrate autodetect feature.
My Idea is the following:

  • the board boots up, autodetect starts on esp, looking for character 0x55
  • The renesas MCU calls begin on modem, with either the default value or a custom one.
  • the esp gets the correct value, sends 0x55 back and starts correctly

if an error occurs on Renesas and the MCU is reset, then:

  • the renesas restarts the baudrate autodetection procedure
  • esp needs to discover that something has happened and restart the detection [missing]

if an error occurs on ESP and the MCU is reset, then:

  • the renesas core needs to detect the fault [missing]
  • renesas core restarts the autodetect procedure

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