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

Include field descriptions for picotool otp list #134

Closed
kilograham opened this issue Sep 3, 2024 · 0 comments · Fixed by #174
Closed

Include field descriptions for picotool otp list #134

kilograham opened this issue Sep 3, 2024 · 0 comments · Fixed by #174
Assignees

Comments

@kilograham
Copy link
Contributor

$ picotool otp list crit1.boot_arch
ROW 0x0040: OTP_DATA_CRIT1
        "Page 1 critical boot flags (RBIT-8)"
    field BOOT_ARCH (bit 3)

vs

$ picotool otp get crit1.boot_arch
ROW 0x0040: OTP_DATA_CRIT1
        "Page 1 critical boot flags (RBIT-8)"

    VALUE 0x000000

    field BOOT_ARCH (bit 3) = 0
        "Set the default boot architecture, 0=ARM 1=RISC-V. Ignored if ARM_DISABLE, RISCV_DISABLE or SECURE_BOOT_ENABLE is set."

There is a -n for not showing descriptions; perhaps the default should be to show field descriptions if you are matching on field, and maybe add another option to force showing field descriptions always

@will-v-pi will-v-pi linked a pull request Nov 12, 2024 that will close this issue
will-v-pi added a commit that referenced this issue Nov 20, 2024
* List field desciptions if field matching (fixes #134)
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 a pull request may close this issue.

2 participants