ymodem:set the transport retry as optional

[anjiahao1]

Summary

The waiting duration of the rb/sb command can be determined based on the number of retransmissions, so that ymodem can restart to the normal system after running in the bootloader for a short period of time.

Impact

ymodem

Testing

use stm32 ymodem

[nuttxpr]

[Experimental Bot, please feedback here]

NuttX PR Requirements Review

Verdict: The PR does not fully meet NuttX requirements.

Missing Information:

• Summary:
  • Lacks clarity on why this change is necessary. What problem does it solve?
  • Needs specifics on how the waiting duration is calculated.
  • Should mention which functional part of the code is modified (e.g., ymodem driver).
• Impact:
  • Unclear if this is a new feature or modification.
  • Does this change user interaction with ymodem?
  • Potential impact on compatibility (backward/forward) is not addressed.
• Testing:
  • Insufficient detail.
  • Provide:
    • Specific build host details (OS, CPU, compiler version).
    • Target architecture, board, and configuration.
    • Actual testing logs before and after the change.

Recommendations:

1. Expand the Summary: Clearly articulate the problem, the solution (including calculation details), and the affected code section.
2. Complete the Impact Section: Address all points, especially user impact and compatibility concerns.
3. Provide Detailed Testing Information: Include specific host/target setups and relevant log snippets.

[cederom]

Thank you @anjiahao1 :-)