diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2017-11-07 08:00:21 -0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-11-07 17:52:15 -0800 |
| commit | 0e7186422fea5046f11d15fdcf0647887b2aa4f9 (patch) | |
| tree | 3f6684548de86e8e858f54455fce4b1a89511bae /common | |
| parent | 3919001a390f593ba8e022f223bf1e43b7dbc0a2 (diff) | |
| download | chrome-ec-0e7186422fea5046f11d15fdcf0647887b2aa4f9.tar.gz | |
cr50: fix event definition collision
Events used when TPM task is running are defined in two different
places, one of them shared with other boards running on H1.
Let's avoid collision by redefining Cr50 only events to be different
from shared ones used by dcrypto.
BRANCH=cr50
BUG=b:68729265
TEST=verified that there is no more 'tpm_reset_request: already
scheduled' messages generated when TPM is reset when performing
long dcrypto operation.
Change-Id: Ic9517fa98be21f3ef5f19b82c593d96b0ddbaf6b
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/756914
Reviewed-by: Nagendra Modadugu <ngm@google.com>
Reviewed-by: Duncan Laurie <dlaurie@google.com>
Reviewed-by: Mary Ruthven <mruthven@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/tpm_registers.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/common/tpm_registers.c b/common/tpm_registers.c index 71f0fb4a28..0596c83aa3 100644 --- a/common/tpm_registers.c +++ b/common/tpm_registers.c @@ -669,10 +669,13 @@ static void call_extension_command(struct tpm_cmd_header *tpmh, } #endif -/* Event (to TPM task) to request reset, or (from TPM task) on completion. */ -#define TPM_EVENT_RESET TASK_EVENT_CUSTOM(1 << 0) -#define TPM_EVENT_COMMIT TASK_EVENT_CUSTOM(1 << 1) -#define TPM_EVENT_ALT_EXTENSION TASK_EVENT_CUSTOM(1 << 2) +/* + * Events used on the TPM task context. Make sure there is no collision with + * event(s) defined in chip/g/dcrypto/dcrypto_runtime.c + */ +#define TPM_EVENT_RESET TASK_EVENT_CUSTOM(1 << 1) +#define TPM_EVENT_COMMIT TASK_EVENT_CUSTOM(1 << 2) +#define TPM_EVENT_ALT_EXTENSION TASK_EVENT_CUSTOM(1 << 3) /* * Result of executing of the TPM command on the alternative path, could have |