Attention is currently required from: Yu-Ping Wu.
Nicholas Chin has posted comments on this change by Yu-Ping Wu. ( https://review.coreboot.org/c/coreboot/+/87224?usp=email )
Change subject: Documentation,util: Run util_readme.sh to regen *.md ......................................................................
Patch Set 1:
(1 comment)
File Documentation/util.md:
https://review.coreboot.org/c/coreboot/+/87224/comment/dd1c4263_cbfe5b61?usp... : PS1, Line 73: c
Why does `convert` become lower case ?
util_readme.sh just copies whatever is in the `description.md` file for a utility. Looks like these series of commits occured: - Commit 81e0d689c078 ("Documentation: Move intelp2m from description.md to Documention") added the lowercase "convert the configuration DW0/1..." text to `util/intelp2m/description.md` - Commit 56846091f136 ("util, Documentation: Run util_readme.sh to regen util.md") updated Documentation/util.md to match using the script. - Commit da54bd60af60 ("Documentation: Update information about intelp2m") manually updated `Documentation/util.md` (without using the script) to the "Convert the inteltool ..." that's currently there, so in effect this patch effectively reverts da54bd60af60 and restores the text from 56846091f136.
So I guess `util/intelp2m/description.md` should have been updated to the "Convert the inteltool register dump..." text before regenerating the *.md files using `util_readme.sh` so as to not lose the updated description from da54bd60af60