On Fri, 5 Mar 2021 at 10:22, Peter Maydell <peter.mayd...@linaro.org> wrote: > > On Fri, 5 Mar 2021 at 01:28, Richard Henderson > <richard.hender...@linaro.org> wrote: > > > > On 2/19/21 6:46 AM, Peter Maydell wrote: > > > @@ -1,5 +1,5 @@ > > > -Arm MPS2 and MPS3 boards (``mps2-an385``, ``mps2-an386``, > > > ``mps2-an500``, ``mps2-an505``, ``mps2-an511``, ``mps2-an521``, > > > ``mps3-an524``) > > > -========================================================================================================================================= > > > +Arm MPS2 and MPS3 boards (``mps2-an385``, ``mps2-an386``, > > > ``mps2-an500``, ``mps2-an505``, ``mps2-an511``, ``mps2-an521``, > > > ``mps3-an524``, ``mps3-an547``)' > > I think you should drop the list here, as it has gotten *way* too long. > > > > > @@ -27,6 +27,8 @@ QEMU models the following FPGA images: > > > Dual Cortex-M33 as documented in Arm Application Note AN521 > > > ``mps3-an524`` > > > Dual Cortex-M33 on an MPS3, as documented in Arm Application Note > > > AN524 > > > +``mps3-an547`` > > > + Cortex-M55 on an MPS3, as documented in Arm Application Note AN547 > > > > The list is down here, anyway. > > The title is what generates the text for the table of contents in the > next page up: > > https://qemu-project.gitlab.io/qemu/system/target-arm.html#board-specific-documentation > > I like having every board model listed explicitly in that ToC. (Though we > have broken this rule for the Aspeed boards with "*-bmc"...)
I put this patch into the pullreq as-is, because the series is otherwise ready to go and this was a minor thing; happy to send a followup patch tweaking the title text if we conclude that we want to do that. thanks -- PMM
