The shared runners are now tagged with `arm64-shared'; update our
configuration file to fix CI.

[Backport commit a549c202.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Add GUID for the Arm Corstone-1000 FVP.

[Backport commit 9559ee38.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

The firmware management protocol is not required by EBBR, only capsules in
FMP format.
We expect this protocol to be implemented in practice nonetheless, as all
the firmwares we know of have it.

Downgrade the error for missing fmp into a warning.

[Backport commit e8007253.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

[Backport commit 604fa217.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Use a specific version of Ubuntu instead of `rolling' to fix our CI.

[Backport commit 89b52e8a.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Teach the dt-parser how to parse dt-validate errors in schemas and add a
unit-test.

Reported-by: Sam Moss <sam.moss@arm.com>
[Backport commit a27aae5c.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Adjust the Ubuntu version used for the github runner, to repair the CI.
This has the nice side effect that we do not need a container anymore.

[Backport commit 2076cefe.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Weizhao Ouyang authored Oct 16, 2024 and Vincent Stehle committed Oct 17, 2024

U-Boot introduced the SMBIOS3 table support. Therefore, we need to adapt
the SMBIOS 3.0 format for the smbiosview.log file.

Signed-off-by: Weizhao Ouyang <weizhao.ouyang@arm.com>
[Backport commit 74af018d.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Add GUID for the Renesas RZG2UL-EVKIT or RZV2L-EVKIT.
This GUID is not unique (!) and thus for development only.

[Backport commit 315a93fb.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Some of the U-Boot sniff tests commands are not always available in
production firmware; take it into account and issue only warnings for those
instead of errors.

[Backport commit 4b600ff5.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Fix the minimum number of files we expect under `uefi_dump'.

There are 11 mandatory files there:

  bcfg.log
  devices.log
  devtree.log
  dh.log
  dmem.log
  dmpstore.log
  drivers.log
  ifconfig.log
  memmap.log
  pci.log
  uefi_version.log

There are also two optional files:

  map.log
  smbiosview.log

This means `uefi_dump' should have 11..13 files.

It seems that the optional `smbios.log' was not taken into account in
commit df4cb663 ("check-sr-results: adapt for ir 2.0").

See also #41.

[Backport commit 8671dfff.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

The `map.log' file is not always produced during ACS run (see #41).

This is annoying, but not part of the compliance. Thus, until this is
properly understood workaround the issue by making this file optional.

[Backport commit 6306b2ee.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

[Backport e4ec7722.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Add the GUIDs of the Rockchip TB-RK3588X and of the ArmSoM Sige7.

[Backport commit 34187e90.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Collect some more GUIDs from U-Boot v2023.01 and v2023.07.01.

[Backport commit 21bd4ab7.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

When the last 6 bytes `Node' field of a GUID has leading zeroes, they are
erroneously converted to spaces by the Guid.__str__() method; fix this.

This could cause havocs in e.g. `check-sr-results.py' when calling
`guid-tool.py' with a GUID as argument, as the GUID becomes split into two
arguments due to the spaces, ending up in a confusing error message.

[Backport commit 651b0c66.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Add a few rules to ignore some dt-validate warnings caused by U-Boot SPL
properties.

[Backport da363040.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

The warning messages formats have changed a bit with recent dt-schema tools
and the dt-parser is not able to determine the schema corresponding to a
warning anymore.
Relax the parsing a bit to repair the dt_validate_schema properties.

While at it, add a unit test.

[Backport commit 6297fabf.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Weizhao Ouyang authored May 17, 2024 and Vincent Stehle committed May 22, 2024

Since the linux kernel 6.9 released, bump the linux kernel version to
6.9 for the Devicetree schema.

Signed-off-by: Weizhao Ouyang <weizhao.ouyang@arm.com>
[Backport commit 7d2b1b79.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

check-sr-results.yaml: uname.log adapts to the 6.4 kernel

See merge request !15

Signed-off-by: Weizhao Ouyang <weizhao.ouyang@arm.com>

This gives us validation of the URIs used in our schemas.

[Backport commit a816fcf4.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Be sure to refresh the APT sources before installing packages on github CI.

This fixes the case where the cached sources are out-of-sync, which makes
the install fail.

[Backport commit 1a6384cc.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

The checkout and upload-artifact actions v3 we use in our github CI are
deprecated; refresh them to v4.

[Backport commit 7ff0f4ca.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Weizhao Ouyang authored Apr 18, 2024 and Vincent Stehle committed Apr 18, 2024

[Backport commit eb073d62.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Add a `should-contain' property, equivalent to the `must-contain' property
but issuing only a warning.
Use it in the configurations to relax our checks on SMBIOS.

Update the documentation and the schema accordingly, and add a unit test.

[Backport commit 2c3e376b.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Bump the Linux kernel sources we use for the Devicetree schemas to the
latest stable version: v6.8.1.

[Backport commit 3b2768a0.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

[Backport commit 0186a249.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Enable some more linting options and modify our python scripts accordingly.

[Backport commit e5ffd05e.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Make sure to resolve the current folder path correctly in the test, to
account for links for example.

[Backport commit 2198b9ac.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Limit the number of iterations to cleanup a line in `logreader.py'. This
greatly helps with some OS install logs, where the distro checks its
install medium and emits a ton of progress updates for example.

Add command line options to change this limit in `check-sr-results.py' and
`format-sr-results.py'.

[Backport commit 323e2ca6.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

- Fix a couple of shellcheck warnings.
- Add shellcheck to `make check' for all unit tests.
- Install shellcheck in the CI.

[Backport commit 0ca522f1.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Turn on some more warnings from pylint and fix the reported ones in our
scripts.

[Backport commit d386456e.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

- Add a `.pylint.rc' file with some pylint settings capturing where our
  scripts are right now.
- Add pylint to `make check'.
- Install pylint in the CI.

[Backport commit 2e784f57.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Force the tests run during `make check' to run after yaml files validation
and sequential tests, which are very short.
This does not change the overall test time much and avoids to wait a few
minutes to discover style errors.

[Backport commit 35cf24e4.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

IR v1.1 is no longer available for certification since publication of the
SRS v2.0 (Oct '22) and the 6 months transition period has closed (Apr '23).

We can thus issue only IR v1.2 certifications from the IR v1.x family for
now; bump the detected version accordingly for completeness.

This is similar to commit 9f2d9431 ("check-sr-results: bump ir revision
to 1.2").

[Backport commit 2e7a7f5b.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

Add more type annotations to dt-parser.py `EntryType' and rework bits to
make mypy happy.

[Backport commit 46e20377.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

- Add a `test' target, which depends on running all the tests.
- Add a `valid' target, which depends on validating all the yaml files.

This is to ease partial testing during development.

[Backport commit a6d96f05.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>

[Backport commit 2e942291.]
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>