Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Mar 9, 2022

Depends-on: zephyrproject-rtos/zephyr#43611
Keeping as draft until the Zephyr PR is merged.

Use intel_adsp_cavs20_jsl Zephyr board when building for
Intel Jasper Lake.

BugLink: #4539
Signed-off-by: Kai Vehmanen kai.vehmanen@linux.intel.com

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Depending on how long the Zephyr PR is going to take, you can add it to SUPPORTED_PLATFORMS right now. This will enable other people to try it now without breaking CI.

Same thing with line 84 in the python script.

@kv2019i kv2019i marked this pull request as ready for review March 11, 2022 08:03
@kv2019i
Copy link
Collaborator Author

kv2019i commented Mar 11, 2022

SOFCI TEST

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this one pass shellcheck? Sometimes it's smart enough to know quotes are not needed, sometimes it's not.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's shellcheck? :) I copied the tgl-h check and that fails as well. I'll update.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's shellcheck? :)

:-)

The best thing since sliced bread.

The only way to keep shell scripts relatively maintainable and trouble-free.

Use intel_adsp_cavs20_jsl Zephyr board when building for
Intel Jasper Lake.

BugLink: thesofproject#4539
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
@kv2019i kv2019i force-pushed the topic/fix-jsl-support branch from 56d42a9 to 439a0f4 Compare March 11, 2022 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants