[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH V2 4/4] hw/tpm: fix usage of bool
From: |
Thomas Huth |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH V2 4/4] hw/tpm: fix usage of bool in tpm-tis.c |
Date: |
Sun, 24 Mar 2019 20:41:15 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.0 |
On 23/03/2019 15.26, Jafar Abdi wrote:
> Clean up wrong usage of FALSE and TRUE in places that use "bool" from
> stdbool.h.
>
> FALSE and TRUE (with capital letters) are the constants defined by glib for
> being used with the "gboolean" type of glib. But some parts of the code also
> use
> TRUE and FALSE for variables that are declared as "bool" (the type from
> <stdbool.h>).
>
> Signed-off-by: Jafar Abdi <address@hidden>
> ---
> hw/tpm/tpm_tis.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/tpm/tpm_tis.c b/hw/tpm/tpm_tis.c
> index fd183e8..c1eb094 100644
> --- a/hw/tpm/tpm_tis.c
> +++ b/hw/tpm/tpm_tis.c
> @@ -611,7 +611,7 @@ static void tpm_tis_mmio_write(void *opaque, hwaddr addr,
> while ((TPM_TIS_IS_VALID_LOCTY(s->active_locty) &&
> locty > s->active_locty) ||
> !TPM_TIS_IS_VALID_LOCTY(s->active_locty)) {
> - bool higher_seize = FALSE;
> + bool higher_seize = false;
>
> /* already a pending SEIZE ? */
> if ((s->loc[locty].access & TPM_TIS_ACCESS_SEIZE)) {
> @@ -621,7 +621,7 @@ static void tpm_tis_mmio_write(void *opaque, hwaddr addr,
> /* check for ongoing seize by a higher locality */
> for (l = locty + 1; l < TPM_TIS_NUM_LOCALITIES; l++) {
> if ((s->loc[l].access & TPM_TIS_ACCESS_SEIZE)) {
> - higher_seize = TRUE;
> + higher_seize = true;
> break;
> }
> }
>
Reviewed-by: Thomas Huth <address@hidden>
- Re: [Qemu-trivial] [Qemu-devel] [PATCH V2 4/4] hw/tpm: fix usage of bool in tpm-tis.c,
Thomas Huth <=