Skip to content

Fix --pdports for framework systems#305

Merged
JohnAZoidberg merged 7 commits intomainfrom
pdports-cypd
Mar 31, 2026
Merged

Fix --pdports for framework systems#305
JohnAZoidberg merged 7 commits intomainfrom
pdports-cypd

Conversation

@JohnAZoidberg
Copy link
Copy Markdown
Member

The regular --pdports command does not return useful data on our systems because it uses the chrome PD subsystem instead of the cypress PD subsystem that we use.

@JohnAZoidberg
Copy link
Copy Markdown
Member Author

This should probably replace the existing --pdports command

@JohnAZoidberg
Copy link
Copy Markdown
Member Author

need to add some example scenarios in example md files

@JohnAZoidberg
Copy link
Copy Markdown
Member Author

need to add some example scenarios in example md files
This should probably replace the existing --pdports command

done

@JohnAZoidberg JohnAZoidberg requested a review from kiram9 March 30, 2026 03:33
@JohnAZoidberg JohnAZoidberg changed the title Add --pdports-cypd Fix --pdports for framework systems Mar 30, 2026
The regular --pdports command does not return useful data on our systems
because it uses the chrome PD subsystem instead of the cypress PD
subsystem that we use.

Signed-off-by: Daniel Schaefer <dhs@frame.work>
It's not meaningful if no cable/device is connected

Signed-off-by: Daniel Schaefer <dhs@frame.work>
Misleading previously, because it's not instantenous measurements.

Signed-off-by: Daniel Schaefer <dhs@frame.work>
Signed-off-by: Daniel Schaefer <dhs@frame.work>
Signed-off-by: Daniel Schaefer <dhs@frame.work>
Signed-off-by: Daniel Schaefer <dhs@frame.work>
I noticed that the HPD bit was set even on some adapters

Signed-off-by: Daniel Schaefer <dhs@frame.work>
@JohnAZoidberg
Copy link
Copy Markdown
Member Author

Better than before, so merging.

@JohnAZoidberg JohnAZoidberg merged commit be8932f into main Mar 31, 2026
10 checks passed
@JohnAZoidberg JohnAZoidberg deleted the pdports-cypd branch March 31, 2026 06:32
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