mirror of https://github.com/xemu-project/xemu.git
tpm: remove locty argument from receive_cb
The tpm_state is passed as argument, the assert() is pointless since we give it the value of tpm_state->locty_number already. Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> Reviewed-by: Stefan Berger <stefanb@linux.vnet.ibm.com> Signed-off-by: Stefan Berger <stefanb@linux.vnet.ibm.com>
This commit is contained in:
parent
54aa36d5c8
commit
d1fd6b563d
|
@ -193,8 +193,7 @@ static void tpm_emulator_handle_request(TPMBackend *tb)
|
||||||
error_report_err(err);
|
error_report_err(err);
|
||||||
}
|
}
|
||||||
|
|
||||||
tb->recv_data_callback(tb->tpm_state, tb->tpm_state->locty_number,
|
tb->recv_data_callback(tb->tpm_state, selftest_done);
|
||||||
selftest_done);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int tpm_emulator_probe_caps(TPMEmulator *tpm_emu)
|
static int tpm_emulator_probe_caps(TPMEmulator *tpm_emu)
|
||||||
|
|
|
@ -160,9 +160,7 @@ static void tpm_passthrough_handle_request(TPMBackend *tb)
|
||||||
tb->tpm_state->locty_data,
|
tb->tpm_state->locty_data,
|
||||||
&selftest_done);
|
&selftest_done);
|
||||||
|
|
||||||
tb->recv_data_callback(tb->tpm_state,
|
tb->recv_data_callback(tb->tpm_state, selftest_done);
|
||||||
tb->tpm_state->locty_number,
|
|
||||||
selftest_done);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static void tpm_passthrough_reset(TPMBackend *tb)
|
static void tpm_passthrough_reset(TPMBackend *tb)
|
||||||
|
|
|
@ -371,14 +371,13 @@ static void tpm_tis_receive_bh(void *opaque)
|
||||||
/*
|
/*
|
||||||
* Callback from the TPM to indicate that the response was received.
|
* Callback from the TPM to indicate that the response was received.
|
||||||
*/
|
*/
|
||||||
static void tpm_tis_receive_cb(TPMState *s, uint8_t locty,
|
static void tpm_tis_receive_cb(TPMState *s,
|
||||||
bool is_selftest_done)
|
bool is_selftest_done)
|
||||||
{
|
{
|
||||||
TPMTISEmuState *tis = &s->s.tis;
|
TPMTISEmuState *tis = &s->s.tis;
|
||||||
|
uint8_t locty = s->locty_number;
|
||||||
uint8_t l;
|
uint8_t l;
|
||||||
|
|
||||||
assert(s->locty_number == locty);
|
|
||||||
|
|
||||||
if (is_selftest_done) {
|
if (is_selftest_done) {
|
||||||
for (l = 0; l < TPM_TIS_NUM_LOCALITIES; l++) {
|
for (l = 0; l < TPM_TIS_NUM_LOCALITIES; l++) {
|
||||||
tis->loc[locty].sts |= TPM_TIS_STS_SELFTEST_DONE;
|
tis->loc[locty].sts |= TPM_TIS_STS_SELFTEST_DONE;
|
||||||
|
|
|
@ -30,7 +30,7 @@
|
||||||
typedef struct TPMBackendClass TPMBackendClass;
|
typedef struct TPMBackendClass TPMBackendClass;
|
||||||
typedef struct TPMBackend TPMBackend;
|
typedef struct TPMBackend TPMBackend;
|
||||||
|
|
||||||
typedef void (TPMRecvDataCB)(TPMState *, uint8_t locty, bool selftest_done);
|
typedef void (TPMRecvDataCB)(TPMState *, bool selftest_done);
|
||||||
|
|
||||||
struct TPMBackend {
|
struct TPMBackend {
|
||||||
Object parent;
|
Object parent;
|
||||||
|
|
Loading…
Reference in New Issue