boards: doc: arm: fix documentation structure for mps2 and fvp_baser_aemv8r

[kartben]

Improve/consolidate documentation of these two boards. mps2 is still not perfect but at least this now ensures the toctree is correct/consistent.
Also now uses the fancy table of supported features :)

• https://builds.zephyrproject.io/zephyr/pr/95532/docs/boards/index.html#vendor=arm
• https://builds.zephyrproject.io/zephyr/pr/95532/docs/boards/arm/mps2/doc/index.html
• https://builds.zephyrproject.io/zephyr/pr/95532/docs/boards/arm/fvp_baser_aemv8r/doc/index.html

[marc-hb]

I tested this and confirm it gets rid of most of the remaining #95438 warnings.

[kartben]

@wearyzen something looks off w.r.t CI for these boards?

[wearyzen]

@wearyzen something looks off w.r.t CI for these boards?

Hi @kartben, I had a look at this and I am not sure why this wasn't reported in ci earlier but I'll need some time to fix this. I think if you remove the change in board.yml then its just documentation changes and ci shouldn't run on the board, do you want to try that to unblock this PR?

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[kartben]

@wearyzen something looks off w.r.t CI for these boards?

Hi @kartben, I had a look at this and I am not sure why this wasn't reported in ci earlier but I'll need some time to fix this. I think if you remove the change in board.yml then its just documentation changes and ci shouldn't run on the board, do you want to try that to unblock this PR?

Hey, that's a very good idea, thanks @wearyzen :)

[wearyzen]

I'll make a note of the CI failures and continue working on them but let me know if you want me to create a github issue to track it

[marc-hb]

@kartben you had this in earlier versions like f75a4b0

I cherry-picked one of these earlier versions and found this left in my tree when rebasing.

Just making sure you dropped this intentionally.

--- a/boards/arm/fvp_baser_aemv8r/board.yml
+++ b/boards/arm/fvp_baser_aemv8r/board.yml
@@ -1,6 +1,6 @@
 board:
   name: fvp_baser_aemv8r
-  full_name: Debug with Arm DS
+  full_name: FVP BaseR AEMv8-R
   vendor: arm
   socs:
   - name: fvp_aemv8r_aarch64

[kartben]

@kartben you had this in earlier versions like f75a4b0

I cherry-picked one of these earlier versions and found this left in my tree when rebasing.

Just making sure you dropped this intentionally.

--- a/boards/arm/fvp_baser_aemv8r/board.yml
+++ b/boards/arm/fvp_baser_aemv8r/board.yml
@@ -1,6 +1,6 @@
 board:
   name: fvp_baser_aemv8r
-  full_name: Debug with Arm DS
+  full_name: FVP BaseR AEMv8-R
   vendor: arm
   socs:
   - name: fvp_aemv8r_aarch64

Yeah this change was causing CI to actually try and run lots of integration tests on this board which happen to be apparently broken at the moment so while @wearyzen investigates I took the easy route of dropping this change for now
See

• boards: arm: fix full_name for fvp_baser_aemv8r #95687