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.

Sign up for GitHub

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

Jump to bottom

drivers: i3c: Reset device list addresses when RST command is issued #78784

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

drivers: i3c: Reset device list addresses when RST command is issued #78784

wants to merge 1 commit into from
+36 −1

Conversation

albertofloyd
Copy link
Collaborator

@albertofloyd albertofloyd commented Sep 20, 2024 •
edited
Loading

Reset dynamic addresses when RST command is issued

Fixes #78591

@albertofloyd albertofloyd marked this pull request as draft September 20, 2024 20:47
@albertofloyd albertofloyd mentioned this pull request Sep 20, 2024
Open
@albertofloyd
drivers: i3c: Reset device list addresses when RST command is issued
852206d 
Reset dynamic addresses when RST command is issued

Signed-off-by: Jose Alberto Meza <[email protected]>
@XenuIsWatching
Copy link
Member

I don't think this is the right approach... if something were to be added to also modify desc, something at the i3c_common.c such as a i3c_bus_rstdaa func would make sense which would call i3c_ccc_do_rstdaa_all and then set all the da to 0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dcpleung dcpleung Awaiting requested review from dcpleung

@XenuIsWatching XenuIsWatching Awaiting requested review from XenuIsWatching

At least 2 approving reviews are required to merge this pull request.

Assignees

@dcpleung dcpleung

Labels
area: I3C
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

drivers: i3c: common: ENTDAA after request is not possible after RSTDAA
4 participants
@albertofloyd @XenuIsWatching @zephyrbot @dcpleung