circleci,modules/coreboot,doc: share crossgcc toolchain across Dasharo 24.12 forks; blobs: add lib.sh and unify hash pre-checks

[tlaurion]

dasharo_nv4x is the toolchain provider. dasharo_v56, dasharo_msi_z690, dasharo_msi_z790 consume its crossgcc via the second coreboot_module() argument. All Dasharo boards except novacustom-nv4x_adl are build jobs (no cache) -- they get crossgcc from workspace and clone fork source fresh.

Fix pre-existing hyphen/underscore variable name typo in modules/coreboot that made the toolchain-module nobuild override a silent no-op. Fix pre-existing portability issue: printf instead of echo -e.

Rename CI jobs for clarity: [cache keys], [ME GBE IFD], [cross compiler], [seed:coreboot-VERSION]. Update doc/circleci.md to document the cache model, job types, and Dasharo shared toolchain design.

Add blobs/lib.sh with check_outputs() unifying hash verification across all 10 blob download scripts. All scripts now check output file hashes before and after processing, reporting CHECKING/OK/MISSING/HASH MISMATCH per file. Fix fallthrough bugs, variable name mismatches, and missing pre-checks. (2 commits)

[Copilot]

Pull request overview

This PR reduces redundant CI work for Dasharo coreboot forks by introducing a shared “dasharo_common” crossgcc toolchain module and restructuring the CircleCI workflow so multiple Dasharo forks reuse a single toolchain build via workspace chaining.

Changes:

• Added coreboot-dasharo_common and updated four Dasharo coreboot fork modules to consume it as their toolchain.
• Updated CircleCI workflow so novacustom-nv4x_adl builds the shared toolchain and other Dasharo seed jobs depend on it.
• Updated CircleCI documentation to describe the shared toolchain model and the re-verification procedure based on util/crossgcc/sum/ diffs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
modules/coreboot	Adds a shared Dasharo toolchain module and points Dasharo forks to reuse it via the coreboot_module() toolchain argument.
doc/circleci.md	Documents the Dasharo shared toolchain approach and updates the pipeline shape/edit map accordingly.
.circleci/config.yml	Reworks the Dasharo seed job dependency graph so only one crossgcc toolchain build is performed and reused via workspace.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[tlaurion]

Will see what circleci gives, but yeah, that's why we build 4 buildstacks in master for dasharo forks per their different FSP requirements/changes per bords family, see comment above replicated below in master:

# Dasharo forks cannot share toolchains with upstream coreboot releases because
# Dasharo uses custom FSP packages (3rdparty/dasharo-blobs/) with different/missing
# headers vs upstream (e.g. MemInfoHob.h, FirmwareVersionInfoHob.h).

Seems like it worked locally? Actually, we can share build system and FSP are in specialized repo for board builds!

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[tlaurion]

ok enough. merging, its clear enough and already a big improvement.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

blobs/xx30/download_clean_me.sh:16

• usage/help text is now inaccurate: the script no longer parses a -m <me_cleaner> option, and it also requires COREBOOT_DIR to be set (plus optionally takes an output dir as the first positional arg). Update the usage/help output (and/or reintroduce -m parsing) so users can run the script correctly.

function usage {
  echo "Usage: $0 -m <me_cleaner>(optional)"
}

ME_BIN_HASH="c140d04d792bed555e616065d48bdc327bb78f0213ccc54c0ae95f12b28896a4"

if [[ "${BASH_SOURCE[0]}" == "$0" ]]; then
  if [[ "${1:-}" == "--help" ]]; then
    usage
  else
    if [[ -z "${COREBOOT_DIR}" ]]; then
      echo "ERROR: No COREBOOT_DIR variable defined."
      exit 1

[Copilot]

Pull request overview

Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.