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
i3c: add i3c shell #70773
i3c: add i3c shell #70773
Conversation
1c3b923
to
7b9c755
Compare
I've added a better subcmd which will autocomplete with only the possible I3C devices, as well as only I3C targets are either attached or within it's list. I've also added the i3c/i2c attach/reattach/detach shell commands Unfortunately for the I2C device list, there isn't an easy way to look up a string of the only addresses with in it's list. |
9312548
to
ad42690
Compare
Add an I3C shell. This includes support of all I3C CCC commands that currently have helper functions implemented. This also includes all the read/write shell commands that the i2c shell supported. An Info command is also provided which will print out all i3c and i2c info of an i3c bus. Only SDR read/writes are currently implemented. Signed-off-by: Ryan McClelland <ryanmcclelland@meta.com>
ad42690
to
d35764f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks for the PR!
|
||
if (argc > 4) { | ||
num_bytes = strtol(argv[4], NULL, 16); | ||
if (num_bytes > MAX_I3C_BYTES) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I suggest to let user know he/she was clamped.
Add an I3C shell. This includes support of all I3C CCC commands that currently have helper functions implemented. This also includes all the read/write shell commands for SDR only that the i2c shell supported. An Info command is also provided which will print out all i3c and i2c info of an i3c bus.