Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(401)

Issue 1761006: Some additions to facilitate inclusion in BIOS. (Closed)

Created:
10 years, 8 months ago by Luigi Semenzato
Modified:
9 years, 7 months ago
Reviewers:
gauravsh
CC:
chromium-os-reviews_chromium.org, gauravsh
Visibility:
Public.

Description

Some additions to facilitate inclusion in BIOS. 1. Copy of generated file structures.h (in structures.h.snapshot). 2. Ifdef'ed code that shows how the TPM would be called through the EFI driver.

Patch Set 1 #

Patch Set 2 : fix whitespace #

Total comments: 4

Patch Set 3 : add ifdef comments and fix typo in comment #

Patch Set 4 : Introduce BIOS cpp variable #

Unified diffs Side-by-side diffs Delta from patch set Stats (+106 lines, -4 lines) Patch
A src/platform/tpm_lite/src/tlcl/structures.h.snapshot View 1 2 1 chunk +79 lines, -0 lines 0 comments Download
M src/platform/tpm_lite/src/tlcl/tlcl.c View 1 2 3 8 chunks +27 lines, -4 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Luigi Semenzato
Hi Gaurav, please let me know if this would do.
10 years, 8 months ago (2010-04-23 01:08:27 UTC) #1
gauravsh
lgtm http://codereview.chromium.org/1761006/diff/2001/3002 File src/platform/tpm_lite/src/tlcl/tlcl.c (right): http://codereview.chromium.org/1761006/diff/2001/3002#newcode152 src/platform/tpm_lite/src/tlcl/tlcl.c:152: status = tgc_pass_through_to_tpm(tcg_protocol, tcg_pass_thorugh http://codereview.chromium.org/1761006/diff/2001/3002#newcode163 src/platform/tpm_lite/src/tlcl/tlcl.c:163: #endif just ...
10 years, 8 months ago (2010-04-23 01:16:23 UTC) #2
Luigi Semenzato
Thank you Gaurav, I made those changes. http://codereview.chromium.org/1761006/diff/2001/3002 File src/platform/tpm_lite/src/tlcl/tlcl.c (right): http://codereview.chromium.org/1761006/diff/2001/3002#newcode163 src/platform/tpm_lite/src/tlcl/tlcl.c:163: #endif On ...
10 years, 8 months ago (2010-04-23 15:55:59 UTC) #3
Luigi Semenzato
Any more comments on this? On 2010/04/23 15:55:59, Luigi Semenzato wrote: > Thank you Gaurav, ...
10 years, 8 months ago (2010-04-26 21:19:19 UTC) #4
gauravsh
10 years, 8 months ago (2010-04-26 21:41:21 UTC) #5
This looks good. LGTM.

Powered by Google App Engine
This is Rietveld 408576698