tpm: Rename tpm2_allow_extend()
When that function was introduced we were only using it to check if extending a PCR was allowed, so the name made sense. A few patches ago we used that function to reason about the EventLog creation and general usage of PCRs , so let's rename it to something more generic that makes more sense in all contexts. Signed-off-by: Raymond Mao <raymond.mao@linaro.org> Signed-off-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>
This commit is contained in:
@@ -732,12 +732,13 @@ u16 tpm2_algorithm_to_len(enum tpm2_algorithms algo);
|
||||
*/
|
||||
|
||||
/**
|
||||
* tpm2_allow_extend() - Check if extending PCRs is allowed and safe
|
||||
* tpm2_check_active_banks() - Check if the active PCR banks are supported by
|
||||
* our configuration
|
||||
*
|
||||
* @dev: TPM device
|
||||
* Return: true if allowed
|
||||
*/
|
||||
bool tpm2_allow_extend(struct udevice *dev);
|
||||
bool tpm2_check_active_banks(struct udevice *dev);
|
||||
|
||||
/**
|
||||
* tpm2_is_active_bank() - check the pcr_select. If at least one of the PCRs
|
||||
|
@@ -197,7 +197,7 @@ u32 tpm2_pcr_extend(struct udevice *dev, u32 index, u32 algorithm,
|
||||
if (!digest)
|
||||
return -EINVAL;
|
||||
|
||||
if (!tpm2_allow_extend(dev)) {
|
||||
if (!tpm2_check_active_banks(dev)) {
|
||||
log_err("Cannot extend PCRs if all the TPM enabled algorithms are not supported\n");
|
||||
return -EINVAL;
|
||||
}
|
||||
@@ -896,7 +896,7 @@ u16 tpm2_algorithm_to_len(enum tpm2_algorithms algo)
|
||||
return 0;
|
||||
}
|
||||
|
||||
bool tpm2_allow_extend(struct udevice *dev)
|
||||
bool tpm2_check_active_banks(struct udevice *dev)
|
||||
{
|
||||
struct tpml_pcr_selection pcrs;
|
||||
size_t i;
|
||||
|
Reference in New Issue
Block a user