On 3/12/26 12:16 AM, Arun Menon wrote:
> - Add new fields to the CRB Interface Identifier and the CRB
> Control Start registers.
> - CRB_CTRL_START now has 2 new settings, that can be toggled using the
> nextChunk and crbRspRetry bits.
> - CapCRBChunk bit (10) was Reserved1 previously. The field is reused in
> this revision of the specification.
> - Refer to section 6.4.2.2 of [1]
>
> [1] https://trustedcomputinggroup.org/wp-content/uploads/PC-Client-Specific-Platform-TPM-Profile-for-TPM-2p0-v1p07_rc1_121225.pdf
>
> Signed-off-by: Arun Menon <armenon@redhat.com>
> ---
> hw/tpm/tpm_crb.c | 3 +++
> include/hw/acpi/tpm.h | 5 ++++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/hw/tpm/tpm_crb.c b/hw/tpm/tpm_crb.c
> index 8723536f93..0a1c7ecdc6 100644
> --- a/hw/tpm/tpm_crb.c
> +++ b/hw/tpm/tpm_crb.c
> @@ -59,6 +59,7 @@ DECLARE_INSTANCE_CHECKER(CRBState, CRB,
> #define CRB_INTF_CAP_FIFO_NOT_SUPPORTED 0b0
> #define CRB_INTF_CAP_CRB_SUPPORTED 0b1
> #define CRB_INTF_IF_SELECTOR_CRB 0b1
> +#define CRB_INTF_CAP_CRB_CHUNK 0b1
>
> #define CRB_CTRL_CMD_SIZE (TPM_CRB_ADDR_SIZE - A_CRB_DATA_BUFFER)
>
> @@ -262,6 +263,8 @@ static void tpm_crb_reset(void *dev)
> CapCRB, CRB_INTF_CAP_CRB_SUPPORTED);
> ARRAY_FIELD_DP32(s->regs, CRB_INTF_ID,
> InterfaceSelector, CRB_INTF_IF_SELECTOR_CRB);
> + ARRAY_FIELD_DP32(s->regs, CRB_INTF_ID,
> + CapCRBChunk, CRB_INTF_CAP_CRB_CHUNK);
> ARRAY_FIELD_DP32(s->regs, CRB_INTF_ID,
> RID, 0b0000);
> ARRAY_FIELD_DP32(s->regs, CRB_INTF_ID2,
> diff --git a/include/hw/acpi/tpm.h b/include/hw/acpi/tpm.h
> index d2bf6637c5..03d452d2b5 100644
> --- a/include/hw/acpi/tpm.h
> +++ b/include/hw/acpi/tpm.h
> @@ -149,7 +149,7 @@ REG32(CRB_INTF_ID, 0x30)
> FIELD(CRB_INTF_ID, InterfaceVersion, 4, 4)
> FIELD(CRB_INTF_ID, CapLocality, 8, 1)
> FIELD(CRB_INTF_ID, CapCRBIdleBypass, 9, 1)
> - FIELD(CRB_INTF_ID, Reserved1, 10, 1)
> + FIELD(CRB_INTF_ID, CapCRBChunk, 10, 1)
> FIELD(CRB_INTF_ID, CapDataXferSizeSupport, 11, 2)
> FIELD(CRB_INTF_ID, CapFIFO, 13, 1)
> FIELD(CRB_INTF_ID, CapCRB, 14, 1)
> @@ -168,6 +168,9 @@ REG32(CRB_CTRL_STS, 0x44)
> FIELD(CRB_CTRL_STS, tpmIdle, 1, 1)
> REG32(CRB_CTRL_CANCEL, 0x48)
> REG32(CRB_CTRL_START, 0x4C)
> + FIELD(CRB_CTRL_START, invoke, 0, 1)
> + FIELD(CRB_CTRL_START, crbRspRetry, 1, 1)
> + FIELD(CRB_CTRL_START, nextChunk, 2, 1)
> REG32(CRB_INT_ENABLED, 0x50)
> REG32(CRB_INT_STS, 0x54)
> REG32(CRB_CTRL_CMD_SIZE, 0x58)
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>