Skip to content

Comments

Harmonize sections references titles#145

Merged
vstehle merged 1 commit intoARM-software:mainfrom
vstehle:parnames
Aug 27, 2025
Merged

Harmonize sections references titles#145
vstehle merged 1 commit intoARM-software:mainfrom
vstehle:parnames

Conversation

@vstehle
Copy link
Collaborator

@vstehle vstehle commented Aug 26, 2025

Please have a look and let me know if there is any veto.
I think this is clearer and worth harmonizing.

@vstehle
Copy link
Collaborator Author

vstehle commented Aug 26, 2025

Added missing citation reference to the FF-A footnote.

@vstehle
Copy link
Collaborator Author

vstehle commented Aug 26, 2025

Rebased.

Copy link
Contributor

@xypron xypron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

As we use a Sphinx extension for all our references to the UEFI
specification, all those references mention the section title after the
section number automatically.
Among our other references written manually, some do mention the section
title (FF-A) and some do not (BBR & Devicetree).

Harmonize all our sections references to mention the referenced section
title after the § sign and the section number.
This has the benefit that references become easier to follow across
revisions when updating the referenced specifications.

Also, document the convention in the README.

Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>
@vstehle
Copy link
Collaborator Author

vstehle commented Aug 27, 2025

Thank you @daniel-thompson and @xypron for the review; capturing in the commit message.

@vstehle vstehle merged commit 7bca772 into ARM-software:main Aug 27, 2025
1 check passed
@vstehle vstehle deleted the parnames branch August 27, 2025 13:34
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.

3 participants