Xilinx / Xilinx/embeddedsw

TPM 2.0 Support Integration in xilinx fsbl

Open
#207 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
HTML
Stars
1.2k
Forks
1.1k
PR merge metrics
No merged PRs in 30d

Description

I'm using xlnx_rel_v2021.2 version of zynqmp fsbl to do measured boot using TPM.
I have enabled the TPM support available in the zynqmp fsbl. But I'm facing some issues with the spi communication APIs.
The SpiInstance structure which is static is getting cleared once the TpmRxBuffer array is declared inside the Xfsbl_TpmTransfer fuction. Due to this the program get stuck at Xil_AssertNonvoid(InstancePtr->IsReady == XIL_COMPONENT_IS_READY); inside XSpiPs_PolledTransfer().
I'm using zcu67dr RFSoC and the fsbl is running in OCM. I'm suspecting the limitted size of OCM is causing the issue.
Can someone share some details about the memory constraints to be taken care for fsbl.
where can I test the Optional TPM 2.0 support included in xilinx fsbl xlnx_rel_v2021.2.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing Xfsbl_TpmTransfer, the static SpiInstance, TpmRxBuffer, and the XSpiPs_PolledTransfer assertion, then review the FSBL OCM memory layout for stack or storage limits. Reproduce the failure on the ZCU67DR with xlnx_rel_v2021.2 and determine whether the integration needs a memory-layout change or a documented constraint.

Written by the indexing model from the issue text.

Assessment

Domain
embedded-iot, security
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.