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

BMP remote protocoll and exception/timeout handling #858

Closed
UweBonnes opened this issue Apr 18, 2021 · 2 comments · Fixed by #1389
Closed

BMP remote protocoll and exception/timeout handling #858

UweBonnes opened this issue Apr 18, 2021 · 2 comments · Fixed by #1389
Labels
Feedback wanted Requires additional submitter feedback HwIssue Mitigation Solving or mitigating a Hardware issue in Software Potential Bug A potential, unconfirmed or very special circumstance bug User Testing Needed Looking for user testing reports
Milestone

Comments

@UweBonnes
Copy link
Contributor

Exception and timeout handling with BMP remote protocoll needs to be improved.This is seen e.g. when scanning the ROMTABLE of a STM32WLE5. It is the same chip as the STM32WL55 with an M0 at AP1. AP1 is somehow disabled on theSTM32WLE5, but AP1 registers are available. Reading CIDR on the STM32WLE5 gives a timeout, the remote protocoll probably aborts early for its read timeout and the exception is not set.

@esden
Copy link
Member

esden commented Dec 16, 2021

Is that an enhancement or a bug? This issue is a little bit vague. I would like to be able to categorize it properly.

@dragonmux
Copy link
Member

This seems like we might have inadvertently fixed this with all the improvements to how we talk ADIv5, from implementing proper protocol recovery to fixing the error recovery logic and how we spin on WAIT replies from the device. We'll mark this as a v1.10 item that needs testing, with a view to optimistically closing this issue if we can prove it's been fixed.

@dragonmux dragonmux added User Testing Needed Looking for user testing reports Feedback wanted Requires additional submitter feedback Potential Bug A potential, unconfirmed or very special circumstance bug HwIssue Mitigation Solving or mitigating a Hardware issue in Software Information Needed Maintainers need more information and removed Information Needed Maintainers need more information labels Feb 12, 2023
@dragonmux dragonmux added this to the v1.10 milestone Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feedback wanted Requires additional submitter feedback HwIssue Mitigation Solving or mitigating a Hardware issue in Software Potential Bug A potential, unconfirmed or very special circumstance bug User Testing Needed Looking for user testing reports
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants