Skip to content

Commit

Permalink
xtensa-build-zephyr.py: move PTL out of "all" and into "extra".
Browse files Browse the repository at this point in the history
As reported in thesofproject#9262, this fixes the --all option.

Fixes commit 05bfc36 ("ptl: Add initial PTL configuration")

Signed-off-by: Marc Herbert <[email protected]>
  • Loading branch information
marc-hb authored and Eddy Hsu committed Jul 15, 2024
1 parent b267a71 commit 9190613
Showing 1 changed file with 18 additions and 16 deletions.
34 changes: 18 additions & 16 deletions scripts/xtensa-build-zephyr.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,23 @@ class PlatformConfig:
aliases: list = dataclasses.field(default_factory=list)
ipc4: bool = False

# These cannot be built by everyone out of the box yet.
# For instance: there's no open-source toolchain available for them yet.
extra_platform_configs = {
"ptl" : PlatformConfig(
"intel", "intel_adsp/ace30_ptl",
f"RI-2022.10{xtensa_tools_version_postfix}",
"ace30_LX7HiFi4_PIF",
ipc4 = True
),
"ptl-sim" : PlatformConfig(
"intel", "intel_adsp/ace30_ptl_sim",
f"RI-2022.10{xtensa_tools_version_postfix}",
"ace30_LX7HiFi4_PIF",
ipc4 = True
),
}

# These can all be built out of the box. --all builds all these.
# Some of these values are duplicated in sof/scripts/set_xtensa_param.sh: keep them in sync.
platform_configs_all = {
Expand Down Expand Up @@ -120,18 +137,6 @@ class PlatformConfig:
"ace10_LX7HiFi4_2022_10",
ipc4 = True
),
"ptl" : PlatformConfig(
"intel", "intel_adsp/ace30_ptl",
f"RI-2022.10{xtensa_tools_version_postfix}",
"ace30_LX7HiFi4_PIF",
ipc4 = True
),
"ptl-sim" : PlatformConfig(
"intel", "intel_adsp/ace30_ptl_sim",
f"RI-2022.10{xtensa_tools_version_postfix}",
"ace30_LX7HiFi4_PIF",
ipc4 = True
),

# NXP platforms
"imx8" : PlatformConfig(
Expand Down Expand Up @@ -160,10 +165,6 @@ class PlatformConfig:
),
}

# These cannot be built out of the box yet
extra_platform_configs = {
}

platform_configs = platform_configs_all.copy()
platform_configs.update(extra_platform_configs)

Expand Down Expand Up @@ -334,6 +335,7 @@ def parse_args():

args = parser.parse_args()

# This does NOT include "extra", experimental platforms.
if args.all:
args.platforms = list(platform_configs_all)

Expand Down

0 comments on commit 9190613

Please sign in to comment.