Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Jun 16, 2025

Add identification for the key used in SOF-2.13 to sign sof-ptl.ri and related files.

Add identification for the key used in SOF-2.13 to sign sof-ptl.ri
and related files.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds identification for the key used in SOF-2.13 to sign sof-ptl.ri and related files

  • Introduce PTL_INTEL_PROD_KEY byte fingerprint constant
  • Register PTL_INTEL_PROD_KEY in the key description mapping
Comments suppressed due to low confidence (2)

tools/sof_ri_info/sof_ri_info.py:511

  • [nitpick] Consider adding a brief comment above this constant to describe its origin (e.g., version range or purpose) for future maintainers.
PTL_INTEL_PROD_KEY = bytes([0x89, 0xa8, 0x6a, 0xff, 0x6b, 0x1a, 0x78, 0xae,

tools/sof_ri_info/sof_ri_info.py:653

  • Add a unit test to verify that PTL_INTEL_PROD_KEY is recognized and maps to 'PTL Intel prod key', ensuring the new key is handled correctly.
PTL_INTEL_PROD_KEY : 'PTL Intel prod key',

0x6c, 0xa0, 0x50, 0x65, 0x92, 0x39, 0x96, 0x34,
0x61, 0xa4, 0xeb, 0x6b, 0x11, 0xd9, 0xbe, 0xb8])

PTL_INTEL_PROD_KEY = bytes([0x89, 0xa8, 0x6a, 0xff, 0x6b, 0x1a, 0x78, 0xae,
Copy link

Copilot AI Jun 16, 2025

Choose a reason for hiding this comment

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

[nitpick] This block is very long; you might extract the fingerprint to a separate file or use a hex string with bytes.fromhex() to improve readability.

Copilot uses AI. Check for mistakes.
@lgirdwood lgirdwood merged commit 8c979ad into thesofproject:main Jun 18, 2025
42 of 48 checks passed
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.

3 participants