qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH V2 4/4] hw/tpm: fix usage of bool in tpm-tis.c


From: Daniel P . Berrangé
Subject: Re: [Qemu-devel] [PATCH V2 4/4] hw/tpm: fix usage of bool in tpm-tis.c
Date: Thu, 7 May 2020 12:52:01 +0100
User-agent: Mutt/1.13.4 (2020-02-15)

ping, again, for either qemu-trivial or TPM maintainers to take this
patch.

The other three in this series are merged, but this TPM patch is
still pending.

On Tue, Mar 26, 2019 at 09:55:24AM -0400, Stefan Berger wrote:
> On 3/24/19 3:41 PM, Thomas Huth wrote:
> > 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>
> Reviewed-by: Stefan Berger <address@hidden>
> > 
> 
> 

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




reply via email to

[Prev in Thread] Current Thread [Next in Thread]